Cómo manejar los comentarios anónimos de un colega desconocido a través del administrador

Recibí comentarios de mi gerente de que un colega encontró hiriente un comentario mío sobre la revisión del código. Mantuvo el nombre confidencial.

Para mí es importante que la revisión del código siga siendo una interacción positiva y colaborativa, por lo que me preocupa profundamente.

Revisé mis comentarios recientes y no encontré nada que me pareciera objetable, pero ese es el problema en pocas palabras, ¿no?

¿Qué debo hacer, si es que debo hacer algo, en esta situación?

Si ni siquiera le dirán quién, entonces es tan útil como un voto negativo en SE.
Este tipo de comentarios no específicos es completamente inútil. Al menos debería haber proporcionado las palabras de las que se quejó. De lo contrario, no hay caso para responder. Si no puede decírtelo sin violar la privacidad de la otra persona, pregúntale si él mismo lo habría encontrado hiriente o, mejor aún, si una persona razonable lo habría hecho: es decir, ¿es mi problema o el del otro?
¿Tiene su empresa algún medio para solicitar comentarios directos o anónimos de sus compañeros? Si es así, sugeriría usarlo como una forma de pedir una crítica constructiva. Puede descubrir que ha estado molestando a las personas durante bastante tiempo, incluso fuera de la revisión del código. Si su empresa no lo hace, siempre puede configurar un formulario de Google o utilizar una de las muchas herramientas de encuestas anónimas en línea.
Las revisiones de código pueden ser algo extremadamente positivo para el desarrollo de software. Desafortunadamente, algunas personas se ofenden fácilmente y las revisiones terminan siendo más dañinas para el equipo que cualquier valor que puedan agregar. Esto no es un problema tuyo, es un problema de la persona con un caparazón delgado. Desafortunadamente, tienes que tratar con esta persona. Lo que he visto que funciona es que finalmente se vuelve obvio quién es la persona y, básicamente, nadie proporciona comentarios sustantivos sobre su código y las personas mayores simplemente cambian el código ellos mismos. Esto no ayuda a nadie, pero elimina los sentimientos heridos.
¿Son sus comentarios en la revisión del código del mismo tono que los comentarios que se encuentran a menudo en SE? Si es así, ese puede ser el problema.
No veo una forma razonable de brindarle la información que necesita y al mismo tiempo mantener al denunciante en el anonimato. Por ejemplo, si sabe qué comentario fue, puede ver en los registros de confirmación quién fue (o adivinar).
Supongo que el gerente no dirá qué comentario porque eso también revelaría al quejoso.
¿Alguno de sus comentarios es algo no constructivo o no objetivo? por ejemplo, "Este código es malo" o expresar una opinión negativa en lugar de explicar un problema específico o sugerir una mejor alternativa? Si es así, puede que sea esto.
Su gerente tiene la mayor parte de la culpa aquí... debería haber revisado de qué se trataba el comentario. Si no estaba de acuerdo con la queja, debería habérselo dicho al quejoso; si estuvo de acuerdo con la queja, entonces debería haberte dicho " Este comentario me parece ofensivo" y así podrías discutir con él cómo mejorarlo, sin necesidad de revelar quién se quejó originalmente. Solo decir "alguien me dijo X" no es muy profesional por su parte, es más el comportamiento que encuentras en un salón de clases para niños.
@Brandin: veo un par de formas razonables y las describí en mi respuesta a continuación .
Si alguna vez estoy en una posición de autoridad o soy dueño de una empresa, y mis empleados ni siquiera pueden arriesgarse a la idea de que alguien sepa que no les gustó una interacción con esa persona, forzaré el asunto. No es suficiente tener competencia técnica; Quiero empleados que puedan trabajar en equipo con otros seres humanos . Y si no pueden tolerar ni siquiera la confrontación más leve para discutir sentimientos, actitudes, pensamientos e intenciones con los demás, entonces creo que es hora de que comiencen a salir de mi organización. Para gritar en voz alta, las personas son violetas de piel fina que se encogen a veces.
Esta es más una sugerencia para escribir comentarios no ofenvvos. Las personas tienden a defenderse cuando se les dirige un comentario. Nunca se dirija a la persona en la revisión del código, como 'debería haberlo hecho de esta manera', sino que el problema/asunto debe ser el tema en sus comentarios como 'esta línea/clase tiene un problema y debería ser así'.

Respuestas (12)

  1. "Recibí comentarios de mi gerente de que un colega encontró hiriente un comentario mío sobre la revisión del código".

  2. "Revisé mis comentarios recientes y no encontré nada que me pareciera objetable"

Debe comunicarse con su gerente y utilizar al gerente como intermediario para conciliar estas dos declaraciones.

Una vez que las palabras salen de nuestra boca, toman vida y significado propio. De vez en cuando, la gente leerá en nuestras palabras significados e implicaciones que nunca sospechamos que estaban allí. Les diré por amarga experiencia personal que la creatividad de algunas de las personas con las que me encontré al interpretar mis palabras de una manera que ofendió ellos fue asombroso, y eso es decirlo suavemente.

