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

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

+56

1
2
3
4

function goto($url, $time = 0)
{
	exit('<META HTTP-EQUIV="REFRESH" CONTENT="'.$time.';URL='.$url.'">');
}

kindofbear, 21 Декабря 2012

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

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

+43

1
2
3
4
5
6

function Podergatsya($i)
{
	$i++;
	$i--;
	return $i;
}

Индусская CMS

kindofbear, 21 Декабря 2012

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

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

+68

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
41
42
43
44
45
46
47

/*
    * Todo Гавнокод
    * Надо как то поуниверсальнее сделать
    * А то когда модель вмешавается в отображение, это криворукость
    */
    private static String ___recurGet(Dir dir){
        MorphiaQuery dirs = dir.getDirList();
        String  id = dir.getIdAsStr(),
                name = dir.name;

        String html = "<div class=\"bottom-line\">";
        html += "<a href=\"#\" onclick=\"setPrnt('"+id+"')\" id=\""+id+"\">";
        html += name;
        html += "</a>";
        html += "<div class=\"pull-right\">";
        html += "<a href=\"#\" class=\"icon-trash\" onclick=\"deleteNode('"+id+"')\"></a>";
        html += "</div>";
        html += "</div>";

        if(dirs.count() < 1)
            return html;

        html += "<ul>";
        for(int i = 0; i < dirs.count(); i++){
            html += "<li>";
            html += ___recurGet((Dir)dirs.offset(i).first());
            html += "</li>";
        }
        html += "</ul>";
        return html;
    }

    public static String getTreeAsHtml(String _typ){
        MorphiaQuery dirs = getRootDirList().filter("typ",Type.valueOf(_typ));

        if(dirs.count() < 1)
            return "Категорий нет";

        String html = "<ul id=\"treeView\">";
        for(int i = 0; i < dirs.count(); i++){
            html += "<li>";
            html += ___recurGet((Dir)dirs.offset(i).first());
            html += "</li>";
        }
        html += "</ul>";
        return html;
    }

В модели "категория" написал метод для получения дерева категории в виде html кода...
Причем такая генерация должна происходить в view, а сделал как обычно через ж

haker, 21 Декабря 2012

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

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

+68

01
02
03
04
05
06
07
08
09
10

public Dir delete(){
        MorphiaQuery dirs = getDirList(); //Получаем подпапки
        if(dirs != null){ 
            Iterator<Dir> list = dirs.iterator();  
            while(list.hasNext()){
                list.next().delete(); //Снова вызываем public Dir delete()
            }
        }
        return super.delete(); //Удаляем саму папку
    }

Написал метод для удаления категорий рекурсивно из базы...
Представляю какая нагрузка будет на базу если будет 100 папок, а в ней каждой ещё по 100. В итоге 100*100 = 10000 запросов в базу

haker, 21 Декабря 2012

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

1C / Говнокод #12325

−122

1
2
3

Word.FormFields("Data").Result  = СокрЛП(?(ДатаЧисло(ДатаДок2)<10,"0","")+ДатаЧисло(ДатаДок2)); 
Word.FormFields("Manth").Result = СокрЛП(МесяцПрописью(ДатаДок2)); 
Word.FormFields("God").Result   = СокрЛП(ДатаГод(ДатаДок2));

взято из конфигурации одного автодилера, особенно порадовали имена частей даты

VladC, 21 Декабря 2012

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

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

+70

1
2
3
4
5
6
7

if (!formules.isEmpty()) {
    for (String tag : formules) {
    .....
    // и вот здесь шедевр:
    }
    // а здесь ничего нет
}

Не гоняем цикл вхолостую

alopen, 21 Декабря 2012

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

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

+46

1
2
3
4
5
6
7

$week['monday'] = date("d.m.Y", $week_start);
$week['tuesday'] = date("d.m.Y", $week_start + 1 * 86400);
$week['wednesday'] = date("d.m.Y", $week_start + 2 * 86400);
$week['thursday'] = date("d.m.Y", $week_start + 3 * 86400);
$week['friday'] = date("d.m.Y", $week_start + 4 * 86400);
$week['saturday'] = date("d.m.Y", $week_start + 5 * 86400);
$week['sunday'] = date("d.m.Y", $week_start + 6 * 86400);

1allen, 20 Декабря 2012

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

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

+132

1

public virtual int ReadByte()

Тут в соседнем треде появилась такая тема:

http://msdn.microsoft.com/ru-ru/library/system.io.stream.readbyte.aspx

http://govnokod.ru/12311#comment164854

LispGovno, 20 Декабря 2012

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

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

+135

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

public static string GetCommandLineParameter(string ParameterName)
    {
      ParameterName = ParameterName.ToLower();
      string ParameterIdentifikator = ParameterName.ToLower() + "=";
      
      string RetVal = null;
      foreach(string Arg in Environment.GetCommandLineArgs())
      {
        string ArgLower = Arg.ToLower();
        if(ArgLower.IndexOf(ParameterIdentifikator) == 0)
        {
          RetVal = Arg.Substring(ParameterIdentifikator.Length, Arg.Length - ParameterIdentifikator.Length);
          return RetVal;
        }
      }
      return RetVal;
    }

читаем параметры из командной строки

taburetka, 19 Декабря 2012

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

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

+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
41
42
43
44
45
46
47
48
49

function validEmail($email)
{
	$isValid = true;
	$atIndex = strrpos($email, "@");
	if (is_bool($atIndex) && !$atIndex) {
		$isValid = false;
	} else {
		$domain = substr($email, $atIndex + 1);
		$local = substr($email, 0, $atIndex);
		$localLen = strlen($local);
		$domainLen = strlen($domain);
		if ($localLen < 1 || $localLen > 64) {
			// local part length exceeded
			$isValid = false;
		} else if ($domainLen < 1 || $domainLen > 255) {
			// domain part length exceeded
			$isValid = false;
		} else if (!strrpos($domain, ".")) {
			// domain part does not have .
			$isValid = false;
		} else if ($domain[0] == '.' || $domain[$domainLen - 1] == '.') {
			// domain part starts or ends with '.'
			$isValid = false;
		} else if ($local[0] == '.' || $local[$localLen - 1] == '.') {
			// local part starts or ends with '.'
			$isValid = false;
		} else if (preg_match('/\\.\\./', $local)) {
			// local part has two consecutive dots
			$isValid = false;
		} else if (!preg_match('/^[A-Za-z0-9\\-\\.]+$/', $domain)) {
			// character not valid in domain part
			$isValid = false;
		} else if (preg_match('/\\.\\./', $domain)) {
			// domain part has two consecutive dots
			$isValid = false;
		} else if (!preg_match('/^(\\\\.|[A-Za-z0-9!#%&`_=\\/$\'*+?^{}|~.-])+$/', str_replace("\\\\", "", $local))) {
			// character not valid in local part unless 
			// local part is quoted
			if (!preg_match('/^"(\\\\"|[^"])+"$/', str_replace("\\\\", "", $local))) {
				$isValid = false;
			}
		}
		/* if ($isValid && !(checkdnsrr($domain,"MX") || checkdnsrr($domain,"A"))) {
		  // domain not found in DNS
		  $isValid = false;
		  } */
	}
	return $isValid;
}

Интересно а автор когдата слишал про filter_var?
Это все можна заменить на filter_var($input, FILTER_VALIDATE_EMAIL);
Товаришь программист! Не умеешь срать - не мучай жопу!

v_anonym, 19 Декабря 2012