Nach fast zwei Jahren schlägt Senior immer noch unaufgefordert Änderungen vor, die den Code brechen

Früher war ich ein Junior bei meiner Arbeit. Der 25-jährige Senior, der neben mir saß, ist dazu übergegangen, zu mir zu kommen und Vorschläge zur Änderung und Neuordnung meines Codes zu machen ... Anfangs nahm ich ihn für bare Münze und versuchte, seine Vorschläge umzusetzen, aber ich habe seine Vorschläge gefunden macht meistens entweder total kaputt oder mindert bestenfalls die Qualität meines Codes.

Ich verstehe mich auf persönlicher Ebene gut mit dem Senior und manchmal erreicht er wertvolle Dinge mit seinem eigenen Code – in gewisser Weise seltsam, weil er mir oft zu sagen scheint, ich solle Praktiken implementieren, die er in seinem eigenen Code vermeidet. Die Gespräche darüber nehmen zunehmend meine Zeit in Anspruch, bis sie fast meiner Produktion zur Aufgabe im Wege stehen.

Heute sind wir an einem Punkt angelangt, an dem ich, nachdem ich zahlreiche Vorschläge mit der Begründung zurückgewiesen hatte, dass sie nicht durchführbar seien oder sich auf das Polieren von Kleinigkeiten auf Kosten von MVP konzentrierten, an dem Punkt ankamen, an dem er sagte: „Ja, aber wenn Sie das selbst beenden, was wird müssen andere tun?".

Es gibt viel mehr zu tun, als möglich wäre, und es gibt immer viel zu tun, und ich schlug sogar vor, dass er zu der Aufgabe beitragen könnte, wenn er wollte – etwas, an dem er nicht sehr interessiert zu sein schien.

Nach zwei Jahren, an dem Punkt, an dem ich bald Senior sein werde, hat ein anderer Entwickler angefangen, darüber zu scherzen, dass er sterben will, wenn dieser Senior ihm „hilft“. Eine Seniorin sagte: „Er macht Witze, aber eigentlich nicht“.

Wie gesagt, ich verstehe mich im Allgemeinen gut mit ihm, aber ich bin sehr daran interessiert, dieses Problem zu lösen, bevor es eskaliert. Wie geht man am besten vor?

