1. Си / Говнокод #13875

    +139

    1. 1
    for (normalize(); n > 0; scanf("%d", &temp, n--)){...}

    Но ведь работает!

    Запостил: Nex, 30 Сентября 2013

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

    • ava Bobik 30.09.2013 20:07 # 0

      А чем это лучше for (normalize(); n > 0; scanf("%d", &temp), n--){...} ?
      Ответить

      • ava byss 30.09.2013 20:13 # +5

        Наличием ворнинга.
        Ответить

      • ava bormand 30.09.2013 20:18 # 0

        Возможно вы имели в виду "а чем это лучше 
        normalize();
while (n > 0) {
    // ... то что было в скобках ...
    scanf("%d", &tmp);
    --n;
}
// или
for (; n > 0; --n) {
    // ... то что было в скобках ...
    scanf("%d", &tmp);
}
        "? :)
        Ответить

        • ava TarasB 30.09.2013 21:05 # +9

          фу, какой анскилльный питушиный код
          Ответить

        • ava Bobik 30.09.2013 21:14 # 0

          Это всё синтаксический сахар. А версия от автора будет падать, если scanf окажется stdcall, a main -- omit-frame-pointer.
          Ответить

          • ava bormand 30.09.2013 21:22 # 0

            > если scanf окажется stdcall
            Каким хреном? :) stdcall не умеет в переменное число аргументов...
            Ответить

            • ava Bobik 30.09.2013 21:27 # 0

              Блин, и правда, тупые компиляторы не умеют генерировать stdcall для функций с переменным числом аргументов.
              Но я готов поклясться, что где-то читал, что va_end() как раз нужен для того, чтобы stdcall-подобные соглашения о вызове могли правильно чистить стек для функций с переменным числом аргументов.
              Ответить

              • ava bormand 30.09.2013 21:39 # +2

                > stdcall для функций с переменным числом аргументов
                А как? Обязать функцию просканить все дополнительные параметры, чтобы va_end смог законфигурить разгрузку стека на возврате?

                На ящике с динамитом сидеть конечно можно, но не нужно ;)
                Ответить

                • ava Bobik 30.09.2013 21:39 # 0

                  >А как? Обязать функцию просканить все дополнительные параметры, чтобы va_end мог правильно размотать стек?
                  Именно так :)
                  Ответить

                  • ava bormand 30.09.2013 21:40 # 0

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

                    P.S. На самом деле можно придумать и такое соглашение, но это никому не нужно ;)
                    Ответить

                    • ava Bobik 30.09.2013 21:42 # 0

                      Это уже по-питушиному и анскильно.
                      [size=1]я всё правильно назвал?[/size]
                      Ну и где мой бб-код? :(
                      Ответить

                      • ava 1024-- 30.09.2013 23:36 # 0

                        > [size=1]я всё правильно назвал?[/size]
                        > Ну и где мой бб-код? :(
                        вот здесь: я всё правильно назвал?
                        Ответить

      • ava Qwertiy 30.09.2013 20:33 # +2

        Твой вариант лучше, т. к. в стек не помещаются лишние аргументы.
        А ещё, он понятнее.
        Ответить

      • ava spivti 01.10.2013 14:05 # 0

        продолжим извращаться :

        for (normalize(); n > 0; scanf("%d", &temp), n--){...}


        дальше:

        for (normalize(); n > 0;  n--){
scanf("%d", &temp);
}
        Ответить

        • ava spivti 01.10.2013 14:11 # 0

          normalize() можно вынести вперед, тогда получится, то что я уже видел выше.
          Ответить

        • ava bormand 01.10.2013 15:34 # +3

          n-- это анскилльная питушня, настоящий Сишник напишет --n, ибо оно работает быстрее.
          Ответить

          • ava spivti 01.10.2013 17:43 # 0

            мммм, я этого не знал (не настоящий сишник).
            Ответить

    • ava Bobik 30.09.2013 20:49 # +6

      Кстати, можно соптимизировать:
      for (normalize(); n > 0; scanf("%d", &temp, --n)){...}
      Ответить

      • ava 1024-- 30.09.2013 20:59 # +4

        Без посимвольного сравнения и не сразу узреешь всю соль Вашего комментария.
        Ответить

      • ava bormand 30.09.2013 21:25 # +8

        И убрать ворнинг
        for (normalize(); n > 0; scanf("%d", (--n, &temp))){...}
        Ответить

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