- 1
throw new TSilentException("");
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+163
throw new TSilentException("");
+149
struct SomeData
{
...
double balls;
....
}
+180
...
#define EXT12 //Не комментировать эту директиву. Иначе не работает.
...
#define EXT34 //Эта директива почти не используется. Может перерефакторим без неё?
//Ну убирай.
//Давай ты.
//Там есть пару мест, где она используется. Поэтому я не хочу.
//Но ведь нужно. Эти директивы можно заменить на полиморфизм классов.
//Ну займись. Я не хочу. Вдруг сломается.
...
У нас 2 человека работают над одной веткой проекта на С++. Люди работают в отделах на разных этажах (один на этаж выше другого). Им лень сходить друг к другу или поговорить по жаберу. Я уже как полгода наблюдаю за картиной: они переговариваются через комментарии в коде через систему контроля версий. Я не против переговоров в системе контроля версий, но они уже месяц осбуждают EXT34, а так ничего с ним и не сделали. Хоть бы комментарии удалили, если делать ничего не собираюются. Может с них 0.125 премии снять?
Как-нибудь ещё переговоры повеселее выложу.
+160
QString convToHex( unsigned char *bytes, int size )
{
if( size > 16 )
size = 16;
QString ret;
for( int i = 0; i < size; i++ )
{
char buf[64];
::snprintf( buf, sizeof(buf), "%02x", (unsigned int) bytes[i] );
ret += buf;
}
return ret;
}
QString TarCreator::generateGuid( const std::string &tDeviceSerial,
const std::string &nDatetime, unsigned long long int id )
{
std::ostringstream s;
s << tDeviceSerial;
s << nDatetime;
s << id;
QCryptographicHash hash( QCryptographicHash::Md5 );
hash.addData( QByteArray( s.str().c_str() ) );
QByteArray result = hash.result();
return convToHex( (unsigned char*)result.data(), result.size() );
}
+160
void IRQHandler(void)
{
...
WaitSemaphore(semH);
Из кода для встроенного устройства реального времени.
+164
i = j ? k : l = m;
опять же с форума wasm.ru... пол дня объяснял почему с++ компилятор трактует это выражение как i = j ? k : (l = m), а никак по-другому... не пишите такую чушь никогда)
+166
// 0. original version:
struct Subscriber {
ServiceConfig *services;
};
struct ServiceConfig {
};
// reported problem: as per doc, Customer may have multiple service configurations, yet we can configure only one.
// 1. next "fixed" version, as delivered to the customer:
struct Subscriber {
};
struct ServiceConfig {
Subscriber *subs;
};
// ticket reopened: wtf. changing services for one customer, affects others!!
// 2. next "fixed" version, as delivered to the customer:
struct Subscriber {
ServiceConfig *services;
};
struct ServiceConfig {
};
// ticket reopened: @#$%^& [*BEEP*] @#$%^& [*BEEP*] @#$%^&*
// 3. finally:
struct Subscriber {
};
struct ServiceConfig {
};
struct SubscriberServiceConfig {
Subscriber *subs;
ServiceConfig *services;
};
если бы лично не лицезрел эту переписку (навечно задокументированую в аналах вершн контрол) между нашими разработчиками и консультантами работающими на инсталяции у кастомера, нивжисть бы не поверил что такое даже может иметь место.
самое страшное в этой истории, что консультанты разработчикам с самого начала сказали что в коде данные организованы неправильно по сравнению с тем как данные хранятся в базе.
+178
struct rt_asserter_t {
string func_;
rt_asserter_t( string func ): func_(func){};
void operator = (bool condition) {
if( condition == true ) {
;;;
} else if (condition == false) {
throw new rt_assert_ex(func_);
} else {
throw new unknown_ex(func_);
}
}
};
реальный крупный коммерческий проект.
Какой-то тайный замысел в этой магии.
+159
try
{
for(..)
{
for(..)
{
if(...)
throw;//генерация исключения
}
}
}
catch(..)
{
}
Кому-то религия не позволяет юзать goto. Сорри, если было.
+164
if ((AnalysisType == 0) || (!m_UseExtraInfo)) {
o1 = o2 = -1;
do {
fseek(F, o1, SEEK_END); Buf[0] = fgetc(F);
if ((Buf[0] == 0xFF) || ((Buf[0] == 0x7F) && (AnalysisType == 0))) {
o2 = o1; o1--; FFcnt = 1;
do {
fseek(F, o1, SEEK_END); Buf[0] = fgetc(F);
if ((Buf[0] == 0xFF) || ((Buf[0] == 0x7F) && (AnalysisType == 0)))
FFcnt++;
o1--;
}
while ((Buf[0] == 0xFF) || ((Buf[0] == 0x7F) && (AnalysisType == 0)));
if ((FFcnt == 71) || (FFcnt == 73)) PosTavern = o2;
if ((PosCheater == -1) && (FFcnt > 50)) {
PosCheater = o2; fseek(F, o2, SEEK_END);
do {
Buf[0] = fgetc(F);
if (Buf[0] != 0x00) PosCheater++;
}
while (Buf[0] != 0x00);
}
}
else o1--;
}
while ((o1 > -sz) &&
(((AnalysisType == 1) && (PosTavern == -1)) ||
((AnalysisType == 0) && (PosCheater == -1))));
}
else {
o1 = o2 = -200;
oFFcnt = -1;
do {
fseek(F, o1, SEEK_END);
Buf[0] = fgetc(F); Buf[1] = fgetc(F);
if ((Buf[0] == HL) && (Buf[1] == HR)) {
o2 = o1 + 86;
FFcnt = 0;
do {
fseek(F, o2, SEEK_END);
Buf[0] = fgetc(F);
if (Buf[0] == 0xFF) {
FFcnt++; o2--;
}
}
while (Buf[0] == 0xFF);
if ((oFFcnt == -1) || (abs(72 - FFcnt) < abs(72 - oFFcnt))) {
PosTavern = o1; oFFcnt = FFcnt;
}
}
o1--;
}
while (o1 > -sz);
if (PosTavern != -1)
PosTavern = PosTavern + 86 + 0x91 * (7 - PN);
if (m_ChangeHeroes && ((HL_new >= 0) || (HR_new >= 0))) {
o1 = o2 = -1;
do {
fseek(F, o1, SEEK_END); Buf[0] = fgetc(F);
if ((Buf[0] == 0xFF) || (Buf[0] == 0x7F)) {
o2 = o1; o1--; FFcnt = 1;
do {
fseek(F, o1, SEEK_END); Buf[0] = fgetc(F);
if ((Buf[0] == 0xFF) || (Buf[0] == 0x7F)) FFcnt++;
o1--;
}
while ((Buf[0] == 0xFF) || (Buf[0] == 0x7F));
if (FFcnt > 50) {
PosCheater = o2; fseek(F, o2, SEEK_END);
do {
Buf[0] = fgetc(F);
if (Buf[0] != 0x00) PosCheater++;
}
while (Buf[0] != 0x00);
}
}
else o1--;
}
while ((o1 > -sz) && (PosCheater == -1));
}
}
Кустарно-мистический анализатор сейвов для Героев 3.
Так я писал пять с половиной лет назад...