Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+960
// Encode() takes in reference to data. We now pass in reference to data instead of the
// entire data object to Encode() as it was causing memory leak(with the data object copy
// not getting deleted.
Data &tempData = *data;
Data::Encode(tempData, &encodedRequest, requestSize, 0);
Перевод:
Encode() принимает ссылку на data. Давайте передадим туда ссылку,
а то раньше мы передавали объект целиком, и это приводило к утечкам памяти (копия объекта не удалялась).
Здесь впору процитировать "Бойцовский Клуб":
-А в какой Вы фирме работаете?
-В крупной...
belca,
13 Мая 2010
+144
/*
Некий не работающий кусок закомментированного кода.
*///It's fuckin shit C++!!!Обнаружено в одном из наших проектов.
Говногость,
13 Мая 2010
+116
class DllContainer
{
DllContainer()
{
// тут грузятся дллки в количестве N.
// LoadLibrary() + некоторые операции
}
~DllContainer()
{
// FreeLibrary() и т.п.
}
template <class T>
T* GetComponent(ComponentID id)
{
// аналог QueryInterface.
// ищет компонент, проверяет можно ли статик_кастить
// и вертает указатель нужного типа
}
};
class ComponentUser
{
void Method1()
{
DllContainer loader;
SomethingDoer* comp = loader.GetComponent<SomethingDoer>(ID1);
comp->DoSomething();
}
void Method2()
{
DllContainer loader;
SomethingElseDoer* comp = loader.GetComponent<SomethingElseDoer>(ID2);
comp->DoSomethingElse();
}
void MethodN()
{
DllContainer loader;
ShitPerformer* comp = loader.GetComponent<ShitPerformer>(IDN);
comp->PerformSomeShit();
}
};
недавно обнаружил код примерно такого плана.
крупный коммерческий проект...
g26g,
12 Мая 2010
+128
/********************************************************************
ОПЕРАЦИЯ ЛОГИЧЕСКОЕ ИСКЛЮЧАЮЩЕЕ ИЛИ
********************************************************************/
static inline bool XOR(bool lhs, bool rhs)
{
return (lhs && !rhs) || ( !lhs && rhs);
}Человек не знал, что есть стандартный xor...
m08pvv,
11 Мая 2010
+115
#include "iostream"
using namespace std;
int main()
{
cout << "Vvedite storonu a:";
int a;
cin >> a;
cout << "Vvedite storonu b:";
int b;
cin >> b;
cout << "Vvedite storonu c:";
int c;
cin >> c;
int g=1;
while(g=1){
if(a==0||b==0||c==0){ cout << "Ne treugolnik";
break;}
if(a<=b+c || c<=a+b || b<=a+c){ cout << "Daaa!!! Treugolnik";}
break;}
return 0;
}Вот как мы узнаем, треугольник ли это по сторонам.
hromjo,
11 Мая 2010
+144
string buf;
...
char c_buf[MAX_LEN];
strncpy(c_buf, buf.c_str(), MAX_LEN);в чём ошибка?
skor4ik,
10 Мая 2010
+116
struct Epica
{
public:
signed int Value;
};
//...
Epica* ValEp[10][17];
//...
ValEp[i][jj][0].Value;Угадайте, что с автором?
Говногость,
09 Мая 2010
+98
template<unsigned AxesQount, class Conten>
const Vector<AxesQount, Conten> Vector<AxesQount, Conten>::operator-(void) const
{
return *this*-1.0;
};
template<unsigned AxesQount, class Conten>
const Vector<AxesQount, Conten> Vector<AxesQount, Conten>::operator+(void) const
{
return *this;
};Чужой велосипед. Пытался использовать в своём проекте, но передумал.
Говногость,
07 Мая 2010
+100
int wmain()
{
_rmCfg rmCfg;
double tt;
int i;
String *filePath=S"C:\\rm\\rm.cfg";
String * sqlstr;
rmCfg.isread=0;
String * dir[];
DateTime dt;
if (File::Exists(filePath))
{
StreamReader *sr= new StreamReader(filePath);
rmCfg.gosanPath=sr->ReadLine();
rmCfg.iakServer=sr->ReadLine();
rmCfg.iakUser=sr->ReadLine();
rmCfg.iakPassword=sr->ReadLine();
rmCfg.trackId=sr->ReadLine()->ToInt32(0);
// rmCfg.trName=sr->ReadLine();
Console::WriteLine(S"Viewing directory: ");
Console::Write(rmCfg.gosanPath);
Console::WriteLine();Писано было на втором фреймворке. Так бы и продолжал писать, если бы товарищ не намекнул про сериализацию.
ursus,
05 Мая 2010
+103
#include <iostream>
#include <string>
#include <boost/foreach.hpp>
int main(int argc, char* argv[]) {
std::string str = "Boost FOREACH!";
char ch;
if (boost::foreach_detail_::auto_any_t _foreach_col11 = boost::foreach_detail_::contain( ( str ) , (true ? 0 : boost::foreach_detail_::or_( boost::foreach_detail_::and_( boost::foreach_detail_::not_(boost::foreach_detail_::is_array_( str )) , (true ? 0 : boost::foreach_detail_::is_rvalue_( (true ? boost::foreach_detail_::make_probe( str ) : ( str )), 0))) , boost::foreach_detail_::and_( boost::foreach_detail_::not_( boost_foreach_is_noncopyable( boost::foreach_detail_::to_ptr( str ) , boost_foreach_argument_dependent_lookup_hack_value) ) , boost_foreach_is_lightweight_proxy( boost::foreach_detail_::to_ptr( str ) , boost_foreach_argument_dependent_lookup_hack_value)) )) ) ) {} else if (boost::foreach_detail_::auto_any_t _foreach_cur11 = boost::foreach_detail_::begin( _foreach_col11 , (true ? 0 : boost::foreach_detail_::encode_type( str , boost::foreach_detail_::is_const_( str ))) , (true ? 0 : boost::foreach_detail_::or_( boost::foreach_detail_::and_( boost::foreach_detail_::not_(boost::foreach_detail_::is_array_( str )) , (true ? 0 : boost::foreach_detail_::is_rvalue_( (true ? boost::foreach_detail_::make_probe( str ) : ( str )), 0))) , boost::foreach_detail_::and_( boost::foreach_detail_::not_( boost_foreach_is_noncopyable( boost::foreach_detail_::to_ptr( str ) , boost_foreach_argument_dependent_lookup_hack_value) ) , boost_foreach_is_lightweight_proxy( boost::foreach_detail_::to_ptr( str ) , boost_foreach_argument_dependent_lookup_hack_value)) )) ) ) {} else if (boost::foreach_detail_::auto_any_t _foreach_end11 = boost::foreach_detail_::end( _foreach_col11 , (true ? 0 : boost::foreach_detail_::encode_type( str , boost::foreach_detail_::is_const_( str ))) , (true ? 0 : boost::foreach_detail_::or_( boost::foreach_detail_::and_( boost::foreach_detail_::not_(boost::foreach_detail_::is_array_( str )) , (true ? 0 : boost::foreach_detail_::is_rvalue_( (true ? boost::foreach_detail_::make_probe( str ) : ( str )), 0))) , boost::foreach_detail_::and_( boost::foreach_detail_::not_( boost_foreach_is_noncopyable( boost::foreach_detail_::to_ptr( str ) , boost_foreach_argument_dependent_lookup_hack_value) ) , boost_foreach_is_lightweight_proxy( boost::foreach_detail_::to_ptr( str ) , boost_foreach_argument_dependent_lookup_hack_value)) )) ) ) {} else for (bool _foreach_continue11 = true; _foreach_continue11 && ! boost::foreach_detail_::done( _foreach_cur11 , _foreach_end11 , (true ? 0 : boost::foreach_detail_::encode_type( str , boost::foreach_detail_::is_const_( str ))) ) ; _foreach_continue11 ? boost::foreach_detail_::next( _foreach_cur11 , (true ? 0 : boost::foreach_detail_::encode_type( str , boost::foreach_detail_::is_const_( str ))) ) : (void)0) if (boost::foreach_detail_::set_false( _foreach_continue11 )) {} else for ( ch = boost::foreach_detail_::deref( _foreach_cur11 , (true ? 0 : boost::foreach_detail_::encode_type( str , boost::foreach_detail_::is_const_( str ))) ) ; ! _foreach_continue11 ; _foreach_continue11 = true) {
std::cout << ch;
}
std::cout << std::endl;
return 0;
}Конечно, это не совсем говнокод, но... я офигел когда посмотрел во что превращается простой BOOST_FOREACH
rudvil,
03 Мая 2010
Follow us!