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

    +127

    1. 1
    (&goodsBuff[k].name==&vmass[p].name)

    Сравнение строк

    Запостил: GrayCardinal, 25 Октября 2010

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

    • ava absolut 25.10.2010 11:47 # +1

      Обычная ошибка начинающего
      Ответить

    • ava bugmenot 25.10.2010 15:12 # 0

      здесь более говниста строка mass в идентификаторе (который массив - К.О.)
      Ответить

    • ava andremacareno 25.10.2010 21:26 # 0

      Петросяны :)
      VM - Ass!
      По говнокоду: смеяться надо в условии, в том, что нету strcmp?
      Ответить

    • ava Анонимус 25.10.2010 22:59 # +2

      так трогательно, когда школьники называют переменные "massiv", "schetchik" итд.
      я сам так делал в 98м
      Ответить

    • ava TheCalligrapher 26.10.2010 04:04 # −2

      До очевидного варианта - `vmass` обозначает "vertical mass" (вертикальная масса) - почему-то никто до сих пор не додумался. Любой мало-мальски образованный в области физики человек видя идентификатор `vmass` сразу думает о вертикальной массе :)
      Ответить

      • ava bugmenot 26.10.2010 11:04 # 0

        массу-то там видно, но масса - скаляр, вы случайно с весом не путаете?
        Ответить

      • ava scalar4eblo4no 26.10.2010 16:03 # −1

        Скажите, мало-мальски образованный в области физики человек, это где же это вы видели такой абсурдный термин "вертикальная масса" ?=)
        Ответить

        • ava absolut 26.10.2010 17:20 # +1

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

          • ava scalar4eblo4no 26.10.2010 17:29 # 0

            Ок, на говнокоде нужен смайлик с табличкой "Sarcasm".

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

            • ava absolut 26.10.2010 17:37 # 0

              При этом в 90% случаем из-за того что не удалось отловить интонацию предыдущего оратора.
              Ответить

              • ava nil 30.10.2010 01:33 # 0

                http://govnokod.ru/4456#comment50157
                Эти числа неспроста! Математика — великая вещь!
                Ответить

    • ava TheCalligrapher 26.10.2010 04:15 # 0

      Интересно, а откуда ноги растут у такого кода? Если `name` - указатели или массивы, то типический чайник как правило реализует сравнение как `goodsBuff[k].name == vmass[p].name`, без использования оператора `&`. Код, разумеется, успешно компилируется и чайник до поры до времени не замечает подвоха.

      А здесь мы наблюдаем дополнительное использование оператора `&`. Зачем? Среднестатистический чайник до такого бы просто не додумался.

      В Си коде такое может быть оправдано, если каким-то образом массивы `goodsBuff[]` и `vmass[]` перекрываются (это могут быть указатели в один и тот же массив) и автор просто хотел отловить возможный факт того, что `goodsBuff[k].name` и `vmass[p].name` - один и тот же объект. (В С++ коде возмрожна еще попытка проверки того, что ссылки `name` указывают на один и тот же объект).
      Ответить

      • ava absolut 26.10.2010 10:09 # +1

        Всё о чём вы здесь говорите относительно оправдания кода, я, более чем уверен, в голову автора не попадало вообще. & для массива опускается компиляторами (не знаю на счет всех, но в некоторых точно). А зачем присутствует в коде? Да потому что это крутой значок :)
        Ответить

        • ava Govnoeb 26.10.2010 16:50 # −1

          найди мне компилятор, который будет вылетать на ассерте
          {
char p[16];
assert(&p[0] == p);
}
          Ответить

        • ava TheCalligrapher 26.10.2010 23:29 # +1

          "& для массива опускается компиляторами" - откуда это? Оператор `&`, будучи примененным к массиву, компиляторами никогда не опускается. Результат такого `&` численно будет совпадать с адресом первого элемента массива, но тип этого результата будет "указатель на массив", а не "указатель на элемент". Т.е. эффект оператора `&`, примененного к массиву, существует, но существует он только на уровне понятий языка ("тип указателя") и в сгенерированном машинном коде сам по себе не отражается. (Это, наверное, можно ошибочно принять за опускание).
          Ответить

          • ava absolut 27.10.2010 07:38 # +1

            Про тип согласен. Но в ГК же не типы сравниваются, а значения. С другой стороны - код-то сишный. Как в Си тип проверять будете?
            Ответить

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