По колено в коде — Говнокод.ру

Ruby / Говнокод #11684

−157

1
2
3
4
5
6
7
8

class SomeModel < ActiveRecord::Base
  after_save :some_method

  def some_method
    self.save
  end

end

Рекурсивная рекурсия рекурсивна.

mr.The, 31 Августа 2012

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

PHP / Говнокод #11683

+57

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16

function GetOrderSkidka(&$arrData)
    	{
	    	if($this->flag_opt){
	    	 	$arrData['skidka']  = ($this->admin_mode) ? $arrData['skidka'] : 0;
	    	 	$arrData['allsum']  = $arrData['sum'] - $arrData['skidka'];
	    	 	return;
	        }
	    	if(!$this->flag_in_action){
	    		$arrData['cnt_s_prod'] = $arrData['cnt'];
	    	}elseif(in_array($this->flag_action_type,array(2,3))){
	    		$this->calcCntProd($arrData);
	    	}else{
	    		$arrData['cnt_s_prod'] = 0;
	    	}
	    	$this->discount->GetOrderSkidka($arrData);
    	}

Работаю с сайтом, в котором все методы классов работают со своими параметрами таким образом.
Метод может ничего не возвращать, а вызывать другие методы (которые также могут вызывать какие-то методы),
которые в зависимости от множества условий меняют переданные по ссылке параметры.
Итог работы модифицированный параметр- массив. Только XDebug выручает.

minramilka, 31 Августа 2012

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

Java / Говнокод #11682

+121

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22

try {
	repository.saveAll(entities);
} catch (final ConstraintViolationException e) {
	throw new SettingsServiceException(ErrorFormatter.formatValidationErrors(
			"Errors have been detected when saving", e));
} catch (final PersistenceException e) {
	for (val t: Throwables.getCausalChain(e)) {
		if (t instanceof EntityExistsException) {
			val failedObject = (IEntity) ((ExceptionInfo) t).getFailedObject();
			
			// failedObject is returned with the state in which it is in the database.
			// Let's find the version that we tried to save, instead.
			for (val entity: entities) {
				if (entity.getId() != null && entity.getId().equals(failedObject.getId())) {
					throw new SettingsServiceException("Attempted to save duplicate value: " + entity);
				}
			}
		}
	}
	
	throw e;
}

Попытка вернуть пользователю человеко-читаемое сообщение о том, где, собственно, дубликат.

Костыли вы мои, костыли...

someone, 31 Августа 2012

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

PHP / Говнокод #11681

+54

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40

<?php

class SimpleController extends Zend_Controller_Action
{
    public function init()
    {
        $action = $this->_request->getActionName();
        $is_action_available = in_array($action, explode(',', 'terms,privacy,about,copyright,support,legal,features,help,refund'));
        $is_incorrect_action = CR_Settings::siteIsFree() && $action == 'refund';
        if (!$is_action_available || $is_incorrect_action)
            $this->_helper->redirector->setGotoRoute(array('action' => 'index'), 'default', true);
    }
    
    public function termsAction(){
    }
    
    public function privacyAction(){
    }
    
    public function aboutAction(){
    }
    
    public function copyrightAction(){
    }
    
    public function supportAction(){
    }
    
    public function legalAction(){
    }
    
    public function featuresAction(){
    }
    
    public function helpAction(){
    }
    
    public function refundAction(){
    }
}

Zend Framework проверка на существование action :)

arkasha, 31 Августа 2012

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

C++ / Говнокод #11680

+26

01
02
03
04
05
06
07
08
09
10

result.push_back(TVector<2>
				(
					(A-√(D))/C,
					(E-Line.K()*√(D))/C
				));
				result.push_back(TVector<2>
				(
					(A+√(D))/C,
					(E+Line.K()*√(D))/C
				));

LispGovno, 31 Августа 2012

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

Куча / Говнокод #11679

+131

1
2
3
4
5

if (http_status_code == 200 || http_status_code == 500) {
  .... //success!!
} else {
  .... // error!!
}

Было замечено в одном андроид приложении.

mr.The, 30 Августа 2012

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

C# / Говнокод #11678

+141

1
2
3
4
5

var number = new Random();
if (number.Next(10)%2  == 0)
{...}
else
{...}

Нашёл такой шедевр в индокоде, которые мне дали модернизировать.

badstarosta, 30 Августа 2012

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

PHP / Говнокод #11677

+57

1
2
3
4
5
6
7

function SetImage($row=array())
	{
     foreach($row as $property=>$value)
     	{
     	eval("\$this->".$property." = '".$value."';");
     	}
	}

minramilka, 30 Августа 2012

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

C# / Говнокод #11676

+107

1
2
3
4
5
6
7
8

if (_appliesTo.Contains("CH"))
{
    depElig &= true;
}
else
{
    depElig &= false;
}

Вроде такого boolshit тут ещё не было.

roman-kashitsyn, 30 Августа 2012