Compañero de trabajo renombró todas mis consultas [cerrado]

63

No sé si debería estar muy irritado o qué. Yo solo construí más de 300 consultas para una gran base de datos, y desarrollé una convención de nomenclatura para poder encontrarlas más tarde. Nadie más en mi oficina sabe cómo construir una consulta, pero vine ayer para encontrar que todos habían sido renombrados. Ahora estoy teniendo dificultades para encontrar cosas, y estoy tratando de averiguar qué hacer.

Hablé con la persona responsable, y ella minimizó todo el asunto. Ella dijo que los renombró para poder encontrarlos más fácilmente. Desafortunadamente, soy la única que sabe cómo construirlos, editarlos y mantenerlos, y la única razón por la que necesitaba encontrarlos era para probar las consultas. La nueva convención de nomenclatura no tiene ningún sentido, y siento que hemos dado un paso atrás en el proceso de desarrollo.

Lo que estoy tratando de resolver es:

1) ¿Estoy exagerando?

2) ¿Cuál es la mejor manera de manejar esto? Odio mencionarle esto a mi jefe, pero después de hablar ayer con mi compañero de trabajo, ya puedo decir que siente que no hizo nada malo.

anon
fuente
29
Aunque trabajamos en equipos, existe un concepto de quién posee qué, y se debe solicitar permiso antes de cambiar el código que otra persona ha creado. Una vez que lo hice (me da vergüenza decirlo) y me regañaron. Cuando me pasó a mí, lo cambié y les pedí que no lo hicieran.
Mike Dunlavey
30
Duelo al amanecer! ...
46
Tienes una copia de seguridad / SVN ¿verdad? Restaurar justo antes de sus cambios, y relajarse.
JYelton
11
¡Si alguien cambiara el nombre de mi código, les arrebataría la vida como Shang Tsung!
Glenn Ferrie
24
Si tiene hijos, cámbieles el nombre.
Matt

Respuestas:

81
  1. En realidad no, eso es algo increíblemente irrespetuoso.

  2. Has hablado con ella y no lo hemos hecho, pero parece que estarías en tu derecho de restaurar las convenciones de nomenclatura anteriores desde una copia de seguridad o revertirlas si están en un control de fuente. SÍ notifique a su jefe y compañero de trabajo si hace esto, y proporcione su razón (no puede mantener su propio trabajo).

Sin embargo, lo último que desea es entrar y salir de esto, por lo que debe manejarlo ya que la situación parece justificable, pero al menos debe documentarse en caso de que se convierta en parte de un patrón de falta de respeto.

DKnight
fuente
81
Por otro lado, si su papel en el proyecto es en realidad "probador", no tiene absolutamente ningún derecho a cambiar el código fuente. Período. Pero me gustaría agregar que, para evitar este tipo de situación, debe componer un pequeño documento (¡viñetas!) Que indique la convención de nomenclatura, ya que, en el futuro, su empresa puede contratar a alguien para que trabaje con usted, o incluso coordinar usted, y estaría dentro de sus derechos cambiar el nombre si no hay una convención oficial .
Bruno Brant
1
Estoy de acuerdo con esta respuesta Si no hace nada, establece un precedente que puede exacerbar futuros desacuerdos.
JYelton
2
Explíquele cómo funciona la convención de nomenclatura
GerManson
13
Asegúrate de quitarle sus derechos de edición a la base de datos mientras estás en ella ...
Ant
1
@Ant: Si el ejemplo del OP es toda la historia, creo que podría ser un poco duro. Si hay más (o nunca deberían tener derechos de edición), puede que tenga razón.
BCS
117

¿Por qué no lo manejas simplemente como adultos? Siéntate, sin confrontaciones, y elabora una lista de pros y contras para un esquema de nombres, acuerda uno y hazlo oficial escribiendo un breve documento que lo describa. Obtenga interés genuino en su aporte para que se sienta (y esté) involucrada.

Si es principalmente una cuestión de gustos y si ella es el tipo de persona que absolutamente tiene que tener las cosas a su manera, entonces alégrate de ser la persona más grande y déjalo ir. La vida es demasiado corta para tener un concurso de esquemas de nombres.

