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

    +32

    1. 1
    Sleep("pause");

    Написал как-то ночью. Без комментариев.

    Запостил: qwertyuio, 13 Октября 2012

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

    • ava bormand 14.10.2012 06:52 # 0

      Долго будет спать... хотя, конечно, зависит от адреса, который будет дан сегменту кода.
      Ответить

      • ava LispGovno 14.10.2012 15:11 # +2

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

      • ava govnomonad 14.10.2012 16:15 # +2

        Дык оно же вообще не скомпилится. Или вижуал студия уже научилась приводить const char* к unsigned int?
        Ответить

        • ava govnomonad 14.10.2012 16:19 # +3

          Я буду всегда читать комментарии до конца
          Я буду всегда читать комментарии до конца
          Я буду всегда читать комментарии до конца
          Я буду всегда читать комментарии до конца
          Я буду всегда читать комментарии до конца
          Я буду всегда читать комментарии до конца
          Я буду всегда читать комментарии до конца
          Я буду всегда читать комментарии до конца
          Ответить

    • ava TarasB 14.10.2012 10:00 # −5

      показать все, что скрытокрестоблядство...
      Ответить

      • ava bormand 14.10.2012 10:45 # +6

        Виндоапиблядство, или где-то еще есть Sleep() с заглавной буквы?
        Ответить

        • ava TarasB 14.10.2012 13:31 # −4

          Этот код написан на С++, это кривые костыли С++ виноваты в том, что этот код такой.
          Ответить

          • ava absolut 14.10.2012 14:43 # +1

            Этот код запостили как код на С++, остальное недоказуемо.
            Ответить

            • ava LispGovno 14.10.2012 15:07 # +1

              это код на C
              импортировали windows.lib и не подключали windows.h
              и импортировали функцию Sleep:
              extern Sleep();
              В результате Sleep принимает произвольные параметры и типы.
              В крестах это не скомпилируется и не выполнится успешно.
              Ответить

          • ava bormand 14.10.2012 15:47 # +4

            На самом деле С++ это не скомпилит. А вот сишка - с ворнингом, но соберет. Поэтому данная претензия к крестоблядскому крестокомпилятору не принимается.

            @LispGovno
            > импортировали windows.lib и не подключали windows.h
            > и импортировали функцию Sleep:
            > extern Sleep();
            Да сишка и так соберет. Можно даже не париться с забыванием прототипа.

            P.S. Где здесь с++, qwertyuio?
            Ответить

            • ava absolut 14.10.2012 15:50 # +1

              Почему не соберет? Вы сигнатуру функции видели? Может это и не WinAPI вовсе :)
              Ответить

              • ava bormand 14.10.2012 15:52 # +4

                > Вы сигнатуру функции видели?
                Ну разве что там int Sleep(const char*)... Но это еще большее извращение чем приведенный код ;)
                void Sleep(const char *time) {
    if (strcmp(time, "pause") == 0) {
        printf("Press any key to continue...\n");
        getchar();
    } else {
         sleep(atoi(time));
    }
}
                Ответить

                • ava LispGovno 14.10.2012 17:34 # 0

                  Не поддерживает локализацию и не кросплатформенно.
                  Ответить

                  • ava bormand 14.10.2012 19:36 # +1

                    С этими пунктами согласен (плюсанул), но не суть. Локализацию добавить не так уж сложно (да хоть тот же gettext), а sleep можно выбрать ифдефами в зависимости от платформы, или воспользоваться sleep'ом из более-менее высокоуровневых либ.

                    Еще недостаток - надо бы приводить time к нижнему регистру.
                    Ответить

                    • ava LispGovno 14.10.2012 19:44 # −1

                      Со всеми пунктами согласен. Но с моей стороны было глупо придираться к говнокоду. Себя минусанул.

                      >gettext
                      Вот это круто... Я не в теме. POSIX? В винде нет?

                      >а sleep можно выбрать ифдефами
                      C++11: std::this_thread::sleep_for
                      Ответить

                      • ava roman-kashitsyn 14.10.2012 22:22 # 0

                        >> gettext
                        > Вот это круто... Я не в теме. POSIX? В винде нет?

                        К POSIX это не относится. Под винду тоже вроде вариант есть. Правда, схема локализации выглядит довольно сложно по сравнению, скажем, с java api.
                        Ответить

    • ava absolut 14.10.2012 15:59 # +2

      Вообще, речь наверное должна идти о вызове system( "pause" );
      Ответить

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