Habla con tu gerente y dile que no tienes ni idea de lo que dijiste que fue hiriente y que definitivamente necesitas una aclaración. Dile que te disculparás si eres el origen de la falta de comunicación y que quieres aclarar las cosas si no eres el origen de la falta de comunicación.

Si me colocaran en un puesto como el suyo y el gerente se mostrara reacio a identificar a la persona o el comentario de retroalimentación, le señalaría que solo puedo abordar aquellas quejas que son lo suficientemente específicas como para ser procesables. No se puede esperar que adivine cuál es la queja. Si un gerente me pusiera en una posición en la que tengo que adivinar cuál es la queja, lo atacaría sin dudarlo un segundo. Su posición es su problema, no el mío. Me informó de una queja con la expectativa de que la abordaría. Me guste o no, espero su total cooperación para darme los detalles de la queja. Período.

Sin embargo, es posible que tengas que aceptar que esto no es posible. Saber el comentario específico te dirá exactamente quién está ofendido, e incluso un falso genéricamente similar podría hacer lo mismo.
Moz - Preste atención a lo que lee: el colega dijo que encontró hirientes los comentarios. Si aún no sabe quién resultó herido después de que el gerente dijera que era el colega, no hay nada que pueda hacer por usted.
Lo siento, usé la palabra incorrecta. Por favor, por "ofendido" léase "herido".
Creo que @Móż está señalando que el nombre del denunciante hasta ahora se ha ocultado del OP, y revelar exactamente qué parte de la retroalimentación generó la denuncia inevitablemente revelaría la identidad del denunciante, por lo que el gerente puede ser reacio a hacerlo. .
@AakashM: le señalaría al gerente que solo puedo abordar aquellas quejas que son lo suficientemente específicas como para ser procesables. No se puede esperar que adivine cuál es la queja. Si un gerente me pusiera en una posición en la que tengo que adivinar cuál es la queja, lo atacaría sin dudarlo un segundo. Su posición es su problema, no el mío. Me informó de una queja con la expectativa de que la abordaría. Me guste o no, espero su total cooperación para darme los detalles de la queja. Período.
@VietnhiPhuvan Creo que el último comentario es mejor que tu respuesta. OP debería negarse absolutamente a participar en juegos de adivinanzas y, de hecho, debería presionar al gerente lo más fuerte posible.
@VietnhiPhuvan Para ampliar eso, si esto es grave, OP tiene derecho a (a) ser tratado como inocente hasta que se demuestre su culpabilidad, (b) ser confrontado con un cargo específico y responsable, y (c) confrontar al acusador. De lo contrario, es una tontería sin sentido y poco profesional.
@EJP Es posible que no necesariamente desee (c). Si la otra persona se ha ofendido por un comentario razonable del OP, es probable que no tome bien la confrontación. Y, por el contrario, si el OP fue realmente ofensivo y no se dio cuenta, es probable que el OP empeore las cosas. El gerente está en una posición perfecta para mediar en esto. Si (como usted dice) el OP puede obtener ofertas de lo que objeta la otra persona, entonces el gerente puede decidir el curso de acción apropiado sin potencialmente iniciar una gran fila en el equipo.
Sí. excepto que no puede mediar algo de lo que la otra persona no es consciente. "Hiciste algo mal" no es acitonable en absoluto. Tal vez algún príncipe/princesa presioux se sintió ofendido por un comentario de "esta es la llamada de API incorrecta" cuando en realidad ES la llamada de API incorrecta; solo mire con qué facilidad se ofende la gente en estos días. A menos que el OP haya escrito un comentario CLARAMENTE insultante, es bastante difícil entrar en un juego de adivinanzas, o incluso analizar si todo está justificado o no para empezar.
Si bien estoy de acuerdo con usted en que el OP debe manejar esto hablando con su gerente, no puedo en conciencia votar a favor de su respuesta, porque creo que la manera en que sugiere que lo hagan es muy mal elegida y es poco probable que convenza al gerente de que el OP realmente quiere mejorar sus habilidades de comunicación en lugar de simplemente echar la culpa. En cambio, voté a favor de la respuesta de Steve Jessop , que brinda el mismo consejo básico, pero con (en mi opinión) una actitud mucho más constructiva y apropiada.

¿Qué debo hacer, si es que debo hacer algo, en esta situación?

No se supone que las revisiones de código sean concursos de popularidad. No se preocupe, su gerente no está lo suficientemente preocupado como para darle los detalles, por lo que es solo un "aviso", así que tenga cuidado con los comentarios en el futuro. Asegúrese de que sean profesionales y constructivos.

