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

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

+161

1

$period = in_array($bullRegionId,array_keys($periods)) ? $periods[$bullRegionId]['text'] : $periods['default']['text'];

Интересна сама проверка на существование ключа, не правда ли? :-)
Код мой. Коллега учуял, что от него несколько плохо пахнет и предложил запостить сюда))

Nemoden, 09 Сентября 2011

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

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

+153

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

function resetIsetSorting()
	{
		$db = 'i_set';
		$co = -1;
		$q = $this->db->get($db);
		$r = $q->result();
		foreach($r as $row)
		{
			$co++;
			$upd['sort'] = $co;
			$this->db->where('id_set', $row->id_set); 
			$this->db->update($db, $upd);	
		}
	}
	function isetSortExchange($sort, $canBeSort)
	{
		$db = 'i_set';
		$bigNumber = 1470000;
		$this->db->where('sort', $canBeSort);
		$q = $this->db->get($db);
		if ($q->num_rows() == 1)
		{
			$upd['sort'] = $bigNumber;
			$this->db->where('sort', $canBeSort); 
			$this->db->update($db, $upd);	
			//
			$upd['sort'] = $canBeSort;
			$this->db->where('sort', $sort); 
			$this->db->update($db, $upd);
			//
			$upd['sort'] = $sort;
			$this->db->where('sort', $bigNumber); 
			$this->db->update($db, $upd);
		}
	}

Первая функция призвана обновить поле sort на уникальные последовательные значение. Сначала хотел провернуть такое дело на SQL в PHPmyADMIN, но мозгов моих оказалось мало, и я сделал это на CodeIgniter. Говнокод ли это?

Вторая функция призвана для обмена значениями полей sort. Это нужно для сортировки чего либо в баз. Чутьё мне подсказывает что это можно проделать каким-то белее правильным путём, без использования $bigNumber = 1470000; (а вдруг количество строк достигнет этого числа??!!). Говнокод ли это?

Всё рабочее, только что проверил. Свежий код, только что написан!

increazon, 08 Сентября 2011

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

JavaScript / Говнокод #7812

+161

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

<select onchange="goTo();" id="navSelect">
<option value="/way_to_hell/">Вопросы-ответы</option>
.
.
.
</select>

<script>
function goTo()
{
  var objSel = document.getElementById("navSelect");
  window.location = objSel.options[objSel.selectedIndex].value;
}

</script>



//можно вообще вот так:
<select onchange="window.location = this.options[this.selectedIndex].value">

dr_fet, 08 Сентября 2011

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

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

+162

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

function validemail($email){

 if (!ereg("^[^@]{1,64}@[^@]{1,255}$", $email))

 return false;



 $email_array = explode("@", $email);

 $local_array = explode(".", $email_array[0]);

 for ($i = 0; $i < sizeof($local_array); $i++) {

  if (!ereg("^(([A-Za-z0-9!#$%&'*+/=?^_`{|}~-][A-Za-z0-9!#$%&'*+/=?^_`{|}~\.-]{0,63})|(\"[^(\\|\")]{0,62}\"))$", $local_array[$i]))

  return false;

 }

jQuery, 08 Сентября 2011

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

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

+173

1

define("RANDOM_NUM", rand(0, 10000000));

поясню: кодер думал, что каждый раз будет вызываться функция и генерить новое число, т.к. в коде далее идет ее использование как уникального идентификатора. Скорее всего, это один из первых скриптов того, кто писал.

jokz, 08 Сентября 2011

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

JavaScript / Говнокод #7809

+176

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

<a id="hide" href ="javascript:
var text = $('#ccc').html();
function hide(){
    $('#ccc').html('');
    $('#hide').attr('href', 'javascript: show(); ');
}
function show(){
    $('#ccc').html(text);
    $('#hide').attr('href', 'javascript: hide(); ');
}
hide();
">Скрыть/Показать</a>

<div id="ccc">скрываемвый текст</div>

Я думаю комментарии излишни. Я считаю что это идеальный говнокод. И отличная замена $('#ccc').toggle();

taral, 08 Сентября 2011

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

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

+161

1
2
3
4
5
6

<?
if($page!='/volga' && $page!='/volga/index.php'){
// Тут всякий код
} if($page=='/volga' or $page=='/volga/index.php'){?>
// И тут тоже	
<?}?>

Tairesh, 08 Сентября 2011

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

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

+66

1
2
3
4
5
6

Integer obj = (Integer)dump.get("size");
        if(obj == null) {
            return;
        }
        int size = obj;
        for(int i=0; i<size; i++) {

Самое странное, что автор явно знает, что такое автобоксинг, но всё равно использовал его коряво.

lucidfox, 08 Сентября 2011

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

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

+77

1
2
3
4
5
6

public abstract class Data {
		// The tone of Commander Riker's voice makes me suspect that
		// he is not serious about finding Ambassador T'Pel charming.
		// My experience suggests that in fact he may mean the exact
		// opposite of what he says. Irony is a form of expression
		// I have not yet been able to master.

lucidfox, 08 Сентября 2011

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

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

+84

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

Query q;
      //для хэширования

      q =
         session.createQuery("select idElType from " + LinkToAtributeValue.class.getName()
         + " where idEl=" + idObject.toString());

      Vector<Integer> vec = new Vector<Integer>(q.list());
      int t;
      for(int i = 0; i < vec.size() - 1; i++) {
         t = vec.get(i);
         for(int j = i + 1; j < vec.size(); j++) {
            if(t == vec.get(j)) {
               vec.remove(j);
            }
         }
      }

"DISTINCT для трусов"
или
"Хорошего кода должно быть много"

maxt, 08 Сентября 2011

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

Говнокод: по колено в коде.

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

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

JavaScript / Говнокод #7812

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

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

JavaScript / Говнокод #7809

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

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

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

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