- 1
- 2
- 3
- 4
- 5
- 6
if (this.bmp == null) {
} else if (this.bmp.Size.Height < this.ViewSize.Height) {
DisposeBmp();
} else if (this.bmp.Size.Width < this.ListData.RowSize.Width) {
DisposeBmp();
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+119
if (this.bmp == null) {
} else if (this.bmp.Size.Height < this.ViewSize.Height) {
DisposeBmp();
} else if (this.bmp.Size.Width < this.ListData.RowSize.Width) {
DisposeBmp();
}
Еще один Американский перл.... Присмотритесь к 1-му "if". И так по всему коду.
+75
String q = "SELECT login, accessLevel FROM accounts ";
if (m.equals("1"))
q = q.concat("WHERE accessLevel<0");
else if (m.equals("2"))
q = q.concat("WHERE accessLevel>0");
else if (m.equals("3"))
q = q.concat("WHERE accessLevel=0");
q = q.concat(" ORDER BY login ASC");
из l2j
+163
<input type="submit" value="<?=lang('Send','Отправить')?>" />
Говнокод или нет?
+138
-(void)terminateSearchThreadInBackground:(NSNumber*)threadPtr
{
NSAutoreleasePool* pool = [[NSAutoreleasePool alloc] init];
SearchThread* thread = (SearchThread*)[threadPtr unsignedLongValue];
delete thread;
[pool release];
}
И вновь я в шоке от нашего проекта. По какой-то неведомой причине передать в качестве параметра указатель на поток - это очень не трушно. Зато значительно более трушно - создать из него NSNumber, предварительно преобразовав к unsigned long...
+998
#define TRACE_VAR ATraceClass *mTrace
#define TRACE_VAR_INIT mTrace = ATraceClass::getInstance()
class ATraceClass {
public:
void log(int severity, const char *format, ...);
/* ... */
}
class ATraceUser {
public:
ATraceUser::ATraceUser() {
TRACE_VAR_INIT;
}
int SomeMethod1();
/* ... */
TRACE_VAR;
}
int ATraceUser::SomeMethod1()
{
/* ... */
if (errorCondition)
{
return mTrace->log( ERR, "Something went terribly"\
"wrong when doing the operation %s\n",\
mOperation),-1;
}
/* ... */
}
в догонку к #3355 - это уже как бы пример стиля использования логгера егоным же автором.
пришел ко мне как-то народ, показал нечто что я тут иллюстрирую в SomeMethod1() и спросил:
(1) а на кой эти слеши? может нам и весь остальной код нужно ими разукрасить??
(2) а как это void можно из int функции возвращать? и как это вообще работает? вроде в С/С++ такое нельзя....
у меня до сих пор в голове не укладывается каким .... надо быть что бы такой код писать.
−123
my @orda = ();
my @whera = (); # Niggaz in da haus!
Найдено в коде метода, формирующего SQL-запрос для доступа к БД.
+1003
// укорочено для краткости.
void ATraceClass::LogFunction(const char *format, va_list ap)
{
/* ... declarations ... */
for (p=format; (*p)&&(!isTruncated); p++)
{
if ( * p == '%' )
switch( * ++p )
{
case 'c':
{ /* ... */ }
break;
case 'd':
case 'i':
{ /* ... */ }
break;
case 'l': // это %ld
{ /* ... */ }
break;
case 'L': // это %Ld
{ /* ... */ }
break;
case 'f': // a это %g
{ /* ... */ }
break;
case 'F': // a это %Lg
{ /* ... */ }
break;
case 's':
{ /* ... */ }
break;
default: // this is an ERROR case, but we will not perform coding at this point ... maybe later
break;
}
else {
/* ... */
}
}
}
наболело. одно чудило (с большой букмы Му) наговнокодило это дело под эгидой "stdio это С, мы в С++ можем круче!!!" ну с тех пор и переизобретают велосипед - с квадратными колёсами. даже %% не догадались сделать.
уже как года два давлю вот это Г, но наши немецкие умельцы копи-пастят это в новые модуля быстрее чем я успеваю это удалять.
самое гадкое что народ пишет код как обычно ожидая стандартные stdio шорткаты, и потом долго гадает куда блин сообщение подевалось.
+156
function __get($propertyName) {
....
if (!isset($propertyName[$this->propertyTable])) {
var_dump($this->propertyTable);
throw new Exception("Wrong property name \"$propertyName\"");
}
.....
}
Глубоко в дебрях...
+113
public static bool DefaultSuppressVersionCheck = true; //default is false
Как пишут программисты США.
+165
function is_bot() {
if (defined("IS_BOT")) {
return 1;
}
elseif (defined("IS_NOT_BOT")) {
return 0;
}
elseif (!defined("IS_BOT") and !defined("IS_NOT_BOT")) {
if (file_exists(INCLUDE_PATH."includes/core/is_bot.php")) {
require_once(INCLUDE_PATH."includes/core/is_bot.php");
if (function_exists('is_bot_real')) {
return is_bot_real();
}
else {
define("IS_NOT_BOT", true);
return 0;
}
}
else {
define("IS_NOT_BOT", true);
return 0;
}
}
else {
define("IS_NOT_BOT", true);
return 0;
}
}
PHPNuke, mainfile.php