Ich stelle diese Frage an C ++ - Programmierer, weil: a) Nur ein C ++ - Programmierer die technischen Vorzüge der Beispiele beurteilen kann; b) Nur ein Programmierer hat ein Gefühl für das Temperament eines anderen Programmierers, der Code wie diesen schreibt.
HR und die Direktoren sind sich bewusst, dass es ein Problem gibt, nur weil sie Beweise vor Ort sehen. Ich rufe an, ob wir dem fraglichen Programmierer mehr Zeit geben. Viele der Fehler sind sehr grundlegend - meine Frage (an Programmierer) lautet, ob jemandem, der sich als leitender C ++ - Entwickler ausgibt, aufgrund von Beispielen seines aktuellen Codes der Vorteil eines Zweifels eingeräumt werden sollte. Nicht-Programmierer - auch Leute außerhalb der C ++ - Programmierung - können dies nicht beurteilen.
Als Hintergrund wurde mir die Aufgabe übertragen, Entwickler für ein etabliertes Unternehmen zu managen. Sie haben einen einzigen Entwickler, der sich (seit Ewigkeiten) auf die gesamte C ++ - Codierung spezialisiert hat, aber die Qualität der Arbeit ist miserabel. Codeüberprüfungen und -tests haben viele Probleme aufgedeckt, eines der schlimmsten sind Speicherlecks. Der Entwickler hat seinen Code noch nie auf Lecks getestet, und ich stellte fest, dass die Anwendungen mit nur einer Verwendungsminute viele MB auslaufen können. Der User berichtete von enormen Verlangsamungen und sagte: "Das hat nichts mit mir zu tun - wenn sie beendet und neu gestartet werden, ist alles wieder gut."
Ich habe ihm Werkzeuge zur Erkennung und Verfolgung von Lecks gegeben und mich viele Stunden mit ihm zusammengesetzt, um zu demonstrieren, wie die Werkzeuge verwendet werden, wo die Probleme auftreten und was zu tun ist, um sie zu beheben. Wir sind 6 Monate später, und ich habe ihn beauftragt, ein neues Modul zu schreiben. Ich habe es überprüft, bevor es in unsere größere Codebasis integriert wurde, und war bestürzt, die gleiche schlechte Codierung wie zuvor zu entdecken. Der Teil, den ich unverständlich finde, ist, dass ein Teil der Codierung schlechter als amateurhaft ist. Zum Beispiel wollte er eine Klasse (Foo), die ein Objekt einer anderen Klasse (Bar) füllen könnte. Er entschied, dass Foo einen Verweis auf Bar haben würde, zB:
class Foo {
public:
Foo(Bar& bar) : m_bar(bar) {}
private:
Bar& m_bar;
};
Aber (aus anderen Gründen) brauchte er auch einen Standardkonstruktor für Foo und anstatt sein ursprüngliches Design in Frage zu stellen, schrieb er dieses Juwel:
Foo::Foo() : m_bar(*(new Bar)) {}Jedes Mal, wenn der Standardkonstruktor aufgerufen wird, ist ein Balken durchgesickert. Um die Sache noch schlimmer zu machen, reserviert Foo Speicher aus dem Heap für zwei andere Objekte, hat aber keinen Destruktor oder Kopierkonstruktor geschrieben. Jede Zuweisung von Foo verliert also tatsächlich 3 verschiedene Objekte, und Sie können sich vorstellen, was passiert ist, als ein Foo kopiert wurde. Und - es wird nur besser - er wiederholte dasselbe Muster bei drei anderen Klassen, so dass es kein einmaliger Ausrutscher ist. Das ganze Konzept ist auf so vielen Ebenen falsch.
Ich würde mich verständnisvoller fühlen, wenn dies von einem absoluten Neuling kommen würde. Aber dieser Typ macht das schon seit vielen Jahren und hat in den letzten Monaten sehr gezielt trainiert und beraten. Mir ist klar, dass er die meiste Zeit ohne Mentoring oder Peer Reviews gearbeitet hat, aber ich fange an zu spüren, dass er sich nicht ändern kann. Also meine Frage ist, würden Sie mit jemandem bestehen, der solch offensichtlich schlechten Code schreibt?
quelle
Antworten:
Mein Rat wäre, ihn mit diesem speziellen Beispiel zu konfrontieren und zu sehen, was er sagt. Wenn er bestreitet, dass der Code irgendetwas Schlechtes enthält, kann man leider nicht viel tun. Wenn er akzeptiert, dass er einen Fehler begangen hat (auch wenn er in dieser Hinsicht defensiv ist), gibt es zumindest Raum für Verbesserungen. Wenn Sie die Zeit und Mühe akzeptieren, ihn zu verbessern, müssen Sie oder ein leitender Programmierer sich hinsetzen und gemeinsam programmieren, bis alle diese Tendenzen abgeflacht sind (seien Sie bereit, diese Person mindestens 1 Monat lang zu beschäftigen).
Ein schlechter Programmierer, mit dem ich normalerweise arbeiten kann, aber ein Programmierer, der sich nicht verbessern kann, kann ich nicht.
quelle
Nein. Ich würde jeden professionellen C ++ - Entwickler entlassen, dem das grundlegende Verständnis der Speicherverwaltung fehlt.
Wenn es jemand ist, der aus Java oder C # kommt oder so, würde ich ihm eine gewisse Lattitude geben, aber das ist reine Inkompetenz.
quelle
Wir können den weiteren Teil Ihrer Frage nicht beantworten. Nämlich, wenn Sie diesen Mitarbeiter behalten oder entlassen. Es ist schwierig, einen Mitarbeiter zu entlassen, aber das ist eine Entscheidung, die außerhalb des Zuständigkeitsbereichs einer solchen Community liegt.
Sie haben Ihre Frage aktualisiert, um die Antworten auf C ++ - Programmierer einzuschränken. Zu meinem Hintergrund / meinen Qualifikationen: Ich schneide meine Zähne in C und kann in C ++, C # und Java programmieren. Und ich mag es, Rennbedingungen zwischen Threads zu jagen, weil ich denke, dass es Spaß macht. Ja, ich "kriege" ein bisschen herum.
Ihre Antwort und Entscheidung dreht sich darum : Ob sich jemand ändern kann oder nicht, hängt von der Person ab und davon, ob sie sich ändern möchte.
Aber fangen wir mit ein paar Fragen an:
Sie müssen sicherstellen, dass Sie all diese Fragen mit Ja beantworten können. Wenn nicht, liegt immer noch eine Beweislast für Ihre Kommunikation mit ihm vor.
Seine Antwort auf Ihre jüngste Rezension wird der aufschlussreichste Teil sein.
Wenn er es versucht hat und einige Anzeichen von Fortschritt zeigt, müssen Sie möglicherweise Ihren Mentoring-Prozess überprüfen. Wenn überhaupt, sollten Sie in Betracht ziehen, ihn mit einem erfahrenen Programmierer zusammenzubringen, damit er sofort Feedback erhält, während er Designentscheidungen trifft. Ich bin kein Fan von Pair Programming, aber in einem solchen Fall kann es sehr nützlich sein. Es ist nicht immer ein praktischer Weg, ihn dazu zu bringen, den alten Code immer weiter zu überarbeiten.
Wenn er es nicht versucht hat, müssen Sie seine Motivation besser verstehen. Hat er nicht verstehen , dass er muss härter versuchen? Ist es ihm einfach egal? Gibt es andere Bereiche im Team, in denen seine Fähigkeiten besser passen würden und die ihn mehr interessieren? Wenn er es nicht versuchen wollte, dann musst du verstehen warum.
Von dort aus wissen Sie, ob er sich ändern möchte und ob er sich ändern kann . Kein Wunsch, sich zu ändern, ist gleichbedeutend damit, sich nicht ändern zu können. Wenn es Verlangen und einen gewissen Fortschritt gibt, sollten Sie nachdrücklich überlegen, wie Sie versuchen, ihn zu rehabilitieren.
quelle
Ich fürchte, sein schlechter Code ist nicht das einzige Problem in Ihrem Team.
Sie sagen, er ist schon länger in der Firma. Eine solche Person zu entlassen, ist selten eine gute Idee (es sei denn, er ist ein Wally-Mitarbeiter). Ihr Wissen über Kundenbedürfnisse, Produkte, die Sie besitzen usw. ist oft viel wertvoller als der Code, den sie schreiben.
Lösungen:
quelle
Die Entscheidung, jemanden zu feuern, fällt jedem schwer. Ihre Situation wird jedoch durch mehrere Faktoren verschärft:
Das heißt, Sie haben die letzten 6 Monate damit verbracht, dem Entwickler den Fehler seines Verhaltens zu zeigen, und sein neuer Code hat sich noch nicht verbessert.
Zu diesem Zeitpunkt müssen Sie wirklich ein proaktives Management starten, damit Sie innerhalb von 3 Monaten Zeit haben
oder
Dazu müssen Sie sich mit dem Entwickler zusammensetzen, schriftlich / per E-Mail erläutern, was falsch ist, wie sich der Entwickler verbessern kann, und ganz klar angeben, dass er in 3 gekündigt wird, wenn die erwartete Verbesserung nicht eintritt Monate.
Sie müssen sich auch darüber im Klaren sein, dass die Verbesserung von diesem Punkt an für den Rest seiner Anstellung im Unternehmen und nicht nur für den Zeitraum von drei Monaten erwartet wird!
Sie sollten auch Ihren eigenen Vorgesetzten und die Personalabteilung (falls vorhanden) informieren.
Während dieses Vorgangs müssen Sie den Entwickler aktiv verwalten und die Aufgaben / den Code alle 1-2 Tage überprüfen und sicherstellen, dass sie auf dem neuesten Stand sind. Dabei müssen Sie die nicht vorhandenen Aufgaben genau beschreiben und erläutern, wie Sie sie verbessern können.
quelle
Entweder war dir nicht klar, wie ernst du seine schlechte Verarbeitung nimmst (dh es ist möglich, dass er sieht, wie viel Zeit du mit ihm verbracht hast, wenn er sich verbessern will, anstatt eine absolute Notwendigkeit), oder er hat eine unglaublich schlechte Haltung, die nicht nachhaltig ist. Wenn diesem Entwickler nicht bereits klar ist, dass Sie über seine Position zu diesem Thema nachdenken, muss dies klar formuliert werden (vorausgesetzt, Ihre Führung ist mit Ihrer Kündigungsbefugnis einverstanden). Der Schock wird hoffentlich eine Veränderung bewirken.
Die Einstellungsentscheidung hat jedoch weitaus größere Auswirkungen als nur dieser Typ. Sie müssen die Auswirkungen auf das Team berücksichtigen. Wenn seine Haltung gedeihen darf, kann dies entweder zu Ressentiments von anderen führen oder andere fühlen, dass solche Dinge auch in Ordnung sind. Aus Sicht der Mannschaft muss jedoch klar sein, dass dies aus den richtigen Gründen geschah und er reichlich Gelegenheit hatte, sich zu verbessern.
Ein Juwel, das ich vor Jahren aufgegriffen habe, ist die Tatsache, dass Leute mit technischem Wissen, das andere nicht haben, das Management dazu bringen können, dass sie nachlassen. Es ist schlecht für das Team. Sie haben möglicherweise Angst, den einzigen c ++ - Entwickler zu verlieren, aber sie können ersetzt werden. Natürlich gibt es Risiken, wenn er die veröffentlichten Produkte gut kennt, aber ich habe oft gesehen, dass Menschen mit scheinbar unersetzlichem Produkt- / technischen Wissen leichter als erwartet ersetzt wurden. Teams und Organisationen können häufig Lücken schließen, die zunächst schwer zu schließen sind. Natürlich, wenn er nicht über C ++ - Kenntnisse oder organisationsspezifisches Wissen verfügt, von dem Sie glauben, dass es schwer zu ersetzen ist, gibt es viel weniger Probleme!
quelle
Natürlich solltest du nicht. Denken Sie daran, dies ist keine Wohltätigkeitsorganisation, Sie tauschen Geld gegen Arbeit. Wenn er nicht seine Seite des Geschäfts hält, dann würde ich wie bei jeder Transaktion aufhören zu zahlen.
quelle
Wenn Sie ihm eine Chance geben möchten, entwickeln Sie einen standardisierten Test, der Messdaten zu Speicherverlusten erfasst. Überwachen Sie wöchentlich seinen Fortschritt und bestehen Sie darauf , den von ihm geänderten Code zu sehen und nach Verbesserungen zu suchen. Wenn er es zu diesem Zeitpunkt nicht schafft, entlassen Sie den nutzlosen Beschimpfer.
quelle