java icon indicating copy to clipboard operation
java copied to clipboard

Adding analyzer feedback for `captains-log` concept exercise

Open manumafe98 opened this issue 1 year ago • 2 comments

pull request

closes #2674

Reviewer Resources:

Track Policies

manumafe98 avatar Mar 22 '24 18:03 manumafe98

I've been going through the community solutions and these solutions stood out to me:

Solution Observations
Link Hard-coding the number of planet classes (10) instead of using the length of the array, could be an actionable comment.
Link This solution uses a Java version >= 17, judging by the usage of the RandomGenerator.nextDouble(double) method added in Java 17. However, those Java versions also contain RandomGenerator.nextDouble(double,double) and RandomGenerator.nextInt(int,int) which support passing both a lower and upper bound, which makes the solution a bit easier to read. Could be an informative comment.

sanderploegsma avatar Apr 04 '24 08:04 sanderploegsma

I've been going through the community solutions and these solutions stood out to me:

Solution Observations Link Hard-coding the number of planet classes (10) instead of using the length of the array, could be an actionable comment. Link This solution uses a Java version >= 17, judging by the usage of the RandomGenerator.nextDouble(double) method added in Java 17. However, those Java versions also contain RandomGenerator.nextDouble(double,double) and RandomGenerator.nextInt(int,int) which support passing both a lower and upper bound, which makes the solution a bit easier to read. Could be an informative comment.

I liked that options, seems that they are more appropriate and can actually help the student!

manumafe98 avatar Apr 04 '24 13:04 manumafe98