2016-03-20 18 views
0

Kürzlich habe ich meine Firma gewechselt und in meiner neuen Firma sind folgende Kodierungsstandards ein Muss. Während meiner Tech-Lead-Review erhielt ich einen Review-Kommentar: "Wir sollten keine privaten Methoden innerhalb privater Methoden aufrufen."Klarheit über Kodierungsstandards für private Methoden

Also ist es eine generische Praxis, die überall verfolgt wird? Was passiert, wenn die Größe meines privaten Methodencodes zunimmt (z. B. mehr als 50 oder 60 Zeilen)? Wie können wir dann in einem solchen Fall die Größe und Komplexität von Methoden reduzieren?

Die Definition verschiedener Funktionalitäten in verschiedenen Methoden macht es auch einfacher zu verstehen, aber wenn der oben erwähnte Standard für private Methoden befolgt wird, dann ist in diesem Fall ein möglicher Weg zur Handhabung der Methodenkomplexität möglich.

+4

Wenn ich an Ihrer Stelle wäre, würde ich anfangen, nach einem anderen Job zu suchen. Das klingt nach einer Regel, um Regeln zu formulieren, da sie in der guten Praxis keine Grundlage hat. Ihre Zweifel daran sind begründet. –

+0

@JimGarrison Ja, ich frage mich, welche Art von [Zinn Gott] (http://www.dictionary.com/browse/tin-god) mit solch einer Regel kam, und welche Rechtfertigung erfunden wurde, wenn überhaupt. – Andreas

+0

Ich hatte immer meine Zweifel an dieser Regel. Vielleicht sollte ich jetzt anfangen, nach einem neuen Job zu suchen :): D – Raghav

Antwort

6

Nein, das ist nicht üblich. Ich habe noch nie von einer solchen Regel gehört, und ich kann mir keinen Grund dafür vorstellen. Es ist absolut nichts falsch daran, eine private Methode von einer anderen privaten Methode aus aufzurufen.

+0

Stimme völlig zu. Jedes Mal, wenn Sie einen gemeinsamen Code haben, sollte er in eine separate Methode zur Wiederverwendung eingefügt werden, anstatt dieselbe Logik überall zu replizieren. DRY ([Wiederhole dich nicht] (https://en.wikipedia.org/wiki/Don't_repeat_yourself)). Sie sollten auch Methoden, die zu groß sind, in kleinere überschaubare Aufgaben mit genau definierter Verantwortung aufteilen. Hilft Klarheit und Code Übersicht. – Andreas