Wie lange sind Sie schon im Unternehmen?
Weiß er, dass sie Ihren Code brechen lassen? Könnte es auch kaputt gehen, weil es ein Designproblem mit Ihrem Code gibt?
@cgTag verzeihen Sie mir, dass ich vielleicht ein bisschen paranoid bin, aber Leute fragen mich oft solche Dinge und andere Fragen wie „Willst du den Junior in deiner Berufsbezeichnung loswerden?“ Daher muss ich sagen: Ich strebe derzeit keinen Jobwechsel oder Wechsel in ein anderes Unternehmen an. Aber wenn Ihre Frage zum Nennwert ist, da bin ich mir sicher, und dazu bestimmt ist, Ihnen bei der Beantwortung meiner Frage zu helfen: sechs Monate.
@PeterDavidCarter In den ersten Monaten erkunden die Leute eher die neue Arbeitskultur. Sie lernen während dieser Zeit, was die Politik ist. (wer hat wirklich das Sagen, warum und wie setze ich mich durch). Ich hatte das Gefühl, dass der Senior Sie mit dem Hubschrauber überfliegt, weil Sie ein neues Mitglied des Teams waren. 6 Monate ist ungefähr die Zeit, in der ich erwarten würde, dass er/sie beginnt, sich zurückzuziehen. Ich denke, Sie können ihn ruhig bitten, Ihnen mehr Freiheiten zu geben, oder dies mit der Personalabteilung besprechen. Von der Frage her ging ich davon aus, dass Sie gerade erst angefangen hatten.
@cgTag Er war neu in unserem Team, das angeblich ein Aushängeschild für unser Unternehmen ist, allerdings mit einem hohen Anteil an Junioren, die teilweise aus höheren Positionen kamen
"Ja, aber wenn du das selbst fertig machst, was müssen andere Leute tun?" - könnte das vielleicht ein Witz gewesen sein? Klingt nach einer...
Wenn das alles Witze sind, dann scheint es merkwürdig, dass sie ernsthaft formuliert sind und darauf bestanden wird, dass die Vorschläge befolgt werden. Zumal solche Beharrlichkeiten in der Regel je nach Grad der Probleme mit der eigenen Arbeit dieser Person variieren. Natürlich ist die bestimmte Aussage, die Sie zitieren, bis zu einem gewissen Grad ein Witz, aber sie verbirgt einen ernsten Punkt unter dem Humor. Persönlich mache ich keine Witze, die das Potenzial haben, Probleme mit dem Produkt zu verursachen, es sei denn, ich bin mir sicher, dass sie als solche interpretiert werden. Ich hätte gehofft, dass das jedermanns Meinung war.
Ich werde den meisten Antworten nicht zustimmen. Vielleicht versucht er, Sie zu sabotieren. Wieso den? weil es mir passiert ist. Ich habe dieses dumme/neutrale Gesicht (ich bin 28 Jahre alt und es sieht aus, als wäre ich 20). Nach 5 Jahren bekam ich einen neuen Job bei einem Senior (jeder weiß, dass er wirklich ein beschissener Kollege ist). Er dachte, ich sei ein Junior, ein dummer SQL-Datenbankadministrator. Ich wusste, dass er eine "falsche Person" war (ich kenne diesen Begriff nicht auf Englisch), weil er diese ruhige, friedliche Person war. Ich habe mich nicht geirrt. Bei der ersten "Aufgabe", die er mir gab, bemerkte ich, wie dumm er war. Ich bemerkte es Woukd alles kaputt gemacht hat, und er hat das absichtlich gemacht. 2 Jahre damit. Ich habe einen neuen Job.
@GreenBaloon es geht nicht darum, dass die Leute dumm sind, es geht darum, dass sie ihre ganze Gehirnenergie verbrauchen, um andere Leute zu blockieren, anstatt das zu tun, was sie tun, oder anderen zu helfen. Am Ende steht das dem Fortschritt im Weg.
@PeterDavidCarter Es gibt einen Humorstil, bei dem Witze absichtlich mit todernster Stimme erzählt werden und es dem Publikum überlassen wird, "es zu verstehen". Es gibt sogar einen Namen dafür (den ich vergessen habe). Ich tue es ziemlich oft natürlich
@ig-dev Ich denke, der Begriff, an den Sie denken, ist deadpan .
Ich denke, ich werde Ihnen dann sagen, basierend auf Ihrem Rat: Erhöhen Sie die Kernreaktortemperatur so hoch, wie Sie möchten. Jeder wird sterben, aber einige Leute haben vielleicht ein Lachen.
Seit wann sind Sie mit zwei Jahren Senior?
@Kabard Es scheint, dass das bald passieren wird
@Kabard, wenn nicht, habe ich mein persönliches Projekt und ein paar Leute, die unter mir arbeiten und mich respektieren, wenn ich etwas sage
@JoeW Wenn ich Code ändere und ihn kaputt mache, ist das nie ein Designproblem. Es ist ein Problem, dass ich Dinge anfasse, die ich nicht verstehe. Wenn es ein Designproblem gibt, das dazu führt, dass Dinge kaputt gehen, wenn sie geändert werden, dann ändere ich das nicht.

Antworten (5)

Ich bin ein Junior bei meiner Arbeit. In letzter Zeit ist der Senior, der hinter mir sitzt, oft dazu übergegangen, zu mir zu kommen und Vorschläge zur Änderung und Neuordnung meines Codes zu machen ...

Er gibt den Manschettenratschlag ab. Er versucht dir zu helfen, aber er macht es nicht gut.

Anfangs habe ich ihn für bare Münze genommen und versucht, seine Vorschläge umzusetzen, aber ich habe festgestellt, dass die Vorschläge, die er macht, normalerweise entweder völlig kaputt gehen oder bestenfalls die Qualität meines Codes beeinträchtigen.

