Kurze Version
Wenn der Job darin besteht, eine Bewerbung zu pflegen, müssen Sie folgende Fähigkeiten während eines Interviews testen:
Die Fähigkeit, die große Codebasis mit ihren Dokumentationen, Unit-Tests usw. zu verstehen .
Die Fähigkeit, den Code umzugestalten und Änderungen vorzunehmen, ohne alles zu beschädigen.
Wenn Sie Leute bitten, Code zu lesen, können Sie diese Fähigkeiten nicht beurteilen.
Lange Version
Wurden Sie gebeten, Code zu schreiben? Wenn ja, wie Sign in seiner Antwort angemerkt hat , ist dies ausreichend. Wenn wir ein bisschen verallgemeinern, eine Person, die klaren, leicht verständlichen Quellcode schreibt, könnte den von anderen Personen geschriebenen Quellcode lesen.
Wenn Sie nicht aufgefordert wurden, Code zu schreiben, wurden Sie wahrscheinlich von einer Person aus der Personalabteilung interviewt. Solche Interviews können nicht zu technisch sein und sind meistens wertlos, da sie nicht Ihre Fähigkeiten und Ihre Fähigkeit, gut zu arbeiten, beeinträchtigen, sondern die Anzahl der Jahre, die Sie am College verbracht haben, und andere Dinge, die nichts mit dem Job zu tun haben.
Es gibt noch ein paar weitere Gründe, warum Sie nicht aufgefordert werden, Code für einen Wartungsjob zu lesen:
1. Es ist schwierig, zuverlässig zu tun
Was würden Sie konkret tun, wenn Sie ein Interviewer wären? Lassen Sie Ihre Kandidaten einen Code lesen. Welcher Code? In welcher Sprache? Wie gut oder schlecht geschrieben? Mit oder ohne Kommentar? Mit oder ohne Dokumentation?
Was sagt es über den Kandidaten aus? Wie gut korreliert es mit der Codebasis selbst?
Angenommen, Sie müssen eine ältere VB.NET-App warten. Sie wissen, dass der Quellcode meistens hässlich und ungetestet ist und einige Kommentare veraltet oder irreführend sind. In den letzten drei Monaten arbeitete ein sehr geschickter Entwickler an der Lösung. Er überarbeitete und testete die kritischsten Teile der Anwendung, fügte Kommentare hinzu, wenn dies erforderlich war, und schrieb vor allem eine detaillierte Dokumentation über die Gesamtarchitektur, die kritischen Teile und die Fallstricke.
Sie stellen jetzt einen Entwickler ein, der diese Codebasis verwaltet. Würden Sie während eines Interviews einen älteren (hässlich ungetesteten) Code oder den Code, der vom vorherigen Entwickler überarbeitet wurde, angeben?
Würden Sie die Dokumentation geben? Um die Dokumentation zu lesen, muss der Kandidat mindestens einige Stunden verbringen. Dies macht es unmöglich, während eines Interviews zu tun.
2. Das Lesen eines kurzen Codeteils ist nicht dasselbe wie das Lesen des Codes eines vertrauten Projekts
Denken Sie daran, die Aufgabe ist es, ein Projekt zu pflegen. Es ist schwierig, eine große Codebasis in den ersten Tagen oder Wochen beizubehalten, wenn Sie mit dem Projekt nicht vertraut sind. Es ist viel einfacher, dies nach ein paar Monaten zu tun, wenn Sie die gesamte Dokumentation geschrieben haben und einen klaren Überblick über die gesamte Codebasis haben.
Das Wichtigste, was zu testen ist, ist, ob die Person in diesen Monaten effizient sein wird . Es ist dir egal, ob die Person in den ersten zwei Tagen überhaupt nichts verstehen kann.
Wenn Sie eine Person bitten, einen kurzen Code von Grund auf zu lesen, testen Sie nicht, wie diese Person mit einem vertrauten, dokumentierten Codeabse von Tausenden von LOC umgehen kann .
3. Bei der Pflege des Quellcodes wird dieser nicht nur gelesen
Wenn Sie einen Code - Basis pflegen, werden Sie modifizieren sie. Ein Entwickler, der nur Code liest, bringt seinem Unternehmen nichts Nützliches.
Die nützlichen Fähigkeiten sind die Fähigkeit, Refactoring - Code , um Unit - Tests hinzufügen , um die Auswirkungen einer Änderung vorhersagen , etc. Sie nicht testen , diese Fähigkeiten durch eine Person zu fragen Code während des Interviews zu lesen.