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

    +146

    1. 1
    for (int i=0, N=sizeof(a)/sizeof(int); ++i<N-1; s+=(int)(a[i]>a[i-1]&&a[i]>a[i+1]));

    Это говнокод или годный троллинг? Топик http://forum.codenet.ru/threads/69046-Помогите-задача-С/

    Запостил: mrxak, 22 Ноября 2011

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

    • ava Xom94ok 22.11.2011 22:37 # 0

      1. Где здесь C++?
      2. N=sizeof(a)/sizeof(int) - так не надо делать.
      3. Границы диапазона?
      Ответить

      • ava absolut 22.11.2011 22:54 # 0

        1. объявление переменной в цикле
        2. почему?
        3. что границы?
        Ответить

        • ava mrxak 22.11.2011 23:00 # 0

          ну кстати с 2 я согласен.. 
          template<typename T, size_t N>
inline size_t LengthOf(T (&x)[N])
{
  return N;
}

          было бы правильнее.. только тогда это уже не однострочник будет
          Ответить

        • ava Xom94ok 22.11.2011 23:00 # 0

          1. Да ну?
          2. Лучше sizeof(a)/sizeof(a[0]). А ещё лучше - vector.
          3. a[0]>a[1]?
          Ответить

          • ava mrxak 22.11.2011 23:08 # 0

            простите, но на какой итерации вы увидели a[0] > a[1]??
            Ответить

            • ava Xom94ok 22.11.2011 23:11 # +1

              В том-то и дело, что не увидел, а такой случай возможен.
              Ответить

              • ava mrxak 22.11.2011 23:25 # 0

                Вообще это зависит от трактовки формулировки. По мне формулировка топикстартера на коденете "найти количество элементов, которые больше своих соседей." означает, что соседей больше одного. Иначе это должно обговариваться отдельно, равно как и последний элемент тогда.
                Ответить

          • ava absolut 22.11.2011 23:40 # +1

            1. в Си все переменные объявляются в начале блока
            2. о каком векторе речь, если вы здесь С++ не увидели? про размер первого эл-та согласен.
            Ответить

    • ava mrxak 22.11.2011 22:53 # 0

      пардон, с плюсами погорячился, обычный си..
      а что с границами? i-1 за границы не вылезает (так как ++i в условии, а for реализуется в цикл с предусловием), i+1 тоже не вылезает, так как <N-1.. и код компилится и работает, кстати))
      Ответить

    • ava 3.14159265 22.11.2011 23:00 # 0

      gamedev=>codenet mrxakGonvo?
      Ответить

    • ava defecate-plusplus 22.11.2011 23:33 # 0

      еще, конечно, автора спасло, что i, N оказались типа int, а не, как положено, size_t
      но как было сказано выше, случай
      int a[] = {1, 0, 0, 0, 0, 1};
      не обрабатывается никак
      Ответить

    • ava guest8 09.04.2019 12:13 # −999

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

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