1. 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)
  2. 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)
  3. 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)
  4. 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)
  5. 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)
  6. 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)
  7. 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)
  8. 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)
  9. 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)
  10. ActionScript / Говнокод #8484

    −112

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    function READ_MAIL() {
    	if (mc_MESS._currentframe==1) {
    		if (mails_body.substr((mails_num - 1) * 9, 1) == "1") {// sys mail
    			GET_SYS_MAIL(mails_body.substr((mails_num - 1) * 9 + 1, 8))
    		}else if (mails_body.substr((mails_num - 1) * 9, 1) == "2") {// user mail
    			GET_USER_MAIL(mails_body.substr((mails_num - 1) * 9 + 1, 8))
    		}else if (mails_body.substr((mails_num - 1) * 9, 1) == "5") {//present
    			GET_PRESENT_MAIL(mails_body.substr((mails_num - 1) * 9 + 1, 8))
    		}
    	}
    }

    Читаем данные...

    kyzi007, 11 Ноября 2011

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