- 1
- 2
- 3
- 4
QT project:
t=x1; x1=x2; x2=t;
t=y1; x1=y2; y2=t;-
C++ / Говнокод #8292
+147
И еще 90 говнопятнышек!
Налетай: http://www.viva64.com/ru/a/0077/Запостил:
Andrey_Karpov,
25 Октября 2011
Говнокод: по колено в коде.
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
© 2008-2026 «Говнокод.ру»
Обратная связь | Лицензионное соглашение | 
Follow us!
Follow us!
Ошибка там. Смотрите внимательнее.
2) Смотрим на статью.
3) Обнаруживаем разжеванное описание:
Ошибка найдена с помощью диагностики V519: The 'x1' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 2218, 2219. Qt3Support q3canvas.cpp 2219
Первая строчка совершенно корректна и обменивает значения в переменных x1 и x2. Во второй строке должны быть обменены переменные y1 и y2. Эта строка видима была копией предыдущей. В ней надо было заменить все буквы 'x' на буквы 'y'. К сожалению, в одном месте это забыли сделать: "... x1=y2; ...".
Корректный код:
t=x1; x1=x2; x2=t;
t=y1; y1=y2; y2=t;
первый раз не увидел. спс.
Спрошу конкретнее: ЧТО ИНТЕРЕСНОГО В ЭТОЙ ОШИБКЕ?
P.S.: большинство пользователей итак поняли как она могла получиться и в чем заключается, так что либо вы жирный тролль, либо человек совсем без чувства юмора =\
Многий другой код, приведенный в статье, также по сути говнокод.
Просто автор этого ГК can not into STL.
Отсюда мораль: нефиг юзать копипаст.
P.s. И рекламировать свой пмс тут тоже нефиг.
Говнокод же.
std::swap(x1,x2);
std::swap(y1,y2);
И ошибку было бы меньше шансов допустить.
std::swap(x1, y2);
осталось тока корованы с коровами шлюхами ограбить.
Это 60 (из 70) листов с говнокодом.
Остальные 10 - да, реклама. :)
В Word эта статья занимает 70 листов.
Из них 60 - код.
И негодуете зря. Почитайте. Найдете много забавного. Возможно, что-то и сюда запостите.
Дед ма-роч!
Дед ма-роч!
Выходи!
кто он, дети?
педераст
http://govnokod.ru/7781
http://govnokod.ru/7781#comment105845
> http://govnokod.ru/7781#comment115769
:-Р
ЗЫ: А знаете, кто отсутствует в нашем курятнике? Ilfak_Guilfanov!
Соврамши, ваших здесь как минимум двое отметилось.
> кому-то расскажет, какие мы плохие.
Скорее - унылые и постите несмешной говнокод с целью привлечь аудиторию к своему продукту.
Кстати, хотя бы радуга из единорога-то - оригинальная пикча?
вардан_кушнир_vs_клофелинщицы.avi
Готов ответить на вопросы.
В 2011-м году здесь были петухи?
P.S. Находим баги в LLVM 8 с помощью анализатора PVS-Studio - https://www.viva64.com/ru/b/0629/
csrf verification error
Афродита.
Вива ла Дива, Вива Виктория,
Клеопатра.
Функция substr создаёт временный объект типа std::string, который затем разрушается. В результате в current_word сохраняются указатели на уже разрушенный объект.
>
это довольно школьная ошибка. Еще Страуструп приводит пример как из стринга взяли c_str с известынм результатом. До конца выражения он дожил, а дальше всё.
Вот потому в расте такое и не скомпилируется: не может ссылка пережить родителя.
Кстати, кто-нибудь пробовал декомпилировать программу и прогнать через аналолизатор?
Можно и standalone версию.
А что? Со своим назначением он справляется. Ветвления, циклы, свитч-кейзы, вызовы функций распознает. Правда код выдает gcc-шный, еще там для флагов тип int1_t, это что?
Кстати, судя по листингу на картинке декомпилировали данные или мусор, либо ассемблерный код, компиляторы вроде не юзают das и подобные инструкции (это bcd вроде, их вообще никто не использует) и xlat.
Ну, раз ты такое утверждаешь, ты просто обязан объяснить, что он делает.
Вобщем только как помощник человекам годится.
Больше reinterpret_cast'ов богу reinterpret_cast'ов!
Я использовал какую-то инструкцию для BCD, когда мы соревновались в написании самой короткой программы в ASCII-кодах, которая ничего не делает, просто возвращает управление системе.
Кстати, в длинном режиме x86-64 что-то из этих инструкций выкинули (то ли XLAT, то ли всю пачку BCD, то ли и то, и другое).
Если бы. На коротких сниппетах — да.
При дизассемблировании длинных программ с ветвлениями и циклами приходится определять начала цепочек опкодов (на CISC можно поймать забавные багры), локализовывать и классифицировать данные. Проблемы возникают с интерпретацией магических констант. Например, 42 — это просто число или смещение в памяти какого-то объекта?
Далее если это не com-файл для DOS, то среди кода есть перемещаемые данные, которые фиксит загрузчик при старте.
В общем, когда начнёшь писать дизассемблер, способный выдать асм-файл, который можно скомпилировать, не испортив программу, увидишь, что проблемы растут как снежный ком.
Это для hiew или турбо-дебаггера легко написать «дизассемблер», который показывает опкоды для короткой цепочки кода (и то бывают багры на CISC или бесполезный выхлоп из-за отсутствия интерпретации констант).
https://govnokod.ru/13242
>дик-пик
о чем это всё?
>804. Decreased performance. The 'Foo' function is called twice in the specified expression to calculate length of the same string
Кто юзает strlen в 20 году? Ставьте лойс.