Die Qualität des Codes ist sehr subjektiv.

Verfügt Ihre F&E-Abteilung über eine Dokumentation, die die Codequalität umreißt?

Meine Erfahrung ist, dass Junioren die Ratschläge der Senioren nicht in die Praxis umsetzen. Bei der Übersetzung geht viel verloren und es fehlt an einer gemeinsamen Basis, um die Lücken zu füllen.

Vielleicht sagt er Ihnen, dass Sie "A" implementieren sollen, und Sie haben "B" implementiert. Sie können sehen, dass "B" nicht wirklich gut ist. Du fühlst dich, als wäre deine Arbeit auf die Schippe genommen worden, und als nächstes kommt er und sagt, du sollst "C" implementieren, aber du setzt wieder "D" um. Ich kann dir nicht vorwerfen, dass du frustriert bist.

Es ist ein Kommunikationsproblem.

Dokument, Dokument, Dokument.

Wenn er das nächste Mal einen Rat anbietet, bitten Sie ihn, ihn schriftlich niederzulegen. Verwenden Sie eine E-Mail, ein Dokument oder einen Bugtracker, aber lassen Sie es sich schriftlich geben. Antworten Sie ihm mit Ihren Fragen, aber versuchen Sie, Ihre Fragen in einer einzigen Antwort zu organisieren. Fügen Sie Quellcodebeispiele, Referenzdateien oder Links zum Internet hinzu, um zu überprüfen, welche Entwurfsmuster befolgt werden sollten.

Wenn er keine schriftliche Version seines Ratschlags bereitstellt, schreiben Sie ihn auf und leiten ihn zur Überprüfung weiter. Bitten Sie um Klärung von allem, was nicht klar ist.

Lassen Sie sich alles schriftlich geben.

Paar-Programmierung

Wenn er das nächste Mal zu dir kommt, bitte ihn, sich neben dich zu setzen. Geben Sie ihm Tastatur und Maus. Bitten Sie ihn, zu demonstrieren, was er meint. Bitten Sie ihn, einen Teil des Quellcodes zu schreiben. Bitten Sie ihn zu bleiben , während Sie einige seiner Ratschläge ausprobieren. Lassen Sie ihn sehen, wie es Ihnen Probleme bereitet . Geben Sie ihm die Chance zu erklären, wie er es beheben kann.

Bitten Sie ihn, zurückzukommen und es noch einmal zu tun . Lernen Sie gemeinsam Code zu schreiben.

Interessanterweise sind wir als Entwicklungsteam den Ressourcen unseres QA-Teams zum Testen voraus. Also: "Verfügt Ihre F&E-Abteilung über eine Dokumentation, die die Codequalität umreißt?" ist ein wirklich interessanter Kommentar. Wir haben jetzt SonarQube, aber es wurden viele Gespräche über die Bewertung der Codequalität auf objektiver Ebene geführt.
@PeterDavidCarter Sie müssen Richtlinien schreiben, wenn Sie versuchen, etwas sehr Subjektives durchzusetzen. Tools wie Sonar sind eine Form der Dokumentation. Es wird zu einer Autorität, auf die Sie auch andere Entwickler verweisen können, aber es erklärt nicht alles. Was schätzen Sie als Unternehmenskultur? Code-Wartbarkeit, Rabbit-Entwicklung, Produktstabilität, Programmierer mit niedrigerem Gehalt, Senior-Mentoring, Projektkosten und Timing? Diese bestimmen die Qualität des Quellcodes, aber noch wichtiger ist, dass Entwickler verstehen müssen, warum sie ihn beeinflussen.
Sie haben Recht: Es ist entscheidend, mit anderen Menschen zusammenzuarbeiten, ihre Arbeit zu integrieren und ein Projekt auf ein professionelles Niveau zu bringen. Welche Schlagworte auch immer darum geredet werden, das muss man können
Dies geschah wiederholt über fast zwei Jahre und geschieht immer noch. Es ist kein Unfall. Es ist eine Strategie eines schlechten Programmierers, der dort angekommen ist, wo er ist, indem er jeden sabotiert, den er als Bedrohung ansieht.

