Mitarbeiter hat alle meine Abfragen umbenannt [geschlossen]

63

Ich weiß nicht, ob ich sehr irritiert sein sollte oder was. Ich habe allein über 300 Abfragen für eine große Datenbank erstellt und eine Namenskonvention entwickelt, damit ich sie später finden kann. Niemand in meinem Büro weiß überhaupt, wie man eine Abfrage erstellt, aber ich bin gestern reingekommen und habe festgestellt, dass alle umbenannt wurden. Es fällt mir jetzt sehr schwer, Dinge zu finden, und ich versuche herauszufinden, was ich tun soll.

Ich habe mit der verantwortlichen Person gesprochen und sie hat die ganze Sache heruntergespielt. Sie sagte, sie habe sie umbenannt, damit sie sie leichter finden könne. Leider bin ich die einzige, die weiß, wie man sie erstellt, bearbeitet und verwaltet, und der einzige Grund, warum sie sie finden musste, bestand darin, die Abfragen zu testen. Die neue Namenskonvention macht überhaupt keinen Sinn, und ich glaube, wir haben einen Rückschritt im Entwicklungsprozess gemacht.

Was ich versuche herauszufinden, ist:

1) Überreagiere ich?

2) Wie gehe ich am besten damit um? Ich hasse es, dies meinem Chef gegenüber zu erwähnen, aber nachdem ich gestern mit meiner Kollegin gesprochen habe, kann ich bereits sagen, dass sie das Gefühl hat, nichts falsch gemacht zu haben.

anon
quelle
29
Obwohl wir in Teams arbeiten, gibt es ein Konzept, wem was gehört, und Sie sollten die Erlaubnis einholen, bevor Sie den Code ändern, den jemand anderes verfasst hat. Einmal habe ich es getan (ich schäme mich zu sagen) und wurde zurechtgewiesen. Wenn es mir passiert ist, habe ich es zurückgesetzt und sie gebeten, das nicht zu tun.
Mike Dunlavey
30
Duell im Morgengrauen! ...
46
Du hast ein Backup / SVN oder? Wiederherstellen, kurz bevor sie sich umzieht, und entspannen.
JYelton
11
Wenn jemand meinen Code umbenennen würde, würde ich ihm das Leben rauben wie Shang Tsung!
Glenn Ferrie
24
Wenn sie Kinder hat, benennen Sie sie um.
Matt

Antworten:

81
  1. Nicht wirklich - das ist eine unglaublich respektlose Sache.

  2. Sie haben mit ihr gesprochen und wir haben es nicht getan, aber es scheint, als ob Sie im Rahmen Ihrer Rechte wären, die vorherigen Namenskonventionen aus einer Sicherung wiederherzustellen oder sie zurückzusetzen, wenn sie sich in einer Quellcodeverwaltung befinden. Informieren Sie Ihren Chef und Kollegen, wenn Sie dies tun, und geben Sie Ihren Grund an (Sie können Ihre eigene Arbeit nicht aufrechterhalten).

Das Letzte, was Sie wollen, ist ein Hin und Her darüber, obwohl Sie es so behandeln, wie es die Situation zu rechtfertigen scheint, aber es sollte zumindest dokumentiert werden, falls es Teil eines Musters der Missachtung wird.

DKnight
quelle
81
Abgesehen davon hat sie, wenn ihre Rolle in dem Projekt tatsächlich "Tester" ist, absolut kein Recht , den Quellcode zu ändern. Zeitraum. Aber ich möchte hinzufügen, dass Sie, um eine solche Situation zu vermeiden, ein kleines Dokument (Aufzählungszeichen!) Verfassen sollten, in dem die Namenskonvention aufgeführt ist, da Ihr Unternehmen in Zukunft möglicherweise jemanden einstellen oder sogar koordinieren kann Sie, und es wäre in seinem Recht, die Benennung zu ändern, wenn es keine offizielle Konvention gibt.
Bruno Brant
1
Ich stimme dieser Antwort zu. Wenn Sie nichts unternehmen, legen Sie einen Präzedenzfall fest, der zukünftige Meinungsverschiedenheiten verschärfen kann.
JYelton
2
Erklären Sie ihr, wie Ihre Namenskonvention funktioniert
GerManson
13
Stellen Sie sicher, dass Sie ihre Bearbeitungsrechte für die Datenbank entfernen, während Sie gerade dabei sind ...
Ant
1
@Ant: Wenn das Beispiel des OP die ganze Geschichte ist, denke ich, dass das ein bisschen hart sein könnte. Wenn mehr dahinter steckt (oder sie sollten überhaupt keine Bearbeitungsrechte haben), haben Sie vielleicht Recht.
BCS
117

