-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Alex
authored and
Alex
committed
Jun 22, 2024
1 parent
438a051
commit c9afa6a
Showing
2 changed files
with
36 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
--- | ||
layout: single | ||
title: "\"No lo podías haber hecho peor\"" | ||
date: 2024-06-22 09:08:53 +0200 | ||
categories: development | ||
comments: true | ||
lang: en | ||
tags: kanban, agile | ||
image: images/badcodereview.jpg | ||
--- | ||
|
||
{:refdef: style="text-align: center;"} | ||
![dist files]({{ site.baseurl }}/images/badcodereview.jpg) | ||
{: refdef} | ||
|
||
{:refdef: style="text-align: center;font-size:9px"} | ||
Foto de <a href="https://unsplash.com/es/@alvarordesign?utm_content=creditCopyText&utm_medium=referral&utm_source=unsplash">Alvaro Reyes</a> en <a href="https://unsplash.com/es/fotos/dos-hombres-usando-computadora-y-computadora-portatil-fSWOVc3e06w?utm_content=creditCopyText&utm_medium=referral&utm_source=unsplash">Unsplash</a> | ||
{: refdef} | ||
|
||
Recuerdo aquella tarde en la oficina. Era viernes por la tarde y era cuando nos reuniamos todo el equipo para hacer un code review. Por aquel entonces ibamos a la oficina y las dinámicas eran diferentes (que os tengo que contar). | ||
|
||
Pues bien, la manera en que hacíamos el code review era la siguiente, alguien del equipo proponia una clase suya al resto del equipo con la suficiente antelación para que el resto del equipo se la pudiese leer con detenimiento. | ||
|
||
Lo que hacia la gente era imprimirla y escribir si habia algo en la clase de código que merecía ser destacado para hacer alguna propuesta de cambio o mejora. Por ejemplo "en la linea 25 has escrito una variable con un nombre que no se entiende demasiado". Este estilo de comentarios. | ||
|
||
Al llegar el viernes nos reuniamos todos en una sala con la clase de código impresa en papel, preparados para comentarla. Había un moderador (ahora le llamaríamos facilitator para hacerlo más guay) que daba la palabra y ponia límite en las discusiones si estas duraban más de la cuenta. Pues ese día tocaba hacer code review de una clase que habia desarrollado yo. | ||
|
||
Yo en aquel momento tenia unos 3 años de experiencia y la gente que habia en esa sala tenía 10 años de experiencia más que yo. Podemos decir que tenia experiencia pero comparado con mis compañeros era bastante Junior. Pues bien al empezar la sesión un compañero mio me dijo con un tono bastante ironico y sarcástico "No lo podrías haber hecho peor". | ||
|
||
Me sentó en ese momento como una patada porque no estaba acostumbrado a recibir críticas de mi trabajo y pensaba que era algo hacia mi. La clase era muy mejorable desde el punto de vista de la legibilidad, desde el punto de vista de la responsabilidad única y de unas cuantas cosas más xD | ||
|
||
Luego con el tiempo entendi el valor que tenia esa critica para mejorar mi manera de desarrollar. Siempre que veo una clase de código la miro con ojos de en que se podria mejorar esto, se podría hacer más legible esta clase de alguna manera? Sin duda esas sesiones me sirvieron para mejorar y para hacer mejor mi trabajo. | ||
|
||
De manera que si teneis alguien que os de una crítica constructiva sobre vuestro trabajo aceptarla y ver de que manera la podeis utilizar para hacer mejor vuestro trabajo. Aquí estamos hablando de código pero esto puede aplicar a cualquier ambito. | ||
|
||
No me puedo responsabilizar de que tu código sea Spagetti code pero sí de que recibas en tu buzon un correo de que he escrito un nuevo post con el que quiza aprendas algo ;) Escribo cuando me apetece así que no esperes que te envíe cada dia un correo. Hasta la próxima. |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.