graywolf's lair

Inhuman being's diary…

Code WTF #3

| 7 Comments

Сьогодні ще цікавого надибав:

bool test = (mail.Find('@') == -1);
if(tsInspireDate.GetDays()<15 && !active && (mail.Find('@') == -1))
{ /* ...skipped... */ }

І ніби ж не індуси…

P.S. Хоча, звісно, всяке буває… Я і у себе в коді, що часто перероблювався, якось перли знаходив (коли один дрібний фрагмент змінюєш, а попередній шматок коду, що його дублює забуваєш видалити).

7 Comments

  1. в чому прикол? інгріш?

  2. ні. змінна test ініціалізується булевим значенням, яке вказує на ненаявність символу “@” в рядку, але в порівнянні підставлено той самий код, що і для ініціалізації, а сама змінна test ні тут, ні надалі по коду взагалі не використовується.

  3. ну то й що – код же виконується? а “загублені” змінні – це звичайна річ, з них же воду не пити…

  4. Виконується. Просто загублені – так, трапляютсья. Але одна під іншою – це трохи дивно виглядає.

  5. Ех.. Не все що ти не розумієш, то фігня.
    Змінна test – явно для дебагу. Незрозуміло трохи чому автор не вніс її в if.
    Якщо це розумна мова, то 1) буде варнінг.
    2) компілятор відкине цю змінну при реліз зборці.

Залишити відповідь

Required fields are marked *.