Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−2
//сравниваем два числа, функция не использует операторы < и > в целях переносимости на платформы, где они не поддерживаются
auto intcmp( int a, int b ) -> int {
while( a && b ) {
a--;
b--;
}
if ( a == 0 && b == 0 ) // числа равны
return 0;
if ( a == 0 ) // a - меньше
return -1;
if ( b == 0 ) // a - больше
return 1;
assert( true ); // да нам подсунули какие-то неправильные числа
}К слову "auto foo( ... ) -> type" добавили в C++11.
Fai,
27 Июня 2012
+65
if ($_GET['where'] != '') {
if ($_GET['where'] != '') {
die ("<meta http-equiv='refresh' content='0; url=".$page."?where=".$_GET['where']."'>");
} else {
die ("<meta http-equiv='refresh' content='0; url=".$page."?pid=".$_GET['pid']."'>");
}
} else {
die ("<meta http-equiv='refresh' content='0; url=".$page."?pid=".$_GET['pid']."'>");
}Интересно, в каком случае выполнится второй die()?
domaster,
27 Июня 2012
+71
# Разбор time()
function GetTime( $time ) {
# Получение массива из элементов даты
$time = getdate( $time );
# Все элементы с ведущими нулями
$d = ( $time['mday'] > 9 ) ? $time['mday'] : '0' . $time['mday'];
$w = ( $time['mon'] > 9 ) ? $time['mon'] : '0' . $time['mon'];
$y = $time['year'];
$h = ( $time['hours'] > 9 ) ? $time['hours'] : '0' . $time['hours'];
$m = ( $time['minutes'] > 9 ) ? $time['minutes'] : '0' . $time['minutes'];
$s = ( $time['seconds'] > 9 ) ? $time['seconds'] : '0' . $time['seconds'];
# Сборка и возврат результата
return $d . '/' . $w . '/' . $y . ' ' . $h . ':' . $m . ':' . $s;
}Про date() явно не в курсе...
nethak,
27 Июня 2012
−29
calculateSomething(int n){
float* x = new float[n];
/*do smth*/
if ( /*validation*/ ) {
delete[] x;
return;
}
float* y = new float[n];
/*do smth*/
if ( /*validation*/ ) {
delete[] x;
delete[] y;
return;
}
float* z = new float[n];
/*next step*/
if ( /*smth goes wrong*/ ) {
delete[] x;
delete[] y;
delete[] z;
return;
}
/*more calculations with validation*/
delete[] x;
delete[] y;
...
/*more delete[]*/
return;
}Принципиальная схема говногода, найденного на просторах сети.
interested,
26 Июня 2012
+129
if (!File.Exists(filePath))
{
throw new FileNotFoundException("File is not a file!", filePath);
}Вот такая вот философия шестилетней давности. Собственное говно :)
anmiles,
26 Июня 2012
+62
function find( &$db, $id) {
$statement = Contact::findStatement() . " WHERE id = $id";
$dbResult = ActiveRecord::abstractFind( $db, $statement);
foreach ($dbResult as $dbRow) {
return Contact::load( $db, $dbRow);
}
return null;
}
shmaltorhbooks,
26 Июня 2012
+141
<?php
if(!$this->model_forum->community_once(array('id'=>$community))) {
echo file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/no_community.tpl');
}
else if(isset($_GET['topic']) && is_numeric($_GET['topic']) && $topic = $this->model_forum->topic_once(array('id'=>(int)$_GET['topic']))) {
$tpl = file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/add_response_topic.tpl');
$tpl = str_replace('{topic_id}', (int)$_GET['topic'] ,$tpl);
echo $tpl;
}
else {
$tpl = file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/add_topic.tpl');
$tpl = str_replace('{community}',$community, $tpl);
$tpl = str_replace('{topic_date_closing}', '23', $tpl);
echo $tpl;
} ?>оцените на предмет гавнокода
nepster,
26 Июня 2012
+184
__declspec (naked) INT16U getWordOfCPUFlags(void)
{
_asm
{
pushf;
xor eax, eax;
pop ax;
ret;
}
}
void setWordOfCPUFlags(const INT16U WordOfCPUFlag)
{
_asm
{
push WordOfCPUFlag;
popf;
}
}
bool getCPUTraceFlag(void)
{
return 0 != (getWordOfCPUFlags() & 0x100);
}
void setCPUTraceFlag(const bool CPUTraceFlag)
{
const INT16U MaskOfTraceFlag=((INT16U)CPUTraceFlag)<<8;
//setWordOfCPUFlags( (getWordOfCPUFlags() and ~MaskOfTraceFlag) ^ MaskOfTraceFlag);
if(CPUTraceFlag)
setWordOfCPUFlags( getWordOfCPUFlags() | MaskOfTraceFlag);
else
setWordOfCPUFlags( getWordOfCPUFlags() & ~MaskOfTraceFlag);
}Видимо автору кода строка 29 показалась слишком сложной.
HaskellGovno,
26 Июня 2012
+83
private String toHTML(String unicode)
{
String output = "";
char[] charArray = unicode.toCharArray();
for (int i = 0; i < charArray.length; ++i)
{
if ((int)charArray[i]>255)
{
String s = ""+Integer.toHexString(charArray[i]);
switch (s.length())
{
case 1: s="\\u000"+s; break;
case 2: s="\\u00"+s; break;
case 3: s="\\u0"+s; break;
case 4: s="\\u"+s; break;
default: throw new RuntimeException( s +" is tool long to be a Character");
}
output += s;
}
else
{
output += charArray[i];
}
}
return output;
}Эпичнейший говнокод! На серваке top показывает нагрузку 10-12. 3000 пользователей, 100 нод, интеграция с SAP, который пачками проводит документы и выдаёт цены, отчёты по остаткам и т.п. И всё это, как оказалось, капля в море по сравнению с 5 человеками техподдержки, которые сидят в аяксовой консоле мониторинга, для которой HTTP-ответ экранируется данным шедевром. Без этого шедевра нагрузка держится в районе 2-3 даже при достаточно большой активности.
konsoletyper,
26 Июня 2012
+67
$this->bgcolor = $bgcolor ? ($bgcolor <= 0xFFFFFF) ? ($bgcolor > 0xFFFFFF) ? 0xFFFFFF : $bgcolor : 0x000000 : 0xFFFFFF;FFFFFFFFFFFFFFFFFFFFFFF
Razban_Guestov,
26 Июня 2012
Follow us!