156 lines
8.7 KiB
Plaintext
156 lines
8.7 KiB
Plaintext
Auf den folgenden Seiten sind die einzelnen teaminternen Codereviews aufgelistet. Jedes einzelne Treffen wird in einer eigenen Tabelle gef"uhrt.
|
|
|
|
\paragraph{Teaminterner Codereview - 21.11.2011}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 1 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & Einlesen in den Code \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & - Code sehr un"ubersichtlich \newline - fehlende Kommentare \newline - doppelte Klassen auf mehrere Ordner verteilt \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & der Alle Dateien innerhalb des Projekts \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & Ausfindig machen, welche Klassen weggelassen werden k"onnen \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & Batu, Gebhardt, Murati, Scholz \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|
|
|
|
\paragraph{Teaminterner Codereview - 01.12.2011}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 2 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & Aufr"aumen des Codes \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & Das Projekt beinhaltet nicht verwendete Klassen \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & Alle Dateien innerhalb der Ordner Classes und Templates \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & Verbesserung der Projektstruktur durch entfernen von nicht verwendeten Klassen \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & Batu, Gebhardt, Murati, Scholz \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|
|
|
|
\paragraph{Teaminterner Codereview - 15.12.2011}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 3 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & SQL-Abfragen und JSON Format \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & - Code enth"alt SQL-Abfragen, die mit Hilfe von Propel ersetzt werden sollen \newline - JSON Format entspricht nicht dem neuen Datenformat \newline - in den Templates sind HTML-, JavaScript- und PHP-Code nicht getrennt \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & Alle Dateien innerhalb der Ordner Templates und Classes \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & SQL-Abfragen aus den betroffenen Dateien rausschreiben (Aufgabe 1) \newline rausgeschriebene SQL-Abfragen mit Propel realisieren (Aufgabe 2) \newline JSON Format anpassen (Aufgabe 3) \newline Strukturieren des Codes in den Templates mit Hilfe von Platzhaltern (Aufgabe 4) \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & Aufgabe 1: Batu und Murati \newline Aufgabe 2: Gebhardt \newline Aufgabe 3: Scholz \newline Aufgabe 4: Batu und Murati \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|
|
|
|
\paragraph{Teaminterner Codereview - 09.01.2012}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 4 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & JSON, View \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & - Umsetzung des neuen JSON Format noch nicht abgeschlossen \newline - View kommt mit den Platzhaltern in den Template Dateien nicht zurecht \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & Alle Dateien innerhalb der Ordner Templates, view und json \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & neues JSON Format umsetzen (Aufgabe 1) \newline neue View schreiben, die mit Platzhaltern umgehen kann (Aufgabe 2) \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & Aufgabe 1: Scholz \newline Aufgabe 2: Batu, Gebhardt, Murati \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|
|
|
|
\paragraph{Teaminterner Codereview - 23.01.2012}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 5 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & API \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & API ist mit der neuen Datenbank nicht kompatibel \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & Alle Dateien im Ordner api \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & API an die neue Datenbankstruktur anpassen (Aufgabe 1) \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & Aufgabe 1: Batu, Gebhardt, Murati, Scholz \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|
|
|
|
\paragraph{Teaminterner Codereview - 07.02.2012}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 6 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & API, Propel und Datenbank, Benutzerbereich \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & - Propel kann mit den Tabellennamen nicht umgehen \newline - der Benutzerbereich enth“alt Fehler (Daten editieren und l"oschen) \newline - die Heatmap wird nicht geclustert \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & Alle Dateien innerhalb der Ordner propel, user und cluster \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & Tabellennamen in der Datenbank um"andern und Propel neu generieren (Aufgabe 1) \newline Fehler im Benutzerbereich beseitigen (Aufgabe 2) \newline Heatmap geclustert anzeigen (Aufgabe 3) \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & Aufgabe 1: Gebhardt und Scholz \newline Aufgabe 2: Batu und Murati \newline Aufgabe 3: Batu, Murati und Scholz \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|
|
|
|
|
|
\paragraph{Teaminterner Codereview - 21.02.2012}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 7 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & Fehler auf der Webseite \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & - Sensor-Diagramme werden nicht mehr angezeigt \newline - Bedienung des Isolationsmodus nicht benutzerfreundlich \newline - nicht alle Pfade in den Templates sind korrekt gesetzt \newline - es sind nicht genutzte Codeteile vorhanden \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & Alle Dateien innerhalb des Projekts \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & Sensor-Diagramme wieder einblenden (Aufgabe 1) \newline Isolationsmodus leichter bedienbar machen (Aufgabe 2) \newline Pfade in den Templates anpassen (Aufgabe 3) \newline Code strukturieren (Aufgabe 4) \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & Aufgabe 1: Batu und Murati \newline Aufgabe 2: Scholz \newline Aufgabe 3: Murati \newline Aufgabe 4: Gebhardt \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|
|
|
|
\paragraph{Teaminterner Codereview - 07.03.2012}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 8 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & Fehler auf der Webseite \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & Filteroption f"ur die "offentlichen Daten mit Propel nicht ohne weiteres m"oglich (Propel klammert die Ausdr"ucke innerhalb eines SQL-Statements selbstst"andig) \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & get_markers.php und QuerySelect.php \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & Umsetzung der Filterm"oglichkeiten mit Propel (Aufgabe 1) \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & Aufgabe 1: Scholz \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|
|
|
|
\paragraph{Teaminterner Codereview - 27.03.2012}
|
|
\vspace{1cm}
|
|
\begin{tabbing}
|
|
\begin{tabular}{||p{5.4cm}||p{11cm}||}
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Reviewnummer & 9 \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Thema & Abschlussreview \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Teilnehmer & Murat Batu, Ulf Gebhardt, Lulzim Murati, Michael Scholz \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Erkannte Probleme & - abfrage sensibler Daten mit Hilfe der Filter in der API \newline - "Andern des Zeitfilters hat keine Auswirkungen auf die Ergebnisse der Heatmap \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Betroffene Datei & api.php und propel \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Aufgabe & Abschalten der Filter in der API (Aufgabe 1) \newline Kooperation des Zeitfilters mit der Heatmap sicherstellen (Aufgabe 2) \\
|
|
\hline \rule[-2ex]{0pt}{5.5ex} Zust"andige Person & - Aufgabe 1 und 2: Gebhardt \\
|
|
\hline
|
|
\hline
|
|
\end{tabular}
|
|
\end{tabbing}
|
|
\newpage
|