Estou debatendo com meu colega sobre quando executar a revisão de código - antes ou depois dos testes de unidade. Qual é a melhor prática? Alguns fatores que precisamos levar em consideração (pode haver mais): Tamanho da alteração do código - uma grande alteração significa que mais alterações...
Eu sei que há um grande impulso para revisões de código no desenvolvimento comercial. No entanto, as revisões de código são usadas no software de código aberto ou são baseadas na confiança? Se sim, então como eles são realizados? [É um commit atrasado, "um ambiente de pré-commit", existe uma...
Fechado . Esta questão é baseada em opiniões . No momento, não está aceitando respostas. Deseja melhorar esta pergunta? Atualize a pergunta para que ela possa ser respondida com fatos e citações editando esta postagem . Fechado há 4 anos . Como hipótese,...
Atualmente, estou trabalhando na introdução do React na empresa em que trabalho. Como o único que realizou cursos sobre o assunto, quem posso solicitar uma revisão para garantir que o código esteja seguindo as "melhores práticas" e seja implementado
Eu herdei um código terrível que incluí uma pequena amostra abaixo. Existe um nome para esse anti-padrão específico? Quais são algumas recomendações para refatorar isso? // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User...
No momento, estou revisando alguns dos códigos de desenvolvedores juniores que acabaram de ingressar na minha equipe. Estou pensando em como devo entregar o resultado dessa revisão: Devo corrigir o código sozinho? Devo dar a eles feedback sobre o processo de revisão e deixá-los fazer as correções...
Hoje no trabalho, um dos meus colegas revisou meu código e sugeriu que eu removesse uma propriedade somente de conjunto e usasse um método. Como estávamos ocupados com outras coisas, ele me disse para examinar a Property Designseção do livro "Framework Design Guidelines". No livro, o escritor...
Me pediram para criar diretrizes para revisão de CSS, JS e HTML. Eu sei que existem diretrizes de codificação para JS, mas não sei nada sobre HTML e CSS. Para revisar o JS, certamente seguirei essas diretrizes e as mencionarei. Mas e CSS e HTML? Além de erros lógicos e problemas de indentação, há...
Quais são os problemas de design ao retornar uma hashtable de um método público quando você deseja retornar vários itens em vez de criar uma classe e retornar um objeto disso? Se houver problemas, em que circunstâncias faz sentido fazê-lo? Como a resposta a essa pergunta muda dependendo se o...
Eu só queria saber qual é a diferença entre análise de código estática e revisão de código. Como são feitos cada um desses dois? Mais especificamente, quais são as ferramentas disponíveis hoje para revisão de código / análise estática do PHP? Também gostaria de saber sobre boas ferramentas para...
Qual é o melhor processo para revisão de código ao usar o GIT? Temos um provedor GIT externo (Unfuddle) e limites para o uso de recursos - portanto, não podemos ter repositórios remotos dedicados para todos os desenvolvedores. Processo atual: Temos um servidor GIT com uma masterfilial na qual...
Fechadas. Esta questão está fora de tópico . No momento, não está aceitando respostas. Deseja melhorar esta pergunta? Atualize a pergunta para que ela esteja no tópico do Software Engineering Stack Exchange. Fechado há 5 anos . Ao inspecionar manualmente...
Há cerca de três meses, nosso grupo de engenharia lançou o Conselho de Revisão para ser usado em todas as revisões de código por pares. Hoje, tive uma discussão com uma das pessoas envolvidas nesse processo e descobri que já estamos procurando uma substituição (possivelmente algo comercial) por...
Existe uma ferramenta que usa como entrada um patch proposto e um repositório git e identifica os desenvolvedores como os melhores candidatos para revisar o patch? Usaria o histórico do git para identificar os autores que têm mais experiência com os arquivos / seções de código que estão sendo...
Agora estou aprendendo o Gerrit (que é a primeira ferramenta de revisão de código que eu uso). A Gerrit requer uma alteração revisada para consistir em uma única confirmação. Meu ramo de recursos tem cerca de 10 confirmações. A maneira preferida pelo gerrit é esmagar esses 10 commits em um único....
Na minha experiência, a maioria das equipes da empresa revisa o código do membro da equipe com uma pequena quantidade de código, sempre com menos de centenas de linhas. É apropriado revisar grande quantidade de código, por exemplo, um módulo, quando o código estiver completo e pronto para ser...
Code Review.SE é excelente quando você precisa de feedback sobre um pedaço preciso e curto do código. Mas onde obter feedback semelhante sobre o próprio código quando: você tem milhares de LOC, não tenha colegas em seu local de trabalho prontos ou dispostos a revisar o código¹, não tem milhares...
Minha equipe raramente faz uma revisão de código, principalmente porque não temos tempo suficiente e as pessoas não têm energia e vontade de fazê-lo. Mas eu realmente gostaria de saber o que as pessoas pensam sobre o meu código quando o lêem. Dessa forma, entendo melhor como as outras pessoas...
Trabalho com muitos códigos Java e RPG herdados em um aplicativo interno da empresa. Como você pode esperar, muito do código é escrito em muitos estilos diferentes, e geralmente é difícil de ler devido a variáveis mal nomeadas, formatação inconsistente e comentários contraditórios (se...