Behebung eines Fehlers, der bis jetzt noch nie ein Problem verursacht hat

Ich habe kürzlich eine Änderung vorgenommen, die dazu führte, dass Code viel häufiger ausgeführt wurde als früher. Dies führte zur Entdeckung eines Fehlers. Dieser Fehler konnte jedes Mal auftreten, wenn dieser Code ausgeführt wurde. Da er jedoch so selten ausgeführt wurde, ist er nie aufgetaucht.

Als ich den leitenden Entwickler darauf aufmerksam machte, wollte er, dass ich die Änderung, die den Fehler aufdeckte, rückgängig machte, anstatt den Fehler zu beheben.

Mir ist klar, dass wir bis jetzt nur Glück hatten, aber er hört nicht auf die Vernunft.

Sollte ich es trotzdem reparieren?

Aktualisieren

Der Lead hat technisch keine Autorität über mich. Nur eine Amtszeit. Er war bis vor einem Jahr der einzige Entwickler des Projekts und ich denke, er nimmt konstruktive Kritik nicht sehr gut auf. Für das, was es wert ist, habe ich ihn nicht kritisiert. Ich habe nur darauf hingewiesen, dass der Fehler, der nie aufgetaucht ist, nicht bedeutet, dass er nicht da ist.

Ich würde vorschlagen, wenn Sie Bug-Tracking haben, dann senden Sie es. Wenn es kritisch ist, erhöhen Sie es und machen Sie ihn darauf aufmerksam. Lassen Sie Ihren Vorgesetzten es im Tracker herabstufen. Wenn etwas schief geht, haben Sie die Papierspur.

Persönlich würde ich das Problem beheben, es sei denn, es erforderte erheblich mehr Aufwand als es wert war. "Wenn es nicht kaputt ist, beheben Sie es nicht" ist schrecklich, auf Software anzuwenden.

Wenn Ihr leitender Entwickler Ihr Chef ist und er sagt, berühren Sie ihn nicht, dann würde ich das nicht tun.

In den meisten Antworten und Kommentaren wurde vorgeschlagen, die Verantwortung für die Entscheidung zu verringern, indem ein Fehlerbericht erstellt und jemand anderes den Anruf tätigen ließ.

Da ich keinen Bug-Tracker habe (und ich bezweifle, dass es jemand anderes als ich selbst verwenden würde), habe ich das nächstbeste gemacht. Ich ging über den Kopf des leitenden Entwicklers hinweg. Nachdem sie dem Management die Situation erklärt hatten, sahen sie die Dinge auf meine Weise. Sie sagten mir, ich solle das Problem beheben und die Nachfrage des Leads ignorieren . Sie sagten, sie würden gekräuselte Federn glätten, wenn er jemals die Täuschung entdecken und sich beschweren würde.

Keine ideale Lösung, aber zumindest wurde der Fehler richtig behoben.

Erinnern Sie ihn daran, dass der Satz lautet: "Wenn es nicht kaputt ist, reparieren Sie es nicht" und nicht "Wenn der Kunde es nicht bemerkt hat, reparieren Sie es nicht".

Welche Rechtfertigung haben Sie für die vorgenommene Änderung? Wenn Sie nicht darauf hinweisen können, welche Änderungen der Benutzer erfahren würde oder welche technischen Schulden beseitigt wurden, würde ich mich auf die Seite des leitenden Entwicklers stellen, indem ich die Änderung zurückziehe, da dies die Situation nur verschlimmert.

Meiner Meinung nach haben Sie hier mindestens ein paar verschiedene Möglichkeiten:

Wenn Sie einfach weitermachen und den Fehler beheben, riskieren Sie, der Mischung weitere Fehler hinzuzufügen, die in meinen Gedanken nach hinten losgehen könnten. Je nachdem, wie viel Erfahrung Sie haben und wie sicher es ist, böse Überraschungen zu vermeiden, ist dies wahrscheinlich mein Leitfaden.

Wenn du tust, was dir befohlen wurde, ist es nur Schuld, die das Problem sein würde, oder ist es mehr als das? Ich frage mich, was hier anders los ist als das Zeug, das als Prinzipien und Werte bekannt ist. Ich meine, das ist ein Scherz, aber auch ein ehrlicher Punkt, was mit dieser Idee nicht stimmt.

Während mein überwältigender Instinkt darin bestehen würde, die Fehler zu beheben, ohne das Problem zu verbergen, gibt es Szenarien, in denen ich meine Nase halten und das Problem verbergen würde.

1. Code wird intern und gelegentlich verwendet, sodass die Folgen des Fehlers innerhalb des Unternehmens beherrschbar sind.
2. Überwältigende wirtschaftliche Erwägungen, die heute den Versand und eine Fehlerbehebung erforderten, konnten 2 Wochen später mit minimalen Konsequenzen ausgerollt werden.

Beruflich mag ich diese Antworten nicht und würde intern klarstellen, dass diese Situationen ätherisch sind.

Letztendlich sollten Sie nichts tun, was Ihr Vorgesetzter ausdrücklich nicht gesagt hat. Ich glaube, das Beste, was Sie in Ihrer Position tun können, ist, einen Fehlerbericht in der von Ihnen verwendeten Fehlerverfolgungsdatenbank zu erstellen. Auf diese Weise ist zumindest jeder über das Problem informiert und jemand mit mehr Befugnissen kann entscheiden, was damit geschehen soll.

Kopieren Sie die Buggy-Funktion, wenden Sie die Korrektur an, benennen Sie sie um, verkleiden Sie sie möglicherweise ein wenig, und nennen Sie sie stattdessen.

Basierend auf Ihrem Kommentar zu zwei Showstopper-Bugs ist es möglicherweise die beste Wahl, den Buchstaben des Gesetzes zu befolgen, aber seinen Geist zu ignorieren.

Natürlich gibt es einen Nachteil bei der Codierung durch Ausschneiden und Einfügen, aber das scheint das geringste Problem für Sie zu sein.