Wie bekomme ich Feedback von der Community zu großen Codestücken?


8

Code Review.SE ist ideal, wenn Sie Feedback zu einem präzisen, kurzen Code benötigen.

Aber wo kann man ähnliche Rückmeldungen zum Code selbst erhalten, wenn:

  • Sie haben Tausende von LOC,

  • Haben Sie keine Kollegen an Ihrem Arbeitsplatz, die bereit oder bereit sind, den Code zu überprüfen¹,

  • Sie müssen nicht Tausende von Dollar für eine professionelle Überprüfung durch einen Drittentwickler ausgeben? ²

Orte wie CodePlex sind eine gute Idee, um Ihr Projekt bekannt zu machen³, aber nach dem, was ich gesehen habe, ist das Feedback, das Sie zu bekannten Projekten erhalten, Kundenfeedback, dh betrifft die Fehler und Funktionsanforderungen, nicht die Qualität des Quellcodes selbst.

Wie kann die Community auf soziale Weise in die Codeüberprüfung der Codebasis einer bestimmten Größe für ein Open-Source-Projekt einbezogen werden, das nicht die Größe von Firefox oder ähnlichen Produkten aufweist?


¹ Dies ist bei den meisten persönlichen und Open-Source-Projekten der Fall oder bei Projekten, die in Unternehmen durchgeführt werden, in denen keine regelmäßige und vollständige Codeüberprüfung praktiziert wird.

² Dies ist wiederum bei den meisten persönlichen und Open-Source-Projekten der Fall.

³ Auch wenn zu viele auf CodePlex veröffentlichte Projekte nie bekannt werden, entweder weil es niemanden interessiert oder weil sie nicht sehr gut präsentiert werden.


3
Kleingedrucktes macht mich nervös!
AnthonyBlake

2
Was macht Ihr Projekt so interessant, dass andere es sich ansehen möchten?

@ Thorbjørn Ravn Andersen: Auch bei ausreichend interessanten Projekten bleibt das Problem bestehen. StyleCop auf CodePlex zum Beispiel ist zweifellos interessant, aber der Issue-Tracker ist immer noch voll von "Kundenfeedback": Fehlerberichte und Funktionsanfragen, nichts im Zusammenhang mit der Codeüberprüfung.
Arseni Mourzenko

Antworten:


6

Es gibt keine Möglichkeit in den sozialen Medien, die Leute dazu zu bringen, Stunden ihrer Zeit damit zu verbringen, Code für ein zufälliges Projekt zu überprüfen ... oder irgendetwas anderes zu tun. Minuten, die Sie haben können, Stunden nicht so sehr.

Sie haben also zwei Möglichkeiten:

1) Brechen Sie es in Minuten lange Stücke. Sie müssen NICHT ALLE Ihre Tausenden von Codezeilen überprüfen. Extrahieren Sie die wichtigsten Bits und bitten Sie um Überprüfung von einem oder zwei davon. Sie werden wahrscheinlich eine Art allgemeinen Stil / Do-Do-That / Name-Things-Like-This-Kommentar erhalten, den Sie dann auf die gesamte Codebasis anwenden können. Wenn Sie eine Menge Bits haben ... nun, schneiden Sie mehr ab und bitten Sie vielleicht um eine oder zwei Überprüfungen pro Woche und wiederholen Sie diese anhand des Kommentars, den Sie zurückerhalten. Wenn alle Kommentare Dinge sind, die Sie zuvor gesehen haben und die Sie ohnehin nicht ändern möchten, sind Sie fertig.

2) Machen Sie das Projekt für die Menschen wichtig. Ich glaube nicht, dass Sie dies tatsächlich tun können, aber wenn Sie Code haben, der die griechische Finanzkrise wirklich löst, gewinnen Sie.


3

Schwierige Frage; Ich bin mir nicht sicher, ob es eine solide Antwort gibt.

Ich denke, Ihre Frage besteht aus zwei Teilen:
Erstens, wo können Sie die Anzeige veröffentlichen, in der die Überprüfung angefordert wird.
Zweitens, wie man Leute findet, die bereit sind, die Zeit für die Durchführung der Überprüfung einzuplanen.
Sie sind verwandt, aber subtil unterschiedlich.

Die zweite Frage könnte einfacher zu beantworten sein als die erste - sowohl die P.SE- als auch die CR.SE-Community sollten Leute haben, die bereit wären, sich zu engagieren und dabei zu helfen. Persönlich finde ich, dass ich fast immer etwas aus einer Rezension lerne, deshalb finde ich sie lohnend. Die Motivationen variieren. Wenn es sich also um einen interessanten Codebereich handelt, der bei der Identifizierung von Personen oder mehreren Personen hilfreich sein kann.

Der erste ist nicht so einfach zu beantworten. P.SE ist nicht verfügbar, da die Anfrage nicht dem Q & A-Format entspricht. CR.SE scheint auch für Dinge nicht gut geeignet zu sein, es sei denn, Sie haben die Dinge in kleinere Teile (Module / Funktionen) aufgeteilt und hatten eine Menge Beiträge. Auf Anhieb kenne ich keine Websites, auf denen Sie eine solche Anfrage stellen könnten. Wenn das SE-System ein Person-zu-Person-Mailing-System hätte, könnte dies eine Möglichkeit der Rekrutierung sein, aber ich riffle dort.

Angesichts der Tatsache, dass eine Out-of-Band-Überprüfung von mehreren tausend Loc eine Weile dauern wird, würde ich die Geduld in diesem Prozess fördern.

Kurz gesagt, Sie müssen nur einen Kollegen finden, der bereit ist, die Überprüfung durchzuführen.


FWIW, ich habe Ihre Beiträge zu P.SE sehr geschätzt und würde Ihnen im Gegenzug gerne ein paar Stunden Rückblick geben. Die einzige Einschränkung ist, wenn ich die Sprache gut genug kenne.

1

Ich würde einen ersten Stich mit etwas wie Stylecop und Fxcop oder Äquivalenten für Ihre Sprache machen. Als nächstes können Sie die Architektur dokumentieren und sehr schnell Feedback dazu erhalten. Schließlich könnten Sie einige lästige Schnipsel oder ein paar Klassen nehmen und einen Entwickler bitten, sie mit Ihnen zu besprechen, anstatt die gesamte Codebasis. Vorausgesetzt, Sie haben den Codestil nicht überall geändert, sollten einige Klassen einen Einblick in die Codebasis als Ganzes geben.

Durch die Nutzung unserer Website bestätigen Sie, dass Sie unsere Cookie-Richtlinie und Datenschutzrichtlinie gelesen und verstanden haben.
Licensed under cc by-sa 3.0 with attribution required.