1. C++ / Говнокод #6671

    +159

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    bool CSomeClass::Init( int argc, wchar_t* argv[] )

{

      bool isOk = true;

      bool goOn = true;

      for( int i = 0; isOk && goOn; ++i ) {

            switch( i ) {

                  case 0:

                        isOk = checkCommandLine( argc, argv );

                        break;

                  case 1:

                        isOk = checkFileExistence();

                        break;

                  case 2:

                        isOk = initFiles();

                        break;

                  default:

                        goOn = false;

            }

      }

      return isOk;

}

    Запостил: w100, 14 Мая 2011

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

    • ava Lure Of Chaos 14.05.2011 23:29 # −1

      > CSomeClass
      это же Си!
      Ответить

    • ava Dummy00001 15.05.2011 02:44 # +2

      один из способов писания FSM.

      ничего нового. не говно.
      Ответить

      • ava wvxvw 15.05.2011 08:15 # 0

        Так говнокод в том, что получилось очень много лишних слов. Вполне хватило бы 
        return checkCommandLine( argc, argv ) && checkFileExistence() && initFiles();

        Ну, или по крайней мере 2 переменные goOn и isOk легко можно было бы заменить одной (нам же изначально известно, сколько витков должен сделать цикл).
        Ответить

        • ava Dummy00001 15.05.2011 14:46 # +1

          и да и нет. в твоем коде нет никакой внешней сериализации в какая из функций в данный момент выполняется. не смотря на форму кода, самая важная переменная здесь: "i". в паре прог которые я знаю подобный код используется для перезапускаемой инициализации и деинициализации, но там "i" есть мембер класса.

          goOk & isOk, да, редундантны и больше похожи на нечто из корпоративного принудительного стайл гайд.
          Ответить

          • ava Aleskey 16.05.2011 11:53 # 0

            Да, то что i локальная - настораживает. Да и for( int i = 0; isOk && goOn; ++i )...
            Недоделанная корутина? Заготовка?
            Ответить

    • ava TarasB 15.05.2011 12:33 # −10

      показать все, что скрытоЗачем все строчки через одну пустые? Дурацкий язык, непродуманный.
      Ответить

    • ava guest 15.05.2011 14:53 # −1

      Въебал ТарасуБ 3 минуса.
      Ответить

    • ava Sauron 16.05.2011 21:59 # −3

      Говновенгерка намекаэ
      Ответить

    • ava guest 19.05.2011 17:45 # −2 

      bool CSomeClass::Init(int argc, wchar_t* argv[])
{
      bool isOk = true;
      bool goOn = true;

      for( int i = 0; isOk && goOn; ++i )  {
            switch( i ) {
                  case 0:
                        isOk = checkCommandLine( argc, argv );
                        isOk = isOk && checkFileExistence();
                        isOk = isOk && initFiles();
                        goOn = false;
               default:
                    break;
            }
      }

      return isOk;
}
      Ответить

    • ava guest8 08.04.2019 21:07 # −999

      показать все, что скрытоvanished
      Ответить

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