Warum gehst du nicht einfach wie ein Erwachsener damit um? Setze dich ohne Konfrontation hin und finde eine Liste mit Vor- und Nachteilen für ein Namensschema, stimme einer zu und mache es offiziell, indem du ein kurzes Dokument schreibst, das es beschreibt. Echte Interesse an ihrem Input wecken, damit sie sich involviert fühlt (und ist).

Wenn es hauptsächlich um den Geschmack geht und wenn sie die Art von Person ist, die die Dinge unbedingt im Griff haben muss, dann sei einfach froh, dass du die größere Person bist und lass es los. Das Leben ist zu kurz, um einen verdammten Wettbewerb mit Namensschemata zu veranstalten.

Ist das Problem das Namensschema oder hast du das Gefühl, dass du keinen Respekt bekommst? Wenn ja, können Sie vielleicht an Ihrer Arbeitsbeziehung arbeiten. Wenn Sie der Meinung sind, dass es sich nicht lohnt, warum interessiert es Sie dann überhaupt, was sie denkt? :) Eine andere Möglichkeit könnte sein, dass sie wirklich nicht das Gefühl hat, dass es eine große Sache ist, und wenn Sie nett erklären, dass Sie Probleme haben, Sachen zu finden, können Sie es vielleicht wieder ändern.

konrad
quelle
2
Ich wollte sagen, ändere es zurück und bitte sie, das nicht zu tun, aber deine Antwort ist noch besser.
Mike Dunlavey
Das ist ziemlich genau richtig.
Wayne Molina
20
Und dann machen sie es auf die NEUE vereinbarte Konvention zu ändern :)
7
@konrad, Sie übersehen ein entscheidendes Problem - ein Tester hat weder Autorität noch irgendeine Art von Geschäft, das eine solche Bearbeitung vornimmt . @anon ist der verantwortliche Programmierer. Namenskonventionen sind seine Entscheidung und stehen nicht für eine Abstimmung im Ausschuss zur Verfügung, geschweige denn für einseitige Änderungen durch jemanden, der den Code später nie mehr debuggen muss.
Shadur
36
  1. Das Datenbankdesign umfasst Berechtigungen (GRANT und REVOKE).
  2. Das Testen umfasst Testberechtigungen.
  3. Relativ wenige Personen sollten die Berechtigung haben, Datenbankobjekte umzubenennen.
  4. Ihr Mitarbeiter ist nicht einer der wenigen.
Mike Sherrill 'Catcall'
quelle
Das ist ein sehr guter Punkt. Ich habe mich gefragt, warum ein Tester überhaupt Zugang dazu gehabt hätte.
Justin Ohms
Weil es Access ist. In Access gibt es kein GRANT und REVOKE. Es gibt überhaupt keine Berechtigungen.
Kibbee
7
Eigentlich gibt es Berechtigungen in Access, aber ich muss noch jemanden treffen, der versteht, wie sie verwendet werden sollen (geschweige denn, sie zu implementieren)
Mchl
1
Ohne Zweifel hat das gleiche Unternehmen wahrscheinlich nicht einmal Versionsverwaltungssysteme, geschweige denn einen qualifizierten DBA, der Access gesperrt hat. Übrigens sollten Sie nachschlagen, es ist nicht sehr schwer zu tun.
Anonymous Type
21

"Die neue Namenskonvention macht überhaupt keinen Sinn" klingt wie einer der folgenden Fälle:

  1. Sie wendete eine Firmennorm auf sie an. Diese werden häufig verwendet, um sicherzustellen, dass der Code mindestens konsistent ist, und können im besten Fall dazu beitragen, dass periphere Dinge wie kleine benutzerdefinierte Skripte den Code leicht finden. In diesem Fall müssen Sie die Norm verstehen und wissen, warum sie in Ihrer Situation "keinen Sinn ergibt". Wenn Sie immer noch der Meinung sind, dass es für alle Entwickler besser ist , es so zu lassen, wie es war, erklären Sie ihnen, warum genau Ihre Methode überlegen ist, und fragen Sie, ob Sie die Norm ändern könnten (wahrscheinlich in Ordnung, wenn sie der Meinung sind, dass sie überlegen ist) oder verzichten Sie in Ihrem Fall darauf (unwahrscheinlich und auf lange Sicht chaotisch).
  2. Sie hat ihren eigenen Standard vor Ort erfunden und angewendet. Dies ist wahrscheinlicher, wenn sie neu ist, und sie sollte ihre Gründe erklären. Sie könnten etwas lernen und / oder sie könnte etwas lernen, wenn Sie dann Ihre Gründe erläutern.

