1. PHP / Говнокод #8494

    +159

    1. 1
    if($_GET['archive'] and $_GET['archive'] != ''){ $archive = $_GET['archive']; } // stupid fix ?

    cutenews

    jQuery, 11 Ноября 2011

    Комментарии (9)

  2. Java / Говнокод #8493

    +73

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    Credential credential = (Credential) getCurrentObject();
if(isNew(credential)) {
	CredentialDetails details=credential.getCurrentDetails();
	if(details == null)
		throw new NullPointerException("Credential details object is null");

	CredentialDetails updatedDetails = credential.getCurrentDetails();
	updatedDetails.setStartDate(getCurrentDate());
	updatedDetails.setExternalID(credential.getExternalID());
}
if(isNew(credential)==false) {
		
	CredentialDetails details=credential.getCurrentDetails();
	if(details == null)
		throw new NullPointerException("Credential details object is null");
	CredentialDetails updatedDetails = credential.getCurrentDetails();
	updatedDetails.setStartDate(getCurrentDate());
	updatedDetails.setExternalID(credential.getExternalID());
	m_existingDetailsClone.setEndDate(updatedDetails.getStartDate());
	updatedDetails.setPreviousDetails(m_existingDetailsClone);
			
	getModCol().addModified( m_existingDetailsClone );
}

    Такое впечатление что автор кода клинический идиот. Зачем так насиловать несчастные POJO Credential и CredentialDetailsю

    askell, 11 Ноября 2011

    Комментарии (21)

  3. JavaScript / Говнокод #8492

    +159

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    function get_selection()
{
	selection = "";
	if (document.getSelection)
	{
		selection = document.getSelection();
		selection = selection.replace(/\r\n\r\n/gi, "_doublecaret_");
		selection = selection.replace(/\r\n/gi, " ");
		while(selection.indexOf("  ") != -1)
		selection = selection.replace(/  /gi, "");
		selection = selection.replace(/_doublecaret_/gi, "\r\n\r\n");
	}
	else
	if (document.selection) { selection = document.selection.createRange().text; }
	return selection;
}

    bugmenot, 11 Ноября 2011

    Комментарии (5)

  4. PHP / Говнокод #8491

    +162

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    function section($sct,$grp = 'all') {
		global $g,$a,$d,$lists,$master_list,$currency;
		$access = ac_section($sct,$this->adminmode);
		if ($access['r'] == 'no' )  {header("Location: /errors/all/403");}
		$master = q1("SELECT * FROM `_s` WHERE `_id`='".$sct."' AND `_l`='".GetLanguage()."'");
		if (isset($_POST['pcounts'])) {
			$_SESSION['_pcount']=(int)$_POST['pcounts'];
		} elseif (isset($_GET['pcounts'])) {
			$_SESSION['_pcount']=(int)$_GET['pcounts'];
		}
		if (isset($_SESSION['_pcount'])) $master['_pcount']=$_SESSION['_pcount'];
        $master_struct = $this->Struct_fields(qa("SHOW COLUMNS FROM `_s`"));
		$grps = qa("SELECT * FROM `_g` WHERE `_s`='".$sct."' AND `_g`='".$grp."' AND `_l`='".GetLanguage()."'".((!empty($a[0]['con']))?(" AND `header` LIKE '%".$a[0]['con']."%'"):('')).((!empty($a[0]['gby']))?(" ORDER BY `".$a[0]['gby']."`".((!empty($a[0]['gdir']))?(' DESC'):(' ASC'))):(" ORDER BY ".$master['_sorting'])).((!empty($a[0]['gfrom']))?(' LIMIT '.$a[0]['gfrom'].','.((!empty($a[0]['gper']))?($a[0]['gper']):((!empty($master['_gcount']))?($master['_gcount']):('10')))):(' LIMIT 0,'.((!empty($a[0]['gper']))?($a[0]['gper']):((!empty($master['_gcount']))?($master['_gcount']):('10'))))));
		$this->cq++;
		$grps_nav_count = qa("SELECT count(*) `c` FROM `_g` WHERE `_s`='".$sct."' AND `_g`='".$grp."' AND `_l`='".GetLanguage()."'".((!empty($a[0]['con']))?(" AND `header` LIKE '%".$a[0]['con']."%'"):('')));
		$grps_len = sizeof($grps);
		$grps_struct = $this->Struct_fields(qa("SHOW COLUMNS FROM `_g`"));
		$pgs = qa("SELECT * FROM `_p` WHERE `_s`='".$sct."' AND `_g`='".$grp."' AND `_l`='".GetLanguage()."'".((!empty($a[0]['con']))?(" AND `header` LIKE '%".$a[0]['con']."%'"):('')).((!empty($a[0]['by']))?(" ORDER BY `".$a[0]['by']."`".((!empty($a[0]['dir']))?(' DESC'):(' ASC'))):(" ORDER BY ".$master['_sorting'])).((!empty($a[0]['from']))?(' LIMIT '.$a[0]['from'].','.((!empty($a[0]['per']))?($a[0]['per']):((!empty($master['_pcount']))?($master['_pcount']):('10')))):(' LIMIT 0,'.((!empty($a[0]['per']))?($a[0]['per']):((!empty($master['_pcount']))?($master['_pcount']):('10'))))));
		$this->cq++;
		$pgs_nav_count = qa("SELECT count(*) `c` FROM `_p` WHERE `_s`='".$sct."' AND `_g`='".$grp."' AND `_l`='".GetLanguage()."'".((!empty($a[0]['con']))?(" AND `header` LIKE '%".$a[0]['con']."%'"):('')));
		$this->cq++;
		$pgs_len = sizeof($pgs);
		$pgs_struct = $this->Struct_fields(qa("SHOW COLUMNS FROM `_p`"));
		$list_g = '';
		$list_p = '';
// Дальше еще строк 200 подобного кода

    Нашел бэкап сайта, который надо было дорабатывать
    И это только одна функция одного из классов. Всего около 15 классов, в каждом по 30-40 функций.
    При этом есть еще куча файлов вызываемых напрямую(отправка сообщений, добавление/редактирование/удаление данных в админке), хотя и используется mod_rewrite

    долго думал, потом переписал нужные куски заново

    Bagration, 11 Ноября 2011

    Комментарии (28)

  5. PHP / Говнокод #8490

    +160

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    <?php
$lquery = mysql_query("SELECT * FROM language_settings ORDER BY language_id");
$llquery = mysql_query("SELECT * FROM language_settings ORDER BY language_id");
?>

<?php while($ldata = mysql_fetch_array($lquery)){ ?>
<a><?=$ldata['language_name'];?></a>
<?php } ?>

<?php while($lldata = mysql_fetch_array($llquery)){  // Language Loop
$lang_prefix = $lldata['language_prefix'];
language($lang_prefix);
$lang = strtolower($lldata['language_prefix']);
?>
        ....
<?php
$_lid = $lldata['language_id'];
$_nid = $_REQUEST['nid'];
if($_nid!=""){
// newsid=$_nid AND
$qry = "SELECT *
	FROM tbl_products as pc JOIN tbl_products_content as pcc ON pc.product_id=pcc.product_id
	WHERE  pcc.lang_prefix='".strtolower($lldata['language_prefix'])."' AND pc.product_id='{$_nid}' ORDER BY product_order,product_cat_id";
       $cquery = mysql_query($qry);
       $cdata = @mysql_fetch_array($cquery);
}
           ...
}?>

    Вот... попалось недавно такое произведение...

    tsybulskyserg, 11 Ноября 2011

    Комментарии (3)

  6. Java / Говнокод #8489

    +74

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    public class User extends HttpApiDealer{
    private int id = 347;
    private Date lustUpdate;
    private long refreshTime = 10;
    private boolean onlineStatus;
    
    
    public boolean isOnline() {
        Date timeForRefresh = new Date();
        timeForRefresh.setTime(new Date().getTime() - refreshTime);
        if(lustUpdate == null) {
            lustUpdate = new Date();
        }else if(lustUpdate.after(timeForRefresh)) {
            
        }
        
        return true; //заглушка
    }
    
}

    Хм, смущает меня работа с датами в этом коде.

    manyrus, 11 Ноября 2011

    Комментарии (10)

  7. C++ / Говнокод #8488

    +1003

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    /c/Qt/4.7.4/src/plugins/bearer/symbian/symbianengine.cpp:1336..1344

// Waits for 2..6 seconds.
void SymbianEngine::updateConfigurationsAfterRandomTime()
{
    int iTimeToWait = qMax(1000, (qAbs(qrand()) % 68) * 100);
#ifdef QT_BEARERMGMT_SYMBIAN_DEBUG
    qDebug("QNCM waiting random time: %d ms", iTimeToWait);
#endif
    QTimer::singleShot(iTimeToWait, this, SLOT(delayedConfigurationUpdate()));
}

    68 будет в самый раз, да.

    overloop, 11 Ноября 2011

    Комментарии (23)

  8. ActionScript / Говнокод #8487

    −113

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    if(_FRM_==302 || _FRM_==303 || _FRM_==304){//KAT
		if (GETKEY(302) == "A") {return 302}else if (GETKEY(303) == "A") { return 303}else if (GETKEY(304) == "A") {return(304)}
	}else if(_FRM_==305 || _FRM_==306 || _FRM_==307){//DOG
		if (GETKEY(305) == "A") {return(305)}else if (GETKEY(306) == "A") { return(306)}else if (GETKEY(307) == "A") {return(307)}
	}else if(_FRM_==300 || _FRM_==308){// FISH
		if (GETKEY(300) == "A") {return(300)}else if (GETKEY(308) == "A") {return(308)}
	}else if(_FRM_==301 || _FRM_==309){ // BIRD
		if (GETKEY(301) == "A") {return(301)}else if (GETKEY(309) == "A") {return(309)}
	}else if(_FRM_==64 || _FRM_==352 || _FRM_ == 356 || _FRM_==363){// mc_LOJKA
		if (GETKEY(64) == "A") {return(64)
		}else if (GETKEY(352) == "A") {return(352)
		}else if (GETKEY(356) == "A") {return(356)
		}else if (GETKEY(363) == "A") {return(363)}
	}else if(_FRM_==357 || _FRM_==26){ //mc_NEPROLIVAIKA 
		if (GETKEY(357) == "A") {return(357)
		}else if (GETKEY(26) == "A") {return(26)}
	}else if(_FRM_==42 || _FRM_==43 || _FRM_==44){ //mc_BUTILKA
		if (GETKEY(42) == "A") {return(42)
		}else if (GETKEY(43) == "A") {return(43)
		}else if (GETKEY(44) == "A") {return(44)}
	}else if(_FRM_==40 || _FRM_==38 || _FRM_==37){//mc_SOSKA
		if (GETKEY(40) == "A") {return(40)
		}else if (GETKEY(38) == "A") {return(38)
		}else if (GETKEY(37) == "A") {return(37)}
	}else if(_FRM_==30 || _FRM_==31 || _FRM_==32){// mc_PUSTYSHKA
		if (GETKEY(30) == "A") {return(30)
		}else if (GETKEY(31) == "A") {return(31)
		}else if (GETKEY(32) == "A") {return(32)}

    И еще всего сотенка строк, а вообще есть ифы такого плана по 700 строк, просто кусочки их не столь эпичны
    Вычисляем столкновения!.

    kyzi007, 11 Ноября 2011

    Комментарии (22)

  9. ActionScript / Говнокод #8486

    −113

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    var _str = ""
	var _a_=""
	_a_ = KID_DANGER_pos.toString(16);	if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = KID_DANGER_cnt.toString(16); if (_a_.length < 2) { _str += "0" };_str += _a_
	_a_ = KID_EAT_pos.toString(16);	if (_a_.length < 2) { _str += "0" };_str += _a_;_a_ = KID_EAT_cnt.toString(16);	if (_a_.length < 2) { _str += "0" };_str += _a_
	_a_ = KID_PLAY_pos.toString(16); if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = KID_PLAY_cnt.toString(16); if (_a_.length < 2) { _str += "0" };_str += _a_
	_a_ = KID_WASH_pos.toString(16); if (_a_.length < 2) { _str += "0" }; _str += _a_;	_a_ = KID_WASH_cnt.toString(16); if (_a_.length < 2) { _str += "0" };_str += _a_
	_a_ = KID_SLEEP_pos.toString(16); if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = KID_SLEEP_cnt.toString(16); if (_a_.length < 2) { _str += "0" };_str += _a_
	_a_ = KID_EDU_pos.toString(16); if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = KID_EDU_cnt.toString(16); if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = KID_STREET_pos.toString(16); if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = KID_STREET_cnt.toString(16); if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = KID_CLEAN_pos.toString(16); if (_a_.length < 2) { _str += "0" }; _str += _a_;	_a_ = KID_CLEAN_cnt.toString(16);	if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = KID_GARBAGE_pos.toString(16);	if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = KID_GARBAGE_cnt.toString(16); if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = KID_TOILET_pos.toString(16); if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = KID_TOILET_cnt.toString(16); if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = KID_HEALTH_pos.toString(16); if (_a_.length < 2) { _str += "0" };	_str += _a_; _a_ = KID_HEALTH_cnt.toString(16);	if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = KAT_event_pos.toString(16); if (_a_.length < 2) { _str += "0" };	_str += _a_; _a_ = KAT_event_cnt.toString(16);	if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = DOG_event_pos.toString(16); if (_a_.length < 2) { _str += "0" };	_str += _a_; _a_ = DOG_event_cnt.toString(16);	if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = FISH_event_pos.toString(16);	if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = FISH_event_cnt.toString(16); if (_a_.length < 2) { _str += "0" };	_str += _a_
	_a_ = BIRD_event_pos.toString(16);	if (_a_.length < 2) { _str += "0" }; _str += _a_; _a_ = BIRD_event_cnt.toString(16);if (_a_.length <2) { _str+="0" };	_str += _a_

    Эмм, работа со строками...

    kyzi007, 11 Ноября 2011

    Комментарии (73)

  10. ActionScript / Говнокод #8485

    −115

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    function MONEY_REDRAW() {
	mc_ROOM.mc_MONEY_A._0.gotoAndStop(CHR_MONEY_A%10+1)
	mc_ROOM.mc_MONEY_A._1.gotoAndStop(int(CHR_MONEY_A/10)%10+1)
	mc_ROOM.mc_MONEY_A._2.gotoAndStop(int(CHR_MONEY_A/100)%10+1)
	mc_ROOM.mc_MONEY_A._3.gotoAndStop(int(CHR_MONEY_A/1000)%10+1)
	mc_ROOM.mc_MONEY_A._4.gotoAndStop(int(CHR_MONEY_A/10000)%10+1)
	mc_ROOM.mc_MONEY_A._5.gotoAndStop(int(CHR_MONEY_A/100000)%10+1)
	
	mc_ROOM.mc_MONEY_B._0.gotoAndStop(CHR_MONEY_B%10+1)
	mc_ROOM.mc_MONEY_B._1.gotoAndStop(int(CHR_MONEY_B/10)%10+1)
	mc_ROOM.mc_MONEY_B._2.gotoAndStop(int(CHR_MONEY_B/100)%10+1)
	mc_ROOM.mc_MONEY_B._3.gotoAndStop(int(CHR_MONEY_B/1000)%10+1)
	mc_ROOM.mc_MONEY_B._4.gotoAndStop(int(CHR_MONEY_B/10000)%10+1)
	mc_ROOM.mc_MONEY_B._5.gotoAndStop(int(CHR_MONEY_B/100000)%10+1)
	
	mc_SHOP.mc_MONEY_C._0.gotoAndStop(CHR_MONEY_A%10+1)
	mc_SHOP.mc_MONEY_C._1.gotoAndStop(int(CHR_MONEY_A/10)%10+1)
	mc_SHOP.mc_MONEY_C._2.gotoAndStop(int(CHR_MONEY_A/100)%10+1)
	mc_SHOP.mc_MONEY_C._3.gotoAndStop(int(CHR_MONEY_A/1000)%10+1)
	mc_SHOP.mc_MONEY_C._4.gotoAndStop(int(CHR_MONEY_A/10000)%10+1)
	mc_SHOP.mc_MONEY_C._5.gotoAndStop(int(CHR_MONEY_A/100000)%10+1)
	
	mc_SHOP.mc_MONEY_D._0.gotoAndStop(CHR_MONEY_B%10+1)
	mc_SHOP.mc_MONEY_D._1.gotoAndStop(int(CHR_MONEY_B/10)%10+1)
	mc_SHOP.mc_MONEY_D._2.gotoAndStop(int(CHR_MONEY_B/100)%10+1)
	mc_SHOP.mc_MONEY_D._3.gotoAndStop(int(CHR_MONEY_B/1000)%10+1)
	mc_SHOP.mc_MONEY_D._4.gotoAndStop(int(CHR_MONEY_B/10000)%10+1)
	mc_SHOP.mc_MONEY_D._5.gotoAndStop(int(CHR_MONEY_B/100000)%10+1)
	
	mc_ROOM.mc_MONEY_A.play()
	mc_ROOM.mc_MONEY_B.play()
	mc_SHOP.mc_MONEY_C.play()
	mc_SHOP.mc_MONEY_D.play()
}

    Магия чисел

    kyzi007, 11 Ноября 2011

    Комментарии (28)