Code review - MarcSelles/BeatYourBucketAndroid GitHub Wiki

Code review 1 door John Wesselingh

Documentation

• Names 3
• Headers 3
• Comments 1

Presentation

• Layout 2
• Formatting 2

Algorithms

• Flow 3
• Expressions 3

Structure

• Decomposition 3
• Modularization 3

Opmerkingen

• De code is goed geordend met goede namen voor de variabelen, soms zouden er wel wat meer comments bij mogen. Ik denk ook dat er een risico is op dubbele code als je de JSON gaat inladen van de verschillende websites. Verder is de opbouw wel duidelijk.

Mijn Observaties

• Snapte de structuur wel in 1 keer

Code review 2 door Casper Broertjes

Comments

Goede comments, alleen bij lange comments kan er beter /* gebruikt worden.

Namen

Namen zijn gewoon logisch. Niet extra variabele gebruikt.

Structuur

Code is goed in blokken opgedeeld. Veel nuttige comments

Layout

Duidelijk en consequent qua indentatie ed.

##Code Review 3 door ziekte niet kunnen doen