También es posible que el gerente haya malinterpretado el comentario: tal vez el revisor quiso decir que la revisión del código fue una experiencia de aprendizaje dura y dolorosa (descubrir que su trabajo no es tan impecable como cree siempre es un poco doloroso/hiriente), y el gerente malinterpretó esto como una queja. El trabajo del gerente es mediar, comprender el problema, obtener detalles importantes del evaluado, si los hay, y comunicarlos.
Si el gerente pensó lo suficiente como para mencionarlo, entonces debe querer alguna acción (o eso o se siente obligado por la política), por lo que creo que "No se preocupe por eso" es un consejo bastante malo en este caso. El gerente obviamente, en algún nivel, dejó en claro que las revisiones de código no deberían ser un foro para culpar.

Participo en parte con Kilisi: una revisión de código no es un "concurso de popularidad". Se trata de cuánto un conjunto neutral de caracteres cumple con los principios establecidos del lenguaje del código, el estilo y el propósito de la codificación (y, con suerte, también un mayor desarrollo). ¡No es un lugar para lastimar y, por lo tanto, no es un lugar para sentirse ofendido al mismo tiempo!

En este caso específico también se debe considerar a la comunidad y al administrador :

  • Si el revisor tiene amplios conocimientos, pero el remitente no apoya la atmósfera profesional, el "sentimiento de dolor" puede no ser razonable.
  • Si el gerente no tiene buenas habilidades para manejar esta situación, simplemente aceptará al herido y le echará la responsabilidad al revisor. Un enfoque profesional sería pedirle al remitente que aclare qué fue incorrecto, para que así el gerente pueda abordar el tipo de problema con el revisor.

Una de las comunicaciones que apenas tolero es la referencia descuidada. "Te digo algo, se supone que debes trabajar en lo que realmente quiero decir con eso, y luego lo que espero ver, luego cómo encajar allí y luego evitar una situación casi similar conmigo". Totalmente injusto y poco profesional. Así es como entran en juego los efectos del jardín de infantes y servirlos solo empeora las cosas. "Se supone que el experto no debe herir los sentimientos de un remitente". ¿Alguna vez has escuchado esto como una regla general abierta para seguir las pistas? No me parece. En un entorno profesional esto no tiene sentido, porque está fuera de cuestión y de negocio.

La pregunta siempre será: ¿fue el comentario lo suficientemente malo como para justificar esa respuesta? Nunca sabremos. Si fue solo un comentario de revisión constructivo, sin virulencia y golpes al remitente, entonces tenemos un problema: mierda en la ponchera, por así decirlo. La gerencia que no hace su trabajo (potencialmente) se volvería loca cuando algunas personas perciban que podrían jugar el proceso de esta manera. Lo descartaría como "aviso" y simplemente me prepararía, de manera no agresiva, en caso de que surja un problema la próxima vez. Esta vez podría valer la pena dar la vuelta a la mesa y hablar con el gerente en privado.
Al final, no estoy seguro de cuál es el consejo aquí, específicamente esta línea: "En un entorno profesional, esto no tiene sentido, porque está fuera de cuestión y de negocio".
El consejo es que mantenga al gerente y al remitente también en línea si demuestran que no manejan la situación de manera profesional, y verifique si los comentarios son relevantes o razonables. Si el comentario de revisión específico fue, por ejemplo, "poner más esfuerzo en un código limpio", los comentarios no son razonables. Si el comentario fue, por ejemplo, "este código es muy malo", eso no es informativo y los comentarios son razonables. ¿Ves mi punto ahora?
Si 'poner más esfuerzo en el código limpio' es muy amplio y posiblemente inútil si no están seguros de lo que quiere decir con 'código limpio'.
Lo siento, pero esto falla en una hipotética simple: 'esta clase está hinchada y necesita refactorizarse' frente a Torvalds-esque 'quien diablos escribió este fragmento de código sh^% NO mezclamos estas preocupaciones'. Ambos transmiten más o menos el mismo contenido técnico. Alguien podría encontrar objetable el tono del primero, cualquier persona razonable se sentiría ofendida por el segundo.
@182764125216: Como escribió Jared Smith, hay una diferencia importante en ambos casos. Aún así, en un principio con nombre, puede formular preguntas y proporcionar sugerencias para seguir adelante. Una retroalimentación como "me has lastimado" simplemente no es suficiente para poder reaccionar de manera constructiva. Si al remitente no le importa lo suficiente como para contar los detalles de lo que le duele, entonces es solo un intercambio basado en la opinión, y difícilmente puede razonar si tiene algo que ver con las buenas prácticas del proceso, o es una sensibilidad excesiva individual. Y está mal cubrir incondicionalmente los sentimientos sensibles de alguien.

Revisé mis comentarios recientes y no encontré nada que me pareciera objetable, pero ese es el problema en pocas palabras, ¿no?

Revise sus revisiones de código recientes y lea cada uno de sus comentarios en el tono más sarcástico y mezquino que pueda reunir. En voz alta. (Probablemente desde casa)

Si ninguno de sus comentarios parece coincidir, cuando se le da ese tono, probablemente esté bien y su compañero de trabajo estaba teniendo un mal día.