¿El problema es el esquema de nombres o que sientes que no te respetan? Si es así, tal vez pueda trabajar en su relación de trabajo. Si sientes que no vale la pena, ¿por qué te importa lo que ella piense de todos modos? :) Otra opción podría ser que ella realmente no siente que es un gran problema y si explicas amablemente que estás teniendo problemas para encontrar cosas, tal vez puedas cambiarlo de nuevo.

konrad
fuente
2
Iba a decir cambiarlo y pedirle que no hiciera eso, pero tu respuesta es aún mejor.
Mike Dunlavey
Esto es bastante acertado.
Wayne Molina
20
Y luego haga que lo cambie a la NUEVA convención acordada :)
77
@konrad, estás pasando por alto un problema crucial: un probador no tiene autoridad ni ningún tipo de negocio que realice ningún tipo de edición como esta en primer lugar . @anon es el programador a cargo, las convenciones de nomenclatura son propiamente su decisión de tomar, y no están listas para una votación del comité, y mucho menos cambios unilaterales por alguien que nunca tendrá que depurar el código después.
Shadur
36
  1. El diseño de la base de datos incluye permisos (GRANT y REVOKE).
  2. Las pruebas incluyen permisos de prueba.
  3. Relativamente pocas personas deberían tener permiso para cambiar el nombre de los objetos de la base de datos.
  4. Tu compañero de trabajo no es uno de los pocos.
Mike Sherrill 'Catcall'
fuente
Este es un muy buen punto. Me preguntaba por qué un probador habría tenido acceso a esto en primer lugar.
Justin Ohms
Porque es acceso. No hay CONCESIÓN y REVOCACIÓN en Access. No hay permisos en absoluto.
Kibbee
77
En realidad hay permisos de acceso, pero todavía tengo que conocer a alguien que entiende cómo se supone que deben ser utilizados (por no hablar de implementarlas)
MCHL
1
Porque sin duda, esta misma compañía probablemente ni siquiera tiene sistemas de control de fuente, y mucho menos un DBA calificado que ha bloqueado Access. por cierto deberías buscarlo, no es muy difícil de hacer.
Tipo anónimo
21

"La nueva convención de nomenclatura no tiene ningún sentido" suena como uno de estos podría ser el caso:

  1. Ella les aplicó alguna norma de la compañía. A menudo se usan para asegurarse de que el código sea al menos consistente y, en el mejor de los casos, pueden ayudar a cosas periféricas como pequeños scripts personalizados a encontrar código fácilmente. En este caso, debe comprender la norma y por qué "no tiene sentido" en su situación. Si todavía cree que es mejor que todos los desarrolladores lo dejen como estaba, explíqueles por qué exactamente su método es superior y pregunte si puede cambiar la norma (probablemente esté bien si están de acuerdo en que es superior) o renunciar en su caso (poco probable y desordenado a largo plazo).
  2. Ella inventó su propio estándar en el acto y lo aplicó. Esto es más probable si es nueva y debería explicar su razón de ser. Podría aprender algo, y / o ella podría aprender algo si luego explica su razón de ser.

Un punto importante es que no es su código (singular), si es que pertenece, y será modificado por todo el grupo. Ninguna crítica sobre el código debería centrarse en quién lo escribió.

l0b0
fuente
2
+1 buen punto. todos los que responden asumen que el interlocutor tiene un esquema de nombres válido. ¿Qué pasa si él / ella es realmente el "acaparador de información" de las empresas? Quizás sea el probador quien realmente está facilitando que todos los demás (y cualquier persona nueva) en la empresa encuentren consultas.
Tipo anónimo
Buenos puntos. Si el esquema de nombres es bueno, ambas partes podrán usarlo (una vez que se acostumbren) sin importar quién lo haya ideado.
BCS
2
@bcs Incluso en ese caso, la forma correcta de hacerlo sería informar al programador PRIMERO y pedirle que cambie la convención de nomenclatura ellos mismos, en lugar de colarse y esperar a que venga a trabajar al día siguiente. y descubre toda su estructura mutilada.
Shadur
16

Hablé con la persona responsable, y ella minimizó todo el asunto.


Entonces te diré, descaradamente:

Revertir los cambios.

Libra esta guerra. Su gerente debe respaldarlo y solidificar su autoridad.

