code-review

Cuando alguien escribe "nit: removed whitespace" en una confirmación, ¿qué significa "nit"? También lo he visto en mayúscula ... :". Este significa que solo es "quisquilloso"you no tienes que arreglar estos puntos, pero nos gustaría que lo hicieras.

Estoy trabajando en un proyecto donde encuentro que estoy comprobando lo siguiente en muchos, muchos lugares: if(item.Rate = ... Parse(obj.ToString(), out parsedInt); return IsNull(obj) || (parsed && parsedInt == 0); } Hay una manera mejor?

He estado haciendo revisión de código (principalmente utilizando herramientas como FindBugs) de uno de nuestros proyectos fav ... Sets al cerrar Conexiones (usamos Oracle db, de ahí mis correcciones). Java.SQL.api no dice nada en relación.close() javadoc.

Utilicé IntelliJ para "Inspeccionar código", y uno de sus resultados es: La sinopsis del problema puede ser local del paquet ... catch( IOException e ) { LOG.error( "Error while writing heart beat log", e ); } } }

Estoy pensando en usar PDO en toda mi futura webapp. Actualmente (usando lo que he aprendido de hasta ahora), lo que tengo en ... na desventaja o cosas que se pueden mejorar allí. Lo siento si eso parece poco claro o demasiado largo. Gracias de antemano.

Como una buena práctica de git, uno debe confirmar con frecuencia, pero para revisar el código es posible que deba revisar un ... e en varias confirmaciones a la vez. ¿Hay alguna forma de revisar varias confirmaciones y fusionarlas o rechazarlas a la vez?

Estoy haciendo una revisión de código, y he encontrado mucho código con el siguiente formato: public MyResponse MyMethod(str ... Sistema de Llamadas.Imposible.Deseche el objeto 'MyResponse' antes de que todas las referencias a él estén fuera de alcance.

Tengo una pregunta para ustedes sobre la escritura de código en C# : Tuve un largo debate con un colega mío.: Dice que al de ... if(... condition ... ) { ... } } Alguna idea sobre el tema? Gracias de antemano.