Esto es básicamente un tl; dr del consejo de Steverr Robbins . También tiene más buenos consejos , pero básicamente todo se reduce a tener cuidado con lo que escribes, porque se puede tomar de muchas maneras diferentes.

Sí, es muy difícil medir el tono del mensaje por correo electrónico o cualquier otro medio de texto.

Puede ser perjudicial para su gerente, pero lo que ha hecho ha sido más que inútil.

Has perdido el tiempo comprobando dónde podrías haber hecho algo que te pareció ofensivo. Esa pérdida de tiempo la provoca directamente su jefe. Ahora le preocupa cómo escribir sus revisiones de código, lo que podría hacer que sus revisiones de código sean menos útiles en el futuro. Ese daño es causado directamente por su gerente.

Si él no hubiera dicho nada, las cosas habrían estado bien. Si te hubiera dicho qué declaración exactamente se suponía que era hiriente, entonces le habrías explicado a la persona en cuestión que no tenía la intención de ser hiriente, o le habrías explicado que lo era. El problema se habría solucionado.

Pero debido a la estúpida acción del gerente, solo hubo una pérdida de tiempo sin resultado. Creo que el gerente realmente debería pensar en cómo está actuando. Y si el gerente lo toma como algo hiriente, puede comentar. Anónimo probablemente.

PD. Al leer algunos comentarios aquí, algunas personas parecen pensar que el denunciante debería tener anonimato. Bueno, no me importaría quién es el que se queja, pero una vez que sé de qué se está quejando, puedo aclarar cualquier malentendido, disculparme si me equivoqué o decirles que lo que escribí era exactamente lo que quería decir y si les resulta hiriente, ese es su problema.

Aquí también hay otro problema, al menos en mi humilde opinión. Por lo que recuerdo haber hecho revisiones de código en algunas de las organizaciones anteriores para las que trabajé, fue un esfuerzo de equipo. Sería discutido, no como el palo al final de una zanahoria, fue una experiencia de equipo constructiva . El proceso de la regla del puño de hierro no es un buen enfoque, como siempre. Si escribiera un comentario de revisión desagradable, eso me llevaría rápidamente a través de la puerta de la oficina de recursos humanos. Probablemente dos veces, golpeando mi botín al salir...
"Si él no hubiera dicho nada, las cosas habrían estado bien". Muy en desacuerdo. Pretender vivir en un mundo donde la moral de los empleados no afecta la producción es peor que inútil.
"algunas personas parecen pensar que el denunciante debería tener anonimato" o, en todo caso, que tienen anonimato dentro del sistema vigente.
A diferentes programadores les gustan diferentes tipos de comentarios. Algunos programadores realmente quieren comentarios negativos, sin importar cuán pequeños sean, para poder sopesarlos y posiblemente mejorar a largo plazo. Algunos encuentran que cada comentario es una tarea y preferirían simplemente terminar su código. Trabajar con personas que son diferentes requiere técnicas diferentes, y si ajusta la forma en que trata con todos para manejar las peculiaridades de una persona, se volverá menos efectivo.
Creo que esto es un poco duro para el gerente. En muchos casos, un comentario general como este podría ser constructivo, al menos como un primer paso. Puede haber un problema evidente que el revisor podría ver y corregir. Personalmente no lo haría así (no transmitiría una denuncia anónima como esta). Pero no creo que sea tan terrible, siempre y cuando el gerente haga un seguimiento de manera constructiva cuando el OP comunique que no ve un problema.

Revisé mis comentarios recientes y no encontré nada que me pareciera objetable, pero ese es el problema en pocas palabras, ¿no?

En este caso sí, pero no siempre. Idealmente, cuando revisó lo que había escrito, habría encontrado una o más cosas que , al reflexionar , se da cuenta de que podrían ser hirientes, aunque, por supuesto, nunca tuvo la intención de que fueran así cuando las escribió. Esa es la esperanza, al traer algo como esto a su atención sin detalles. Pero no ha funcionado de esa manera.

Asumiré que está obligado a respetar el anonimato de la queja y, por lo tanto, no se le puede decir qué comentario específico causó daño. De acuerdo, no todas las respuestas aquí están de acuerdo con ese anonimato, pero este no es un juicio penal, ni siquiera un procedimiento disciplinario interno, por lo que depende del empleador si lo otorga. Creo que lo mejor que puedes hacer es hacer lo que puedas y que se te vea haciendo lo que puedas dentro de ese sistema. No se limite a decir: "No mejoraré a menos que me diga quién me acusó", eso lo hace parecer poco dispuesto a mejorar, con una orden secundaria de sospecha de que tiene la intención de vengarse. También puede parecer desagradablemente irónico para su jefe, que su respuesta a una crítica de que sus críticas son involuntariamente hirientes, ¡es exigir tratar esa crítica como una acusación criminal!

