Create a gist now

Instantly share code, notes, and snippets.

@scmy /Codereview.md
Last active Apr 28, 2016

What would you like to do?
Codereview Vorlage

Code Reviews Protokolle

Dies ist nicht der Ort, um die Protokolle zu kopieren - sondern dient dem Dozenten (und den Studenten) als Gedankenstütze und Ideentopf. Die Gruppen bekommen indivuell an den Sitzungen Aufträge.

Code Review 1

Traktanden

Kennenlernen
Erklären Vorgehen CodeReview
Anschauen und erklären des Projektes und des Codes.
Erwartungen, Fragen
Smalltalk

Aufträge CR1:

  1. Clean Code Kap 1-5 lesen
  2. Clean Code Sheet studieren
  3. 3-4 Punkte für das Team festlegen und nächstes Mal kommuzieren.
  4. package Struktur bereinigen

Diese Aufträge müssen in github erfasst werden und mit Milestone "CodeReview1" markiert werden. Als DueDate soll der Sonntag vor dem 2. Reviews eingestellt werden. Bewertet werden diese Punkte im CodeReview2.

Code Review 2

Ideen für Aufräge:

  • Das Kapitel 7 aus dem Buch "Effective Java" lesen und eines der Item (z.B. 38 und 44) an 1-2 Klassen anwenden (jedes Gruppenmitglied).
  • Etwas aus den vorgetragenen Punkten in Code umsetzen (jedes Gruppenmitglied)
  • Packages-Struktur gemäss Vorgaben umsetzen

Achtung: alles muss mit Issue geschehen, sonst wird es nicht bewertet.

Code Review 3

keine Vorlage. Aufräge wurden mündliche aufgegeben.

Code Review 4

Ideen für Aufräge:

  • Idividuell zu besprechende Aufträge
  • Clean Code - A Handbookof Agile Software Craftsmanship - Chapter 10 Classes lesen und 2 Dinge davon umsetzen
  • möglichst viele der bishher besprochenen Prinzipien umsetzen und wie bisher gut im github dokumentieren.

Benotung

Um zu benoten mache ich:

  • Stichprobem im Code
  • sehe ich die Code-Reviews Milestones, ihre Issues und commits an und vollziehe nach, was überlegt worden ist
  • Kontrolle, ob auch wirklich alle vom Team sich etwas überlegt und commitet haben.
  • lass ich mich durch neue, weitergehende Überlegungen überraschen...

Das Code Review wird während der Prüfungswoche gemäss separatem Plan benotet.

Sign in to join this conversation on GitHub.