CentralValleyModeling / CalLiteGUI

CalLite GUI-Model package. This Repo is for GUI development.
2 stars 3 forks source link

Java coding conventions ignored #123

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Encountered this while debugging - makes code base readability that much more 
difficult.

public static void CheckForScenarioChange(SwingEngine swix, 
DataFileTableModel[] dTableModels, Boolean[] RegUserEdits,
            GUILinks gl)

// get table values.
        final String NL = System.getProperty("line.separator");

In Java, methods and fields begin in *lower case* and use mixedCamelCase.

should be 

public static void checkForScenarioChange(blah,..)
String nl

If you want to use uppercase method names, switch to C#

Original issue reported on code.google.com by fitzmaur...@gmail.com on 19 Mar 2013 at 6:56

GoogleCodeExporter commented 9 years ago
Stet: OK to use upper case for NL since it is a constant:

final String NL;

Original comment by fitzmaur...@gmail.com on 19 Mar 2013 at 7:42

GoogleCodeExporter commented 9 years ago

Original comment by Tad.Slawecki@gmail.com on 30 May 2013 at 1:43

GoogleCodeExporter commented 9 years ago

Original comment by Tad.Slawecki@gmail.com on 30 May 2013 at 1:58