Entonces, suponiendo que haga algo al respecto, creo que tiene dos buenas opciones aquí:

  1. Trate de no decir nada hiriente en el futuro. No lograrás la mejora que habrías logrado si tu atención se centrara en cosas específicas que haces que son dañinas pero que incluso al reflexionar no puedes ver que son dañinas. Pero puedes salir de tu camino para ser amable. Hay una gran posibilidad de que esto sea lo que espera tu jefe.

  2. Vuelva con su jefe y dígale: "Lo siento, todavía no puedo ver qué estoy haciendo mal aquí, así que realmente agradecería que me ayude a mejorar. ¿Hay alguien que pueda aconsejarme sobre la forma en que escribo?" en general , sin hacer referencia al comentario particular sobre el que se quejó? ¿Hay otros comentarios además del que se quejó, que pueda usar para ilustrar el problema que necesito abordar?". Por supuesto, también puede hacer esto sin tener que recurrir a su jefe, si tiene algún tipo de mentor, o simplemente elija a un colega que crea que podría ver lo que usted no puede ver.

También es posible que tu jefe básicamente no esté de acuerdo con la queja y no espere que hagas nada al respecto, sino que tiene que seguir los pasos. Lo cual es bastante desagradable para usted, ya que ahora tiene una queja en su contra y no tiene medios para remediarla o defenderse. Entonces, si él piensa que te está haciendo un favor al dejarlo pasar, entonces probablemente esté equivocado, y probablemente verá que todo el asunto se repite en el futuro.

Mucho mejor, si no está de acuerdo con la queja, sería que volviera con la persona que se quejó y le dijera: "Lamento que estés lastimado, pero esto está completamente en línea con los comentarios que esperamos recibir durante revisión del código, por lo que te ayudaremos a responderlas con el espíritu que se pretendía, que era criticar la línea de código en la página y no a ti. Este es el resultado de los estándares que hemos establecido para todo el equipo. /compañía, nada que aednichols en particular haya hecho mal, así que por favor no lo tomes en contra ni concluyas que por lo que escribió debe pensar mal de ti". Y nunca decirte que pasó algo.

Finalmente, no puedes realmente "arreglar las cosas" con la persona que resultó herida. Parece que han planteado el problema bajo condición de anonimato, por lo que no esperan una disculpa de su parte . Tal vez podría pedirle a su gerente que le haga una declaración en su nombre, que no tuvo la intención de lastimar y que lamenta haberlo causado. Dado que no sabe lo que hizo mal, esto será muy vago y, por lo tanto, probablemente no valga la pena hacerlo, pero su jefe o usted podrían pensar que ayudaría. Una declaración como esa, que su jefe garantiza que es sincera, podría incluso hacer que la otra parte abandone su anonimato para que todos puedan abordar el problema en su totalidad. Posibilidad remota, pero nunca sabes tu suerte.

Esta debería ser la mejor respuesta. Te mereces un +1 solo por " no esperan una disculpa tuya ", pero el resto de tu respuesta también es perfecta. Ojalá pudiera hacer +2 o +10 en esto.

¿A quien? Todo el mundo.

Una persona te hizo un favor y te hizo saber que tus comentarios eran, por alguna razón, ofensivos para ellos. Tiene la oportunidad de asegurarse de que sus comentarios sean constructivos y positivos para todos. No sabe quién más se ha sentido así también, o puede sentirse así en el futuro. Encuentra una voz y un estilo que funcionen para tantas personas como sea posible, incluyéndote a ti mismo.

¿Cómo sabes que fue un favor? ¿Qué pasa si la persona es ultra-super-hipersensible y el resultado correcto es que deje de tomar la revisión del código personalmente y crezca un poco? En la revisión de código, se encuentra totalmente en desacuerdo con su revisor y regresa con una explicación de lo que está sucediendo y por qué cree que la forma en que lo hizo es la mejor. Se supone que es una conversación entre dos profesionales que dejan de lado sus egos y solo resuelven problemas. Parte del profesionalismo es evitar volverse loco cuando a alguien le falta profesionalidad y ayudar a esa persona a mejorar.
@ErikE Exactamente. Y parte de la profesionalidad es no morder el anzuelo, y no dejarse tirar por alguien demasiado sensible, ¿El favor? Si la persona era increíblemente sensible, entonces esa persona te dio algo de práctica en el manejo de personas demasiado sensibles. Eso es un regalo. Puedes golpear ese lanzamiento, puedes golpear cualquier cosa. Mientras estemos haciendo conjeturas, esto también podría haber sido una queja totalmente legítima. Realmente no importa para el argumento.
Veo lo que estás diciendo. Las lecciones que más necesitamos son a menudo las que menos nos gustan. Supongo que lo que estaba pensando es que de los dos errores, no saber cómo atender perfectamente a la persona más sensible con los guantes de seda más diáfanos mientras puede recibir bien las críticas normales, o ser la persona más sensible que puede. No tolero el más mínimo conflicto o crítica, sé cuál prefiero tener en mi organización. Pero tal vez la realidad es que solo necesito mejorar en esos lanzamientos que mencionaste.
También pienso que puede haber un punto en el que la retroalimentación sea tan positiva para todos que ya no sirva como retroalimentación útil. La codificación de software tiene aspectos objetivos que hacen que sea imposible ceñirse siempre a "considere esto" y "he encontrado eso". A veces, solo tienes que decir "usa un Frob aquí", incluso si la persona reacciona de forma exagerada, porque esa es la forma correcta de resolver ese problema de codificación, y la forma en que se hizo definitivamente fue incorrecta .
Por último, la revisión del código que está provocando a una persona podría ser un regalo para ellos, correcto, para darles práctica en el manejo de críticas leves sin tomárselas tan personalmente, y para aprender a tolerar la incomodidad de una confrontación para resolver un problema. problema relacional. Pero agradezco tu comentario, ya que me dio más en qué pensar desde un lado diferente del problema.
@ErikE No es necesariamente uno u otro. Puedes mover a alguien y darle una crítica bastante mordaz si lo enmarcas adecuadamente. No quiero que el código incorrecto se fusione con el maestro, independientemente, eso no va a funcionar para nadie. La pregunta es cómo entregar esa retroalimentación necesaria sin apagar a la persona. Entonces todos ganan.