Jim G.
fuente
Convenido. Consulte con ella primero que no hay una buena razón por la cual los nuevos nombres son mejores. Si no, simplemente cámbialo de nuevo.
Richard
11
¿Cuándo es un buen consejo "Wage this war"?
Sverre Rabbelier
3
@Sverre Rabbelier: ... Cuando un n00b intenta instalar prácticas de software subóptimas. El OP trató de razonar con ella. Ahora es el momento de solucionar el problema. Debatir ad nauseum con un n00b sobre algo tan obvio es un movimiento improductivo y malgastado.
Jim G.
44
@ Jim Tal vez el OP es el nuevo?
Joe Phillips
3
Si ella no es tu jefa, cambia esa mierda y no mires atrás. No digo guerra de salarios, pero seguro no dejes que alguien actúe como si fuera el dueño del lugar. Los gerentes de proyecto son los que se supone que deben hacer cumplir los estándares de codificación. No se trata tanto de la autoridad como de la experiencia, el orden y un entorno de trabajo productivo.
Jonathan Henson
10

1) No, no estás sobre reaccionando. Alguien cambió su trabajo sin decírselo y lo rechazó cuando le preguntó por qué. Eso es extremadamente irrespetuoso y grosero en mi humilde opinión.

2) ¿Es usted el DBA oficial, o al menos la persona que se ha convertido en el guardián del DB? Si es así, cambie los nombres y escriba un documento de convenciones sobre cómo hace las cosas. Además, escriba un documento de estilo de 'Guía del usuario' para que si alguien tiene que ir a la base de datos y encontrar algo que pueda.

Enviaría esto al grupo, sin señalar con el dedo, con una nota útil de que estaría feliz de sentarse y guiar a las personas a través de algunos de los matices de la estructura.

De lo contrario, invente convenciones en equipo y sígalas en equipo.

En una nota al margen, para alguien que tuvo que probar algo para cambiar los nombres de más de 300 consultas parece bastante infantil. ¿Cuánto tiempo perdió ella haciendo esto, y solo para poder encontrar cosas? En lugar de ir y pedirle ayuda a alguien, ella perdió su tiempo, el suyo y el de la compañía. Sin mencionar que el código probablemente se rompió cuando ella hizo esto, perdiendo el tiempo de otro miembro del equipo también.

Si yo fuera tú, esperaría hasta que te calmes un poco, trataría de hablar con ella nuevamente. Si eso no funciona, tómalo con el jefe. Ese tipo de mentalidad de vaquero eventualmente pondrá a todo el equipo en apuros.

Tyanna
fuente
8

Cambiar el nombre al azar en la base de datos podría causar que un entorno de producción se caiga. Si se hiciera referencia a esos procedimientos en algún lugar del código, podría tener graves consecuencias. Puede hacer retrocesos, pero si un probador como este realmente no sabe lo que está haciendo, no es un gran paso ver que el probador realice algunos cambios en la producción. Eso podría significar la pérdida de negocios, razón por la cual debe intentar implementar roles de usuario separados para desarrolladores y evaluadores. Lo hacemos con nuestros probadores y funciona muy bien. Los probadores a menudo lo aprecian, porque no tienen que vivir con el miedo de arruinar los datos en vivo.

Morgan Herlocker
fuente
5

Parece que no se menciona en otro lugar, pero cualquier fuente (por ejemplo, una consulta) puesta en un lugar público debe estar bajo un sistema de control de versiones.

Luego, si un compañero de trabajo cambia su esquema de nomenclatura, puede volver fácilmente a su esquema de trabajo (y ver sus cambios; y posiblemente volver a retroceder si es necesario). También vincula los cambios a usuarios específicos, para que pueda ver quién arruinó las cosas.

dr jimbob
fuente
2

No mires un caballo de regalo en la boca.

En primer lugar, la propiedad del código colectivo: no deberían ser 'suyos'.

En segundo lugar, si los han renombrado, pregunte el razonamiento en torno al nuevo esquema de nombres. O están usando las consultas, en cuyo caso es una especie de llamada; o es un primer paso para comenzar a brindarle ayuda para mantenerlos.

Si todos piensan que son 'tuyos', nunca los eliminarás y pasarás a algo nuevo.

Ardilla
fuente
2

