Code Review: Unterschied zwischen den Versionen
Aus Das Sopra Wiki
K Änderungen von Langenfeld (Diskussion) wurden auf die letzte Version von Dietsch zurückgesetzt Markierung: Zurücksetzung |
|||
| (7 dazwischenliegende Versionen von 2 Benutzern werden nicht angezeigt) | |||
| Zeile 48: | Zeile 48: | ||
Ein gutes Beispiel aus dem Softwarepraktikum ist: | Ein gutes Beispiel aus dem Softwarepraktikum ist: | ||
<syntaxhighlight lang=" | <syntaxhighlight lang="md">94cb043 see #27: Wayfinding improved to support bridges</syntaxhighlight > | ||
Ein schlechtes Beispiel ist: | Ein schlechtes Beispiel ist: | ||
<syntaxhighlight lang=" | <syntaxhighlight lang="md">959f313 Woololoo</syntaxhighlight> | ||
====Naming (und Coding Conventions)==== | ====Naming (und Coding Conventions)==== | ||
| Zeile 101: | Zeile 101: | ||
{ | { | ||
try | try | ||
{ | { | ||
int resultId = Results[SelectedResultId - 1]; | int resultId = Results[SelectedResultId - 1]; | ||
| Zeile 149: | Zeile 148: | ||
===Architektur=== | ===Architektur=== | ||
Prüfen Sie ob der Code den [https://sopranium.de/CleanCode#Prinzipien architektonischen Grundprinzipien] entspricht. | |||
---- | ---- | ||
*[https://google.github.io/eng-practices/review/reviewer/standard.html The Standard for Code Review] | *[https://google.github.io/eng-practices/review/reviewer/standard.html The Standard for Code Review] | ||
*[https://google.github.io/eng-practices/review/reviewer/looking-for.html What to look for in a Code Review] | *[https://google.github.io/eng-practices/review/reviewer/looking-for.html What to look for in a Code Review] | ||
*[https://github.com/google/eng-practices Google Engineering Principles Documentation] | *[https://github.com/google/eng-practices Google Engineering Principles Documentation] | ||
