- 1
while(*(++a)=*(++b));-
C++ / Говнокод #1750
+44.3
Краткость - сетра таланта.
Запостил:
Говногость,
02 Сентября 2009
Говнокод: по колено в коде.
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
© 2008-2024 «Говнокод.ру»
Обратная связь | Лицензионное соглашение | 
Follow us!
Follow us!
Во всех учебниках так и делается копирование строк (ну, правда, a++ вместо ++a).
Следовательно или это не говнокод или C++ говноязык (конечно это не говнокод :) )
Кроме исключительно небезопасного в сях иногда встречается просто небезопасный :)
1. К C++ имеет отношение, только если его рассматривать как наследник C.
2. Почему вы решили, что это копирование строк? Может слонов. А может счетов пользователей.
3. Что будет, если указатель нулевой?
4. Как оценить размер буфера-приемника?
...
можно продолжать и дальше, но лучше отправить читать Саттера, Александреску, Макконела
вот мой говнокод:
a = b + 1;
сколько тезисов сможешь привести почему так нельзя писать?
по твоим недодумкам:
1. чушь какую-то сказал
2. а что ещё в сях имеет терминатор \0, сработающий как false? слоны, однозначно!
3. а если пернуть на спичку, жопа загорится? автор к сожалению не догадался привести ещё пару десятков предшествующих строк.
4. очередной вывод из воздуха
consider this:
class DerClass
{
public:
...
operator bool () const { return false; }
DerClass& operator = (DerClass const & rhs) { return *this;}
DerClass& operator ++ () { return *this;}
};
void Uzhos(DerClass* a, DerClass* b)
{
while(*(++a)=*(++b));
}
оригинальный код я говном не считаю, кстати, просто тут почему-то многие уверены в том, что это сишный код и обязом строки и обязом копируются.
> код исключительно Сишный и под сишные строки
> ты дурак? где ты видел в сях исключительно безопасный код?
> а что ещё в сях имеет терминатор \0, сработающий как false? слоны, однозначно!
По вашему фрагменту всего один совет - избегайте имен переменных оторванных от предметной области. Код, а не комментарии должны говорить о том, что Вы делаете.
Дискутирование на тему окружаещего кода неуместно: вот тот пример, который я упоминал, полностью:
void scopy(char* s1, char* s2)
{
while(*s1++=*s2++);
}
Авторский синтаксис сохранен.
Проблема скорее в другом, данный алгоритм предполагает завершение по "\0" а не по длинне строки, поэтому memcpy() - не альтернатива, а альтернативой является что-нибудь вроде strcpy(). Но даже для strcpy() мои рассуждения выше верны.
Да, я так и не понял, чем плох ассемблер?
memcpy можно использовать так:
memcpy(...,strlen(str),...);
На новых компах сия команда работает много медленнее, чем цикл. Причина в том, что за счет предсказания ветвлений и возможности параллельного исполнения команд это работает быстрее, тк rep movsx не параллелиться с другими командами.
А по поводу инлайнов, я говорю факт того, что у меня компилилось под netbsd с помощью gcc4
*(++a)=*(++b)
Это значит "Значение переменной, находящейся по адресу (а+1) увеличить в (b+1) раз" ...
= *(++б)
=* (++б)
что из этого сработает, так сказать?