Кресты / Говнокод #2680 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
bool userNameChanged = true;
bool passwordChanged = true;

if(userNameChanged)
{
	// какие-то операции
}

if(passwordChanged)

{
	// ещё какие-то операции
}

Вот ещё один код от нашего шефа-индуса...

Запостил: Highlander Highlander, (Updated )

Комментарии (10) RSS

    • что чо?? зачем тут эти переменные и два if написано?
      Ответить
      • Может подразумевалость что подчиненые-индусы добавят логику обращающую переменные в false?
        Ответить
        • ну если бы в строках
          // какие-то операции
          // ещё какие-то операции
          стояло бы
          passwordChanged = false, то так бы и было написано
          Ответить
  • Получается при объявлении переменных, сразу предполагается, что имя и пароль изменены. И оба блока if отработают, т.к. никаких операций до проверки условия не производилось! Вот если бы было как-то так:
    bool userNameChanged = true;
    bool passwordChanged = true;
    
    someFunc (username); 
    someFunc (userpasswd);
    
    if(userNameChanged)
    {
    	// какие-то операции
    }
    
    if(passwordChanged)
    
    {
    	// ещё какие-то операции
    }
    Ответить
    • Если бы да кабы.
      Поскольку переменные не объявлены как static и не лежат в анонимном неймспейсе, изменить их значение можно из любой единицы трансляции. Так что не обязательно городить огород с магическими вызовами функций.
      Ответить

Добавить комментарий

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?


    8