Perguntas com a marcação «code-smell»

O C # permite o uso de #region/ #endregionkeywords para tornar áreas de código recolhíveis no editor. Sempre que faço isso, faço isso para ocultar grandes pedaços de código que provavelmente poderiam ser refatorados para outras classes ou métodos. Por exemplo, vi métodos que contêm 500 linhas de...

Algo que eu conheço há algum tempo, mas nunca considerei, é que, na maioria dos idiomas, é possível dar prioridade aos operadores em uma instrução if com base em seu pedido. Costumo usar isso como uma maneira de evitar exceções de referência nula, por exemplo: if (smartphone != null &&...

Você conhece esse sentimento quando só precisa mostrar esse novo truque com Expressions ou generalizar três procedimentos diferentes? Isso não precisa estar na escala do Architecture Astronaut e, de fato, pode ser útil, mas não posso deixar de notar que outra pessoa implementaria a mesma classe ou...

Esta pergunta sobre o SO fala sobre a correção do que o OP pensava ser um código de inveja de recursos . Outro exemplo em que vi essa frase bacana sendo citada está em uma resposta recentemente dada aqui em programmers.SE. Embora eu tenha deixado um comentário a essa resposta solicitando as...

Normalmente, eu apenas faço meus testes de unidade juntos, usando copiar e colar e todo tipo de outras práticas ruins. Os testes de unidade geralmente acabam ficando feios, cheios de "cheiro de código", mas isso realmente importa? Eu sempre digo a mim mesma, contanto que o código "real" seja "bom",...

As variáveis ​​de flag são más? Os seguintes tipos de variáveis ​​são profundamente imorais e é mau usá-las? "variáveis ​​booleanas ou inteiras às quais você atribui um valor em determinados locais e, abaixo, você faz check-in ou então para fazer algo ou não, como, por exemplo, usando newItem =...

No código legado, ocasionalmente vejo classes que não passam de invólucros para dados. algo como: class Bottle { int height; int diameter; Cap capType; getters/setters, maybe a constructor } Meu entendimento do OO é que as classes são estruturas para dados e os métodos de operação nesses...

Existem muitos recursos na web referentes e listando odores de código. No entanto, nunca vi informações sobre cheiros arquitetônicos . Isso está definido em algum lugar e existe uma lista disponível? Alguma pesquisa formal foi feita sobre defeitos de arquitetura e seu impacto na velocidade do...

Estou encontrando muitos arquivos de linha de 2-3k, e não parece que eles devam ser tão grandes. Qual é um bom critério para chamar objetivamente um arquivo de código-fonte "muito grande"?, Existe uma quantidade máxima de linhas que um arquivo de código-fonte deve

Ao trabalhar com python pela primeira vez, descobri que acabei escrevendo várias classes no mesmo arquivo, o que se opõe a outras linguagens como Java, que usa um arquivo por classe. Geralmente, essas classes são compostas de uma classe base abstrata, com 1-2 implementações concretas cujo uso...

Eu e um grupo de amigos estamos trabalhando em um projeto há algum tempo, e queríamos inventar uma maneira agradável de OOP de representar um cenário específico para o nosso produto. Basicamente, estamos trabalhando em um jogo de balas no estilo Touhou , e queríamos criar um sistema em que...

É aconselhável criar várias classes em um arquivo .cs ou cada arquivo .cs deve ter uma classe individual? Por exemplo: public class Items { public class Animal { } public class Person { } public class Object { } } Esquivando-se por um minuto que este é um exemplo ruim de boa...

Isso é algo em que estou pensando desde que li esta resposta no controverso tópico de opiniões sobre programação : Seu trabalho é sair do trabalho. Quando você está escrevendo um software para o seu empregador, qualquer software que você criar deve ser escrito de tal maneira que possa ser...

Aqui estamos em 2010, engenheiros de software com 4 ou 5 anos ou experiência, ainda projetando tabelas com 96 colunas de fraturamento. Eu disse a ele que seria um pesadelo. Mostrei a ele que precisamos usar ordinais para fazer a interface do MySQL com C #. Expliquei que as tabelas com mais...

Existe algum propósito para declarar um init()método para um tipo? Não estou perguntando se devemos preferir init()a um construtor ou como evitar a declaraçãoinit() . Estou perguntando se existe alguma justificativa por trás da declaração de um init()método (ver quão comum é) ou se é um cheiro de...

Eu realmente me apaixonei por testes de unidade e TDD - estou infectado por testes. No entanto, o teste de unidade é normalmente usado para métodos públicos. Às vezes, também tenho que testar algumas suposições em métodos particulares, porque algumas são "perigosas" e a refatoração não pode...

Um bom desenvolvedor com quem trabalho me falou recentemente sobre algumas dificuldades que ele teve em implementar um recurso em algum código que herdamos; ele disse que o problema era que o código era difícil de seguir. A partir disso, examinei mais profundamente o produto e percebi o quão...

Muitas vezes, encontro perguntas online, e muitas soluções incluem dicionários. No entanto, sempre que tento implementá-los, recebo esse cheiro horrível no meu código. Por exemplo, toda vez que eu quero usar um valor: int x; if (dict.TryGetValue("key", out x)) { DoSomethingWith(x); } São 4...

Este artigo afirma que uma classe de dados é um "cheiro de código". O motivo: É normal quando uma classe recém-criada contém apenas alguns campos públicos (e talvez até um punhado de getters / setters). Mas o verdadeiro poder dos objetos é que eles podem conter tipos de comportamento ou...

Estou lendo sobre cheiros de códigos comuns no livro de Refatoração de Martin Fowler . Nesse contexto, eu estava pensando em um padrão que estou vendo em uma base de código, e se alguém poderia objetivamente considerá-lo um anti-padrão. O padrão é aquele em que um objeto é passado como argumento...