Su mejor opción en este caso es tomar la próxima revisión de código donde tenga algo que decir y escribir los comentarios y luego pasarlos a su jefe para ver si piensa que son ofensivos. Haga esto durante las próximas revisiones de código, hasta que deje de encontrar ofensivo lo que escribe, o hasta que le pida que deje de hacerlo. Si el jefe no quiere tomarse el tiempo, pregúntele a un colega de confianza. TENGA EN CUENTA que si está enojado por lo estúpido que se hizo algo, es probable que se manifieste en su tono, así que tenga especial cuidado con cualquier comentario que escriba mientras esté molesto o enojado.

Esto logra varias cosas. Primero que te estás tomando las críticas en serio y estás tratando de no ser ofensivo. Luego, le muestra al jefe que sus comentarios son, en general, justos (si lo son) o le da la oportunidad de mostrarle una mejor manera antes de ofender a otra persona. Si los comentarios necesitan ayuda y no ve lo que está mal, borrarlos antes de enviarlos es la mejor manera de averiguarlo y aprender a mejorarlos. Si la otra persona está malinterpretando, entonces el jefe puede decir que aprobó el texto del comentario si hay otra queja que detendrá a una serpiente que está tratando de hacerte quedar mal en su camino.

Eh, eso suena como una buena manera de molestar a un superior. No creo que sea una buena idea.
@DoritoStyle - Ese es un pequeño riesgo, sí. Pero es una buena sugerencia, especialmente si está precedida por otras sugerencias en respuestas anteriores (para pedir detalles al gerente), entonces es culpa del gerente. De hecho, OP debe sugerir este enfoque durante la reunión con el gerente. Si el gerente rechaza ambas sugerencias, deje de ser diligente en la revisión del código. En su cabeza, cualquier problema que no te molestes en atrapar.
"deja de ser diligente al hacer revisiones de código. En su cabeza, hay cualquier problema que no te molestes en detectar". Dudo que las cosas resultaran así en el mundo real. La culpa y el rastro de papel apuntarán directamente a OP solo y ellos soportarán la acción disciplinaria.
Eh, eso suena como una buena manera de molestar a un superior. @DoritoStyle La respuesta aborda eso con, si el jefe no quiere tomarse el tiempo, pregúntele a un colega de confianza. que es lo que su jefe probablemente sugerirá de todos modos. Si no quieren hacerlo, lo dirán. No creo que haya un gran riesgo en preguntar y tiene sentido ir a ellos primero ya que tienen una mejor idea de cuál es el problema.

Tómelo como una oportunidad para mejorar la forma en que comunica sus opiniones sobre el código de otros. En lugar de creer que esto tenía la intención de reprenderlo por su comportamiento, considérelo como una insinuación suave de que debe tener en cuenta no solo lo que dice en una revisión de código, sino también cómo lo dice en una revisión de código.

Sé por experiencia que las reseñas están destinadas a no tener ego y que no destacamos a ninguna persona en particular, pero siguen siendo personas las que escriben el código. Aproveche la oportunidad de preguntarle a su gerente o líder cómo habrían respondido al código que le pareció menos favorable, y simplemente mejore a partir de ahí. Conviértalo en un punto de enfoque para que pueda construir en lugar de que sea algo (potencialmente) negativo.

