Избыточные конструкции кода

Возвращение без толку в конце:

   // stuff
   return;
}

Использование .tostring в строке

if (condition == true)
{
  ...
}

вместо

if (condition)
{
  ...
}

Редактировать:

или даже хуже и оборачиваясь условным тестом:

if (condition == false)
{
  ...
}

что легко читается как

if (condition) then ...

if (foo == true)
{
   do stuff
}

Я все время говорю разработчику, который это делает, что это должно быть

if ((foo == true) == true)
{
   do stuff
}

но он еще не понял намек.

void myfunction() {
  if (condition) {
    // Do some stuff
    if (othercond) {
      // Do more stuff
    }
  }
}

вместо

void myfunction() {
  if (!condition)
    return;

  // Do some stuff

  if (!othercond)
    return;

  // Do more stuff
}

Однажды у меня был парень, который неоднократно проделывал это:

bool a;
bool b;
...
if (a == true)
    b = true;
else
    b = false;

Помещение оператора exit в качестве первого оператора в функции, чтобы отключить выполнение этой функции, вместо одного из следующих вариантов:

• Полное удаление функции
• Комментирование тела функции
• Сохранение функции, но удаление всего кода

Использование exit в качестве первого утверждения делает его очень трудным для обнаружения, вы можете легко прочитать его.

Объявление отдельно от присваивания на языках, отличных от C:

int foo;  
foo = GetFoo();

Использование комментариев вместо системы управления версиями:
-Комментарий или переименование функций вместо их удаления и доверия к исходному контролю может вернуть их вам, если это необходимо. -Добавление комментариев типа «RWF Change» вместо того, чтобы просто вносить изменения и позволять системе управления версиями возложить вину.

Где-то я заметил это, и я считаю, что это вершина логической избыточности:

return (test == 1)? ((test == 0) ? 0 : 1) : ((test == 0) ? 0 : 1);

:-)

Я часто сталкиваюсь со следующим:

function foo() {
    if ( something ) {
        return;
    } else {
        do_something();
    }
}

Но это не помогает им сказать, что все остальное здесь бесполезно. Это должно быть либо

function foo() {
    if ( something ) {
        return;
    }
    do_something();
}

или - в зависимости от продолжительности проверок, которые выполняются перед do_something ():

function foo() {
    if ( !something ) {
        do_something();
    }
}

Использование массива, когда вы хотите установить поведение. Вам нужно проверить все, чтобы убедиться, что его нет в массиве, прежде чем вставлять его, что делает ваш код длиннее и медленнее.

Избыточный код сам по себе не является ошибкой. Но если вы действительно пытаетесь спасти каждого персонажа

return (condition);

тоже является избыточным. Ты можешь написать:

return condition;

Боязнь нуля (это тоже может привести к серьезным проблемам):

if (name != null)
  person.Name = name;

Избыточные if (без использования else):

if (!IsPostback)
{
   // do something
}
if (IsPostback)
{
   // do something else
}

Избыточные проверки (Split никогда не возвращает null):

string[] words = sentence.Split(' ');
if (words != null)

Подробнее о проверках (вторая проверка избыточна, если вы собираетесь зацикливаться)

if (myArray != null && myArray.Length > 0)
  foreach (string s in myArray)

И мой любимый вариант для ASP.NET: разбросанные по всему коду DataBind для отрисовки страницы.

Скопируйте избыточность вставки:

if (x > 0)
{
   // a lot of code to calculate z
   y = x + z;
}
else
{
   // a lot of code to calculate z
   y = x - z;
}

вместо

if (x > 0)
  y = x + CalcZ(x);
else
  y = x - CalcZ(x);

или даже лучше (или более запутанный)

y = x + (x > 0 ? 1 : -1) * CalcZ(x)

Размещение элементов в куче, а не в стеке.

{
    char buff = malloc(1024);
    /* ... */
    free(buff);
}

вместо

{
    char buff[1024];
    /* ... */
}

или же

{    
    struct foo *x = (struct foo *)malloc(sizeof(struct foo));
    x->a = ...;
    bar(x);
    free(x);
}

вместо

{
    struct foo x;
    x.a = ...;
    bar(&x);
}

Из кошмарных обзоров кода .....

char s[100];

с последующим

memset(s,0,100);

с последующим

s[strlen(s)] = 0;

с множеством противных

if (strcmp(s, "1") == 0)

замусорили про код.

Избыточные вызовы .ToString ():

const int foo = 5;
Console.WriteLine("Number of Items: " + foo.ToString());

Ненужное форматирование строки:

const int foo = 5;
Console.WriteLine("Number of Items: {0}", foo);

Самая распространенная конструкция избыточного кода, которую я вижу, - это код, который никогда не вызывается из любого места программы.

Другой - шаблоны проектирования, которые используются там, где нет смысла их использовать. Например, писать везде «new BobFactory (). CreateBob ()» вместо того, чтобы просто писать «new Bob ()».

Удаление неиспользуемого и ненужного кода может значительно улучшить качество системы и способность команды поддерживать ее. Преимущества часто поражают команды, которые никогда не думали об удалении ненужного кода из своей системы. Однажды я провел обзор кода, сидя с командой и удалив более половины кода в их проекте, не изменяя функциональность их системы. Я думал, что они обидятся, но после этого они часто просили меня дать совет по дизайну и отзывы.