Ich kann mich damit irren, aber für mich klingt es so, als würde Ihr Vorgesetzter nur mit Ihnen scherzen. Sowohl die Änderungsvorschläge, die den Code brechen, als auch die schnippische Bemerkung, dass sonst niemand Arbeit zu erledigen habe, klingen für mich nach Sarkasmus.

Die Antwort wäre, darüber zu lachen und weiter (guten) Code zu schreiben.

Da Sie ein Junior sind, erkennen Sie möglicherweise nicht, dass die vorgeschlagenen Codeänderungen schlecht sind, bis Sie sie vornehmen. Frage ihn in diesem Fall, ob er es ernst meint, wenn du dir nicht sicher bist.

Vielleicht ist der Senior witzig und OP ist ein leichtes Ziel. Wahrscheinlich schreibt der Senior irgendwo einen Post "Ich mache jetzt seit zwei Jahren Witze mit diesem Junior und er kapiert es immer noch nicht"
@ig-dev In diesem Fall würde der Senior einen großen Tritt brauchen. Dies absichtlich zu tun, ist nur Mobbing. Und das Unternehmen wird nicht erfreut sein, wenn es herausfindet, dass er die Arbeit eines Kollegen sabotiert.

IMHO, verschieben Sie seine Vorschläge auf ein nachvollziehbares Medium und trennen Sie sie von persönlichen Interaktionen, bei denen Sie sagen, dass Sie miteinander auskommen.

Bitten Sie ihn, Ihnen eine E-Mail zu senden, da er derzeit etwas beschäftigt ist, um seinen gesamten Vorschlag zu verstehen.

Glaub mir, du bekommst vielleicht 10% von dem, was du jetzt hast, von ihm.

Ignorieren Sie einfach Vorschläge, die Ihren Code nicht verbessern. Ich würde die ganze Menge nach dem ersten oder zweiten Fiasko einfach ignorieren.

Fahren Sie mit dem fort, was Sie wissen, und nehmen Sie Ihr Feedback zur Verbesserung aus den normalen Quellen wie Rezensionen usw.

Seniorität bedeutet nicht automatisch besser oder sogar sachkundiger. Die Berufserfahrung vieler Menschen ist von minderer Qualität, selbst wenn sie seit Jahrzehnten in einer Branche tätig sind.

Ignorieren Sie die Vorschläge einfach höflich, es ist kein Schönheitswettbewerb und Popularität steht nicht im Vordergrund, anstatt solide Arbeit zu leisten und Ihre berufliche Karriere voranzutreiben. Nach einer Weile findet er jemand anderen, den er ärgern kann.

Das mag eine kulturelle Sache sein. In manchen Kulturen werden „Vorschläge“ als Befehle verstanden und es wäre höchst konfrontativ, sie nicht zu befolgen. In anderen Kulturen zeigt das Machen von Vorschlägen, dass Sie interessiert sind, aber Sie würden Vorschläge ignorieren, es sei denn, Sie selbst denken, dass sie eine gute Idee waren.

Der Vorgesetzte merkt möglicherweise nicht einmal, dass Sie Vorschlägen wider besseres Wissen folgen. Ich würde vorschlagen, dass Sie für Vorschläge offen sind, aber immer das tun, was Sie für das Beste halten, es sei denn, jemand befiehlt Ihnen etwas anderes und übernimmt die Verantwortung dafür.

Ja. Das ist wahr. Die betreffende Person ist in der Vergangenheit sehr wütend geworden, weil sie die Befehlskette gebrochen hat, aber ironischerweise haben sich die Leute an der Spitze der Kette (obwohl ich irgendwie verrückt bin und ihnen bei zahlreichen Gelegenheiten ein Arschloch war) auf meine Seite gestellt und Die Breaking Changes an der App wurden nicht auf allen kürzlich durchgeführten Instanzen wiederhergestellt.
Nach fast zwei Jahren schlägt Senior immer noch unaufgefordert Änderungen vor, die den Code brechen
AntwortenHierDatenschutz-Bestimmungen1y, 0v