myCSharp.de - DIE C# und .NET Community
Willkommen auf myCSharp.de! Anmelden | kostenlos registrieren
 
 | Suche | FAQ

» Hauptmenü
myCSharp.de
» Startseite
» Forum
» FAQ
» Artikel
» C#-Snippets
» Jobbörse
» Suche
» Regeln
» Wie poste ich richtig?
» Forum-FAQ

Mitglieder
» Liste / Suche
» Wer ist wo online?

Ressourcen
» openbook: Visual C#
» openbook: OO
» Microsoft Docs

Team
» Kontakt
» Übersicht
» Wir über uns
» Datenschutzerklärung
» Impressum

» myCSharp.de Diskussionsforum
Du befindest Dich hier: Community-Index » Diskussionsforum » Entwicklung » Code-Reviews » Code-Review Regeln
Letzter Beitrag | Erster ungelesener Beitrag Druckvorschau | Thema zu Favoriten hinzufügen

geschlossen (weitere Infos)
Zum Ende der Seite springen  

Code-Review Regeln

 
Autor
Beitrag « Vorheriges Thema | Nächstes Thema »
Coffeebean Coffeebean ist männlich
myCSharp.de-Team

avatar-3295.gif


Dabei seit: 25.08.2011
Beiträge: 2.159
Entwicklungsumgebung: VS 2005-2017, VS Code
Herkunft: Deutschland/Schweiz


Coffeebean ist offline

Code-Review Regeln

Beitrag: beantworten | zitieren | editieren | melden/löschen       | Top

Code-Review-Regeln:

1 Code muss compilieren
Niemand nützt es etwas, wenn kein compilierender Code angehängt wird. Der Code, der zum Review gestellt wird, muss in sich ohne Fehler und ausführbar sein.

1.1 "Geschlossenen" Code zur Verfügung stellen
Ein paar Zeilen "herrenlosen" Code zu posten, ist nicht Sinn dieses Forums. Bitte postet immer komplette Methoden, Klassen oder Projekte, die sich ohne weiteres Kopieren und/oder ausführen lassen.
Für ganze Projekte verwendet bitte GitHub, GitLab oder Azure DevOps (Public Repository) - oder ein ähnliches Code-Portal.
Bitte keine ganzen ZIP-Dateien anhängen oder OneDrive/DropBox verlinken.

1.2 Third-Party Libraries
Bitte prüft vor dem zur Verfügung stellen von Code von Drittanbietern, ob dies erlaubt ist, und stellt eine externe dll, die ihr benutzt, direkt zur Verfügung.

2 Wir suchen nicht eure Fehler
Um Fehler zu suchen oder nach richtiger Funktionalität zu fragen, dafür ist dieser Forenbereich der falsche Ort. Es geht hierbei um funktionierende Lösungen, die "nur" auf Stil, Umsetzung, Patterns, Best-Practices etc. untersucht werden sollen.

3 Geduld
Reviews sind sehr zeitaufwändig. Sie kosten neben der Einarbeitungszeit auch Zeit zum Lösungen Überlegen und dem Verfassen von Alternativvorschlägen. Bitte bedenkt dies beim Posten eures Codes. Je mehr Code ihr postet, desto umfangreicher kann eine Antwort werden, und desto länger kann diese dauern.

Auch hier gilt:
4 Stellt genaue Fragen
Die Frage "Wie geht das besser?" kann auf hunderte Arten beantwortet werden. Meistens ist "Anders" ein Äquivalent zu "Besser". Dies liegt fast immer im Auge des Betrachters. Stellt heraus, wo genau ihr ein Review am nötigsten haltet und wo ihr glaubt, eine andere Meinung würde euch am meisten helfen.

4.1 Gebt genug Kontext
Wenn man sich in ein Programm "eindenken" muss, ist das oft komplex. Daher gebt den Helfenden genug Kontext, schnell und ohne Umstände in das Programm einzusteigen und zu verstehen, was ihr tun wollt.

5 Lasst andere Meinungen zu
Falls ihr mehrere, vielleicht sogar verschiedene Meinungen zu eurem Code hört, überlegt euch dessen Ansatz und bewertet selber, ob dieser für euch in Frage kommt oder nicht.

Bitte beachtet immer, dass ihr am Besten wisst, was euren Code voranbringt. Alle genannten Kommentare sind allenfalls Hinweise und versuchte Hilfestellungen.
26.02.2015 21:36 E-Mail | Beiträge des Benutzers | zu Buddylist hinzufügen
Baumstruktur | Brettstruktur       | Top 
myCSharp.de | Forum Der Startbeitrag ist älter als 4 Jahre.
Der letzte Beitrag ist älter als 4 Jahre.
geschlossen (weitere Infos)


© Copyright 2003-2019 myCSharp.de-Team | Alle Rechte vorbehalten. | Dieses Portal verwendet zum korrekten Betrieb Cookies. 21.05.2019 20:51