No sé si esto se ha preguntado, pero ¿qué convención de nomenclatura es la versión oficial? Si su versión es oficial, diré que aborde el problema desde la perspectiva. Entonces, en lugar de decir "La persona X anuló todos mis cambios", simplemente diga "La persona X realizó cambios que son contrarios a las convenciones de nomenclatura oficiales". Si no hay una convención oficial, le sugiero que le haga saber que no aprecia los cambios que se realizan sin consultarlo primero.

En cualquier caso, creo que librar una "guerra" no es la respuesta. Incluso si ganas, pierdes.

Tundey
fuente
Esta es la única respuesta correcta. Si hay un estándar de nomenclatura, entonces los nombres deben cumplir con el estándar, y cualquiera que quiera que tengan otros nombres está equivocado. Si no hay un estándar de nomenclatura, debe haber un estándar de nomenclatura (el equipo, el líder tecnológico o quien sea) debe escribir uno. Las convenciones de codificación como esta son aburridas, pero son una parte esencial del tejido social que permite que un equipo funcione.
Tom Anderson
1

Ese es un comportamiento terrible. Parece que no se arrepiente, así que llévelo a su jefe y haga un caso para que se revoque su acceso hasta que pueda convencerse de no perder el tiempo.

Si su jefe no es técnico, explíquelo en términos que lo entiendan. Imagine comenzar a trabajar en una sala de correos, donde la publicación se clasifica en casilleros listos para su entrega. Decide unilateralmente clasificar los casilleros por piso y luego por apellido en lugar del sistema actual de departamento y luego por piso. Podría facilitarle la vida a corto plazo, pero sería asesinado por el otro personal de la sala de correos.

Es más que grosero. Estaría furioso

Ian
fuente
1

Además de establecer permisos para evitar que las personas al azar los cambien, también debe explicar que es su trabajo probar la funcionalidad, no puede dar ninguna garantía de confiabilidad si las personas al azar realizan cambios en el código.

Craig
fuente
1

Como todos decían, ella no debería haber hecho esto, aunque solo fuera por respeto a usted, ya que usted es el creador del mantenimiento de estas consultas.

Dicho esto, no veo que nadie mencione el hecho de que si cambió el nombre de sus consultas en primer lugar, fue porque no podía entender su convención de nomenclatura.
Por lo tanto, el problema podría resolverse fácilmente documentando su convención de nomenclatura y asegurando que los compañeros de trabajo tengan acceso al documento y puedan encontrar lo que necesitan.

También debe tener cuidado y tener en cuenta cómo otras personas encontrarán y usarán sus consultas: si su convención de nomenclatura no les permite hacer su trabajo de manera eficiente, entonces probablemente necesite mantener una lista más completa de sus consultas, utilizando quizás etiquetas y palabras clave acordadas para que otros puedan encontrar lo que están buscando.

La clave aquí, creo, es que nadie trabaja de forma aislada y la mejor manera de evitar pisar los dedos del otro es comunicarse y ponerse de acuerdo sobre las reglas básicas comunes.

Renaud Bompuis
fuente
0

Yo respondería de la misma manera, minimizando su decisión de revertir todo. Simplemente revierta sus cambios y escriba correos electrónicos muy breves a sus compañeros de trabajo:

"Cambio revertido rXXXX por ahora, porque no entendí que es la convención de nombres. Gracias por intentarlo. :)"

AareP
fuente
0

Sí, estás exagerando.

Hay algo llamado control de versiones que, entre otras cosas, está acostumbrado a no tener que vencer a los $ #! 7 de los compañeros de trabajo cuando se meten con sus cosas. Simplemente regrese a la versión anterior y bloquee el archivo, por lo que le permitirá manejar la ira. Eso le abrirá la oportunidad de explicar que realizar cambios radicales en el código que depende de las cosas de otra persona sin una razón sólida y sin preguntar primero no es simplemente incorrecto, extremadamente poco práctico y prácticamente un pecado.

Por supuesto, esto supone que su convención de nomenclatura es mejor que la de ella y que en realidad puede respaldar esta decisión con argumentos objetivos sólidos, si ese no es el caso, lo más aconsejable es comenzar a cambiar su código tan pronto como sea posible para manejar los cambios y intente proponer una convención de nombres mejor la próxima vez.

No lo lleves a tu jefe, la forma madura de resolverlo es directamente con tu compañero de trabajo, tendrás que trabajar con ella después de eso, por lo que es estúpido dañar la relación para una pelea fácilmente solucionable.

Chepech
fuente