¿Eso no hará que sus comentarios sean menos efectivos para las personas que prefieren que las reseñas sean críticas? En mi experiencia, la mayoría de los programadores son así. Entonces, tal vez para mimar a una persona peculiar, ¿está sugiriendo tratar con todos los demás de una manera que puede ser menos efectiva? Algunas personas incluso responden mejor al sarcasmo.
@DavidSchwartz: Como desarrollador, puedo decir que hay formas de ser crítico con el trabajo de uno que no te hacen sonar como un idiota. Se necesita un buen y delicado acto de equilibrio, y lo importante es recordar que nosotros, como desarrolladores, todavía nos sentiremos mal por las críticas a nuestro trabajo. Por lo que obtuve del OP, se trataba de que sus críticas se interpretaran como un ataque al reseñado, por lo que las palabras que uno dice deben medirse cuidadosamente. Una cosa es denunciar un código incorrecto, pero otra muy distinta denunciar a una persona por su código incorrecto.
Estoy de acuerdo. Pero no es el caso que uno de esos enfoques sea siempre mejor que el otro. Diferentes personas responden mejor a diferentes enfoques. Obviamente, la persona que responde bien a "Eres un tonto inútil" es realmente rara. Pero algunas personas realmente responden mejor a los comentarios sarcásticos, encontrando que suavizan, mientras que otras personas los encuentran insultantes. Un impulso general en una dirección no necesariamente mejorará las cosas y puede empeorar sus relaciones con algunos desarrolladores.

tl;dr inicie una conversación en la oficina, si no sobre este incidente, entonces sobre la apertura y el espacio para la retroalimentación.

...

Suponiendo que está pidiendo ayuda, veo dos preguntas implícitas en la declaración Want to make things right, don't know with whom.

La primera pregunta es how do I make things right?, la segunda es who is giving me feedback?(aunque no usas la palabra 'yo')

Para la segunda pregunta, no necesita estar en Internet, más bien regrese a su oficina y comience a hacer las preguntas allí. ¿Cómo sabríamos :-), no trabajamos allí, verdad? Cosas como esta deben manejarse rápidamente, exponerse y aclararse, porque la retroalimentación siempre se trata del comportamiento (lo que haces), nunca de la persona, por lo que no debería ser un problema ser abierto y hablar sobre ello.

La primera pregunta puede implicar dos cosas: 1. aunque alguien hizo el esfuerzo de darte su opinión (suele ser más difícil dar tu opinión sobre las consecuencias negativas) no sientes que estás equivocado y quieres corregirlo explicándoselo a los demás. persona u otras personas (o aquí...) 2. sabes lo que hiciste mal, quieres encontrar una excusa apropiada y estás pidiendo consejos para una excusa apropiada (es decir, das pastel (aunque el pastel es una mentira) , dices lo siento, explicas tus intenciones cuando das comentarios sobre el código).

Si leo su descripción detallada de la pregunta, está construyendo un caso para la opción 1.

Pero para responder a su última pregunta what should I do?. Simplemente me haría cargo y explicaría que quieres aprender; participe en la conversación con la persona que le dio este comentario de otra persona, en lugar de tratar de investigar en silencio. Y que si esta persona tiene comentarios, no te hará daño a ti ni a nadie más, solo sé abierto. Colóquese, cuando participe en una conversación, como abierto a cualquier comentario, y la persona que no quería dirigirse a usted directamente probablemente se sentirá más segura.

Aparte de las otras respuestas, como la de Vietnhi Phuvan, que ya hace un buen trabajo al explicar qué hacer en general, esto es específico para una revisión de código:

  • asegúrese de revisar el código tal como está y no permita que el autor entre en escena por ningún motivo; puede criticar el código, pero no al autor.
  • si el código tiene un error, anótelo y luego continúe; no se demore una vez que se haya encontrado algo que está en contra del código, no se acumule si alguien más lo mencionó primero; el objetivo de una revisión de código es detectar errores, una vez hecho esto, continuar
  • no intente sugerir mejoras o correcciones durante la revisión del código; si cumple con los estándares, entonces está bien; mejorar los estándares de codificación en su organización porque olvidaron incluir algo no es algo que deba hacerse en una revisión del código

por último, ¿hubo un facilitador para mantener la revisión del código en marcha? Si es así, le pediría al gerente que hable con el facilitador y vea si puede brindarle algún consejo sobre cómo realizar una revisión del código en el futuro.

Mi respuesta es un poco una síntesis de las ideas presentes en algunas respuestas anteriores, y pido disculpas por el plagio. El punto principal de esta respuesta es un enfoque integral con una cascada de opciones .

Como prefacio, asumo que realiza muchas revisiones de código.

En primer lugar, solicite una reunión de seguimiento con el gerente, con la agenda mejorando las revisiones de código.

En esta reunión:

  1. Indica que te tomaste en serio los comentarios.

  2. Indique específicamente que revisó las revisiones de código anteriores y honestamente no pudo detectar un comentario problemático, y para mejorar le está pidiendo orientación.

  3. Revise la cascada de posibles solicitudes suyas y respuestas de él, como se describe a continuación.

  4. Si el gerente no es de ayuda significativa (rechaza la reunión, o dice que es su problema y no el suyo, se niega a proporcionar detalles o incluso un patrón, se niega a supervisar sus comentarios, etc.), entonces simplemente deje de ser un revisor de código diligente. .

    No vayas más allá. Haga el mínimo trabajo de revisión para asegurarse de que no lo acusen de tener una diligencia poco estricta.

    • Resalte los problemas que son errores obvios reales que se le pueden atribuir a usted como revisor. Eso es todo. Los errores son fáciles de comentar en términos completamente neutrales ( "este código hará X, Y y Z cosas malas en las condiciones A, B y C. Considere abordar esto. Gracias" )
    • No utilice las revisiones como herramientas de enseñanza, que es lo que hacen los buenos revisores.
    • No utilice las revisiones para mejorar el estilo, la arquitectura o para resolver problemas a largo plazo en el código.

