07.CodeRulerReview

Michal Zielinski & Chiao-Fen Zielinski's Code Ruler Review

File	Lines	Violation	Comments
MyRuler.java	45, 46, 48, *	ICS-SE-Eclipse-2	2 space indent
MyRuler.java	53	spelling error	"based" instead of "baced"
MyRuler.java	48, 49, 50, *	EJS-25	Capital letter for 2nd, 3rd, 4th…word of variable
MyRuler.java	155	EJS-22	Capital letter for 2nd, 3rd, 4th…word of method
MyRuler.java	140, 165	EJS-5	Indent nested code
MyRuler.java	137, 138, 159, *	EJS-9	Use meaningful variable names
MyRuler.java	136	EJS-28	First for loop should start with "i"
MyRuler.java	30, 153	EJS-42	Descriptions given for only some parameters
MyRuler.java	156, 162, 171	EJS-76	Use { }

Your code is not overly complex in terms of the amount of loops and thus, should execute relatively quickly. With that in mind, you could implement a more sophisticated algorithm for your peasants to acquire land and still be under the 0.5 second limit per turn. Maybe have each of them look around for the closest land controlled by another ruler or that is unclaimed. That way they all won't be traveling in one group as well.