	Fundamental Programming Concepts Summer 2000

Lab 2 Grading

Part	Points
Sum/Product/Average program	15
Correctness 2 points each for calculating correct sum, product, average (6 points) 1 point each for making sum an int, product an int, average a double, and all 3 inputs ints (4 points) Style 2 points for comment at head of class (lab, name, NetID, date) 2 points for correct indentation and consistent bracing 1 point for variable specifications Other -5 for not turning in .mcp file No credit if .java file is missing
Distance program	15
Correctness 3 points for correctly calculating distance using Math methods 1 point for using doubles for all variables 1 point for correct output (has both points in the output along with the distance) Calculation method 1 point for correct signature -- static double name(double, double, double, double) 1 point for returning correctly 3 points for doing all calculation in this method (i.e., none in main) Style 2 points for comment at head of class (lab, name, NetID, date) 2 points for correct indentation and consistent bracing 1 point for variable specifications Other -5 for not turning in .mcp file No credit if .java file is missing
Total	30

General comments:

The import statement goes at the very top of the file, even before your name/userid comment.
Keep your lines of code reasonably short. Code that goes off the side of the screen is hard to read and thus very poor style.
Judicious use of blank lines makes code much more readable. You don't need too much white space, but a little bit can help a lot.
It is often poor style to compute a value in an output statement, as in this example:
System.out.println("Product: " + (i*j*k));
Try storing the result of a computation in a well-named variable and then printing, as in:
System.out.println("Product: " + product);This is often much easier to read.
The point of a method like computeDistance (in part II) is to separate calculation from the input and output sections of your program. It defeats the purpose of good method structure to put all three tasks --input, processing, and output-- into computeDistance.