Ahora, en cuanto a lo que puede querer preguntar en la reunión (después de indicar que ambos tomaron en serio sus comentarios Y se esforzaron por tratar de ver dónde se equivocaron):

  1. Pregunte si hay un patrón generalizado de un problema de comentarios hirientes de su parte, o si se trata de un problema aislado. Si hay un patrón, podría ser la retroalimentación de un individuo específico (pero muchos casos) o de muchas personas en el equipo.

    • Si hay un patrón generalizado de varias personas, su gerente debería poder, en el peor de los casos, señalarle el patrón con precisión o, en el mejor de los casos, señalar ejemplos específicos de malos comentarios sin preocuparse por señalar a un denunciante. Pregúntele por tales patrones/ejemplos.

    • Si hay un patrón pero solo 1 persona de muchas quejas, pregúntele al gerente si está de acuerdo con las quejas. Si lo hacen, nuevamente, debería poder al menos enunciar el patrón incluso si no nombra a la persona o muestra un comentario específico.

    • Si es único, pregunte qué fue tan atroz que un comentario de (presumiblemente) cientos calificó una escalada al gerente. Lo más probable es que haya problemas más profundos aquí en juego y esto sea solo una excusa. Puede ser que quieran sacarte del equipo/compañía y usar esto como una excusa. Puede ser que alguien tenga un problema en su contra y no pueda encontrar nada más para colgarse de usted. Puede ser que alguien reaccionó de forma exagerada, su gerente decidió ser complaciente para asegurarse una vida fácil para él a expensas suyas, y ahora no puede dar marcha atrás.

      Si se niega a ofrecer el patrón para los primeros 2 puntos, o si se niega incluso a responder si existe un patrón, vea el n.° 2 a continuación y trate al n.° 1 como "el gerente no cooperó", cuando decida si debe desconectarse. como revisor según los consejos de alto nivel anteriores.

  2. Sugiérale que, en el futuro, revise sus comentarios y marque los que parezcan "hirientes".

    Si está de acuerdo, usted CYAed cualquier problema (él asume la responsabilidad).

    Si no está de acuerdo, trate al #2 como "el gerente no cooperó", cuando decida si debe retirarse como revisor según el consejo de alto nivel anterior.

  3. Como alternativa al n.º 2, sugiera que otra persona que sea un revisor de código establecido y cuya opinión sea más cierta, revise sus comentarios y marque los que parezcan "hirientes".

    La ventaja es que, de nuevo, es CYA, y además, si hay otros revisores de código pero no puede señalar uno cuyos comentarios nunca son "hirientes", es una indicación de que no le gustas a alguien y no de tus comentarios reales. .

    La desventaja es que esto dañará tu ego y puede dañar un poco tu reputación.

    Si no está de acuerdo sin una buena razón, trate al n.° 2 como "el gerente no cooperó", cuando decida si debe retirarse como revisor según el consejo de alto nivel anterior.

  4. Pida una segunda opinión independiente. Indique que le gustaría que alguien en cuyo juicio tanto usted como su gerente confíen revise los comentarios, probablemente alguien de otro equipo.

    De esa manera, el gerente no tiene excusa de que no quiere una confrontación personal cuando se niega a mostrarle un comentario específico.


PD Habiendo dicho eso, hay formas obvias de hacer que los comentarios de revisión de código sean objetivamente menos personales y, por lo tanto, menos legítimamente dañinos.

Publiqué un ejemplo de un buen estilo de comentario para lograrlo al comienzo de la respuesta.

Estoy seguro de que hay TONELADAS de guías específicas sobre cómo hacer eso, pero sospecho que "cómo hago que mis comentarios sean menos personales e hirientes en el futuro" está fuera del alcance de lo que está preguntando, y para el caso, son más ontópicos en SDLC que en Workplace.SE

-1 Intencionalmente no rendir al máximo de la capacidad de uno en cualquier tarea, que no sea un sacrificio calculado para hacer algo mejor, siempre es una mala decisión, en cualquier industria. Languidecer en cualquier puesto, incluso en un mal ajuste con un liderazgo deficiente, siempre tendrá un impacto en el empleado, incluso si (poco probable) afecta más a la empresa/liderazgo. Si, de hecho, la situación descrita anteriormente (una vez) revelara una situación insostenible (personalmente no estoy de acuerdo), los esfuerzos diligentes pueden enfocarse en otra parte, y 'sacrificar' la diligencia por, digamos, una búsqueda diligente de trabajo, tutoría, voluntariado, sería ser mejores sugerencias.