The Java Critiquer: Incremental Development of Educational Critiquing Systems

Lin Qiu

Enter code

Automatic critiquing

Review critiques

Manual critiquing

Return code

public void increaseRadius() {radius = radius + 1;

diameter = radius*2;

} public boolean isValid() { if (radius > 0)

return true;else

return false;

}

Java has a simpler way to write this. Use ++radius;

You never need to write an if to return true in one case and false inthe other. Just write return (radius >0); instead.

Use more descriptive variable names.

int x;

Don't cram things together without spaces, ex. x+y*z

Critiquing SystemHuman Critiquer

Realize

Familiarize

Publicize

Optimize

Generalize

Add a critique

Refine the critique

Add a pattern

Refine the pattern

Share the critique and pattern

Develop automation

Provide public access

Collect critiques

Improve automation

Improve critiques

Incremental authoring

accumulating

intelligence

Critique Authoring

Pattern Authoring