Ein wichtiger Punkt ist, dass es sich nicht um Ihren (singulären) Code handelt, sondern vielmehr um einen Code, der zu der gesamten Gruppe gehört und von dieser modifiziert wird. Keine Kritik an Code sollte sich darauf konzentrieren, wer ihn geschrieben hat.

l0b0
quelle
2
+1 guter Punkt. Alle anderen Antworten setzen voraus, dass der Fragesteller ein gültiges Namensschema hat. Was ist, wenn er / sie wirklich der "Informationshorter" des Unternehmens ist? Vielleicht ist es der Tester, der es allen anderen (und allen neuen) im Unternehmen tatsächlich leichter macht, Fragen zu finden.
Anonymous Type
Gute Argumente. Wenn das Benennungsschema gut ist, können beide Parteien es verwenden (sobald sie sich daran gewöhnt haben), unabhängig davon, wer es erfunden hat.
BCS
2
@bcs Selbst in diesem Fall wäre es die richtige Methode, den Programmierer ZUERST zu informieren und ihn / sie zu bitten, die Namenskonvention selbst zu ändern, anstatt sie einzuschleichen und darauf zu warten, dass er / sie am nächsten Tag zur Arbeit kommt und entdecken Sie seine / ihre gesamte Struktur zerstört.
Shadur
16

Ich habe mit der verantwortlichen Person gesprochen und sie hat die ganze Sache heruntergespielt.


Dann sage ich dir unverhohlen:

Machen Sie die Änderungen rückgängig.

Führe diesen Krieg. Ihr Manager sollte Sie unterstützen und Ihre Autorität festigen.

Jim G.
quelle
Einverstanden. Fragen Sie sie zuerst, ob es keinen guten Grund gibt, warum die neuen Namen besser sind. Wenn nicht, ändern Sie es einfach zurück.
Richard
11
Wann ist "Wage this war" jemals ein guter Rat?
Sverre Rabbelier
3
@Sverre Rabbelier: ... Wenn ein n00b versucht, suboptimale Software-Praktiken zu installieren. Das OP versuchte mit ihr zu argumentieren. Jetzt ist es Zeit, das Problem zu beheben. Ad nauseum mit einem n00b über etwas so Offensichtliches zu diskutieren, ist unproduktiv und verschwendete Bewegung.
Jim G.
4
@ Jim Vielleicht ist der OP der Neuling?
Joe Phillips
3
Wenn sie nicht Ihre Chefin ist, ändern Sie diese Scheiße und schauen Sie nicht zurück. Ich sage nicht Krieg führen, aber lassen Sie nicht einfach jemanden so tun, als ob ihm der Ort gehört. Projektmanager sind diejenigen, die Codierungsstandards durchsetzen sollen. Es geht weniger um Autorität als um Erfahrung, Ordnung und ein produktives Arbeitsumfeld.
Jonathan Henson
10

1) Nein, Sie sind nicht überreagiert. Jemand hat deine Arbeit geändert, ohne es dir zu sagen, und sie abgeblättert, als du sie gefragt hast, warum. Das ist imho extrem respektlos und unhöflich.

2) Sind Sie der offizielle DBA oder zumindest die Person, die zum Bewahrer der DB ernannt wurde? Wenn ja, ändern Sie die Namen zurück und schreiben Sie ein Konventionsdokument auf. Schreiben Sie auch ein Dokument im Stil eines 'Users Guide' (Benutzerhandbuch), damit jemand in die Datenbank gehen und etwas finden kann, wenn er dies tun muss.

Ich würde dies an die Gruppe senden, ohne mit den Fingern zu zeigen, mit einer hilfreichen Notiz, dass Sie sich gerne hinsetzen und die Leute durch einige der Nuancen der Struktur führen würden.

Wenn nicht, dann erfinde als Team Konventionen und folge ihnen als Team.

Nebenbei bemerkt, für jemanden, der etwas testen musste, um die Namen von mehr als 300 Abfragen zu ändern, scheint das ziemlich kindisch zu sein. Wie viel Zeit verschwendete sie damit und nur damit sie etwas finden konnte? Anstatt nur jemanden um Hilfe zu bitten, verschwendete sie ihre Zeit, Ihre Zeit und die Zeit des Unternehmens. Ganz zu schweigen davon, dass der Code wahrscheinlich kaputt gegangen ist, als sie das getan hat, wodurch auch die Zeit eines anderen Teammitglieds verschwendet wurde.

Wenn ich Sie wäre, würde ich warten, bis Sie sich etwas abgekühlt haben, versuchen Sie es erneut und sprechen Sie mit ihr. Wenn das nicht funktioniert, nehmen Sie es mit dem Chef auf. Diese Art von Cowboy-Mentalität wird das gesamte Team irgendwann in Schwierigkeiten bringen.

Tyanna
quelle
8

Das willkürliche Umbenennen in der Datenbank kann leicht zum Ausfall einer Produktionsumgebung führen. Wenn auf diese Prozeduren irgendwo im Code verwiesen wird, kann dies schwerwiegende Folgen haben. Sie können Rollbacks durchführen, aber wenn ein Tester wie dieser wirklich nicht weiß, was er tut, ist es nicht weit davon entfernt, zu sehen, dass der Tester einige Änderungen an der Produktion vornimmt. Dies kann zum Verlust des Geschäfts führen. Aus diesem Grund sollten Sie versuchen, separate Benutzerrollen für Entwickler und Tester zu implementieren. Wir machen das mit unseren Testern und es funktioniert großartig. Tester wissen dies oft zu schätzen, da sie keine Angst haben müssen, Live-Daten zu vermasseln.

Morgan Herlocker
quelle
5

Es scheint an keiner anderen Stelle angesprochen zu werden, aber jede Quelle (z. B. eine Abfrage), die an einem öffentlichen Ort abgelegt wird, sollte einem Versionskontrollsystem unterliegen.

Wenn dann ein Mitarbeiter Ihr Benennungsschema ändert, können Sie problemlos zu Ihrem Arbeitsschema zurückkehren (und deren Änderungen anzeigen und gegebenenfalls zurücksetzen). Sie binden Änderungen auch an bestimmte Benutzer, sodass Sie sehen können, wer die Dinge durcheinander gebracht hat.

Dr. Jimbob
quelle
2

Schau keinem Geschenkpferd in den Mund.

Erstens, kollektiver Code-Besitz - er sollte nicht "Ihnen" gehören.

Zweitens, wenn sie umbenannt wurden, fragen Sie nach der Begründung für das neue Benennungsschema. Entweder verwenden sie die Abfragen - in diesem Fall ist es eine Art Anruf. oder es ist ein erster Schritt, um Ihnen dabei zu helfen, diese zu pflegen.

Wenn jeder denkt, sie gehören dir, wirst du sie niemals los und machst mit etwas Neuem weiter

Eichhörnchen
quelle
2

Ich weiß nicht, ob dies gefragt wurde, aber welche Namenskonvention ist die offizielle Version? Wenn Ihre Version offiziell ist, werde ich das Problem aus der Perspektive ansprechen. Anstatt also "Person X hat alle meine Änderungen zurückgesetzt" zu sagen, sagen Sie einfach "Person X hat Änderungen vorgenommen, die den offiziellen Namenskonventionen widersprechen". Wenn es keine offizielle Konvention gibt, dann schlage ich vor, sie wissen zu lassen, dass Sie die vorgenommenen Änderungen nicht schätzen, ohne Sie vorher zu konsultieren.

In beiden Fällen denke ich, einen "Krieg" zu führen, ist nicht die Antwort. Selbst wenn Sie gewinnen, verlieren Sie.

Tundey
quelle
Dies ist die einzig richtige Antwort. Wenn es einen Namensstandard gibt, sollten die Namen dem Standard entsprechen, und jeder, der möchte, dass sie andere Namen haben, ist falsch. Wenn es keinen Namensstandard gibt, sollte es einen Namensstandard geben - das Team oder der technische Leiter oder wer auch immer - sollte einen schreiben. Kodierungskonventionen wie diese sind langweilig, aber sie sind ein wesentlicher Bestandteil des sozialen Gefüges, das einem Team das Arbeiten ermöglicht.
Tom Anderson
1

Das ist ein entsetzliches Verhalten. Es hört sich so an, als ob sie es nicht bereut. Bringen Sie es zu Ihrem Chef und lassen Sie sich den Zugang entziehen, bis sie überzeugt ist, nicht herumzuspielen.

Wenn Ihr Chef nicht technisch ist, erklären Sie es in Begriffen, die er verstehen wird. Stellen Sie sich vor, Sie beginnen die Arbeit in einem Postraum, in dem die Post in versandfertige Taubenlöcher sortiert wird. Sie entscheiden sich einseitig, die Taubenlöcher nach Stockwerk und Nachname zu sortieren, anstatt nach dem aktuellen Abteilungssystem und Stockwerk. Es könnte kurzfristig Ihr Leben erleichtern, aber Sie würden von den anderen Poststellenmitarbeitern ermordet.

Es ist unhöflich. Ich wäre wütend.

Ian
quelle
1

Abgesehen davon, dass Sie Berechtigungen festlegen, die verhindern sollen, dass zufällige Personen sie ändern, sollten Sie auch erklären, dass es ihre Aufgabe ist, die Funktionalität zu testen. Sie können keine Garantie für die Zuverlässigkeit geben, wenn zufällige Personen Änderungen am Code vornehmen.

Craig
quelle
1

Wie alle sagten, hätte sie dies nicht tun sollen, schon aus Respekt vor Ihnen, da Sie der Urheber dieser Abfragen sind.

Abgesehen davon sehe ich niemanden, der die Tatsache erwähnt, dass sie, wenn sie Ihre Abfragen überhaupt umbenannte, Ihre Namenskonvention nicht verstehen konnte.
Das Problem kann also leicht gelöst werden, indem Sie Ihre Namenskonvention dokumentieren und sicherstellen, dass Mitarbeiter Zugriff auf das Dokument haben und finden, was sie benötigen.

Sie müssen auch vorsichtig sein und berücksichtigen, wie andere Leute Ihre Abfragen finden und verwenden: Wenn Ihre Namenskonvention es ihnen nicht ermöglicht, ihre Arbeit effizient zu erledigen, müssen Sie wahrscheinlich eine gründlichere Liste Ihrer Abfragen führen, indem Sie möglicherweise verwenden Tags und vereinbarte Stichwörter, damit andere finden können, wonach sie suchen.

Der Schlüssel hier ist meiner Meinung nach, dass niemand isoliert arbeitet und der beste Weg, um zu vermeiden, dass man sich die Zehen wechselt, ist zu kommunizieren und sich auf gemeinsame Grundregeln zu einigen.

Renaud Bompuis
quelle
0

Ich würde auf die gleiche Weise reagieren - Ihre Entscheidung, alles zurückzusetzen, herunterzuspielen. Machen Sie einfach ihre Änderungen rückgängig und schreiben Sie Ihren Mitarbeitern eine kurze E-Mail:

"Die Änderung von rXXXX wurde vorerst rückgängig gemacht, da die Namenskonvention nicht verstanden wurde. Vielen Dank, dass Sie es trotzdem versucht haben. :)"

AareP
quelle
0

Ja, Sie reagieren übermäßig.

Dort gibt es eine sogenannte Versionskontrolle, die es unter anderem verwendet, um die $ #! 7 von Kollegen nicht zu besiegen, wenn sie sich mit Ihren Sachen anlegen. Machen Sie einfach einen Rollback zur vorherigen Version und sperren Sie die Datei, damit sie mit dem Ärger umgehen kann. Dies eröffnet Ihnen die Möglichkeit zu erklären, dass radikale Änderungen an Code, der von fremden Inhalten abhängig ist, ohne stichhaltigen Grund und ohne vorherige Rückfrage nicht einfach falsch, äußerst unpraktisch und praktisch eine Sünde sind.

Dies setzt natürlich voraus, dass Ihre Namenskonvention besser ist als die von ihr und dass Sie diese Entscheidung tatsächlich mit soliden objektiven Argumenten untermauern können, wenn dies nicht der Fall ist Versuchen Sie das nächste Mal, eine bessere Namenskonvention zu finden.

Bring es nicht zu deinem Chef, die ausgereifte Art, es zu lösen, liegt direkt bei deiner Kollegin. Danach musst du mit ihr zusammenarbeiten, also ist es dumm, die Beziehung für einen leicht lösbaren Streit zu beschädigen.

Chepech
quelle