PHP — Говнокод.ру

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

+163

1
2
3
4
5
6
7

$birthDay = $elements['birthDay']->getValue();

$d = $birthDay[0].$birthDay[1];
$m = $birthDay[3].$birthDay[4];
$y = $birthDay[6].$birthDay[7].$birthDay[8].$birthDay[9];

$birthDay = mktime(0, 0, 0, (int)$m, (int)$d, (int)$y);

Вот так вот в ПХП 5.2 конвертим дату из dd.mm.yyyy в timestamp

paranoid, 05 Апреля 2010

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

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

+166.2

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

<?
$fileSelect = array();
$sql  = 'SELECT * FROM bd_file';
$stmt = db_prepare($sql, $file); $res  = db_query($stmt);
while($row = db_fetch_row(res))
{
	if (!is_file(filefolder.'/'.$row[название_файла]))) {
		$fileSelect[$file] = $file;
	}
}
unset($fileSelect);
?>

В добавок к #2929.
Этим образом идет проверка существование документов в панели администратора

MoLe-X, 05 Апреля 2010

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

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

+162.8

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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67

<?php 
   function db_connect($host, $user, $pass, $db) {
       global $glink;
       $glink = mysql_connect($host, $user, $pass) or db_give_error(3, '', db_error($glink));
        mysql_select_db($db, $glink) or db_give_error(4, '', db_error($glink));
		mysql_query("SET CHARSET cp1251",$glink);
       return $glink;
   }
   
   function db_query($sql, $log = 0) {
       global $glink;
       $res = mysql_query($sql, $glink) or db_give_error(5, $sql, db_error($glink));
       return $res;
   }
            
   function db_fetch_row($res) {
       $row = empty($res) ? false : mysql_fetch_row($res);
       return $row;
   }
   
   function db_num_rows($res, $stmt = '') {
       $rows = mysql_num_rows($res); 
       if (null === $rows) db_give_error(6, '', db_error($glink));
       return $rows;
   }
  
   function db_num_fields($res, $stmt = '') {
       $fields = empty($res) ? false : mysql_num_fields($res);
       if (null === $fields) db_give_error(7, '', db_error($glink));
       return $fields;
   }
    
   function db_field_name($res, $fld) {
       $name = (empty($res) or !isset($fld)) ? false : mysql_field_name($res, $fld);
       if (false === $name) db_give_error(8, '', db_error($glink));
       return $name;
   }
   
   function db_affected_rows($res) {
       global $glink;
       $rows = mysql_affected_rows($glink); 
       if (null === $rows) db_give_error(9, '', db_error($glink));
       return $rows;
   }

   function db_insert_id($id = 0) {
       $id = mysql_insert_id();
       return ($id) ? $id : 1;
   }
   
   function db_next_id($table, $field, $seqname = 'seq') {
       return 0;
   }
   
   function db_errno($res) {
       return mysql_errno($res);
   }
   
   function db_error($res) {
       return mysql_error();
   }
   
   function db_free_result($res) {
       $res = mysql_free_result($res);
       return $res;
   }
?>

Все из того же проекта (http://govnokod.ru/2929). авторам видимо надоело писать mysql_ и они решили сделать свой аналог db_. При чем смысла от этого полный NULL. Говнокод, он и в Африке говнокод

MoLe-X, 05 Апреля 2010

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

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

+165

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

$fileSelect = array();
if (is_dir($filefolder)) {
    if ($dh = @opendir($filefolder)) {
        while (($file = readdir($dh)) !== false) {
            if (filetype($filefolder . $file) == "file" and $file != "0.txt") {
                $sql  = 'SELECT * FROM bd_file WHERE file_file = ?';
                $stmt = db_prepare($sql, $file); $res  = db_query($stmt);
                if (!db_num_rows($res)) {
                    $fileSelect[$file] = $file;
                }
            }
        }
    }
    closedir($dh);
}
...
unset($fileSelect);

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

P.S. минимальная стоимость сайта у этой фирмы 3.000.000 рублей

MoLe-X, 05 Апреля 2010

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

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

+159.2

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

<?php 
include( "dbopen.php" ); 

function ShowTree($ParentID, $lvl) { 

global $link; 
global $lvl; 
$lvl++; 

$sSQL="SELECT id,title,pid FROM catalogue WHERE pid=".$ParentID." ORDER BY title";
$result=mysql_query($sSQL, $link);

if (mysql_num_rows($result) > 0) {
echo("<UL>\n");
while ( $row = mysql_fetch_array($result) ) {
$ID1 = $row["id"];
echo("<LI>\n");
echo("<A HREF=\""."?ID=".$ID1."\">".$row["title"]."</A>"."  \n");
ShowTree($ID1, $lvl); 
$lvl--;
}
echo("</UL>\n");
}

}

ShowTree(0, 0); 

mysql_close($link); 

?>

Построение дерева с запросом в цикле:)))

http://www.codenet.ru/webmast/php/tree.php

ar4ibal, 05 Апреля 2010

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

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

+167.8

1
2
3

if($_GET['admin']=='da'){
	echo 'adminka_true';
}

А самое страшное, что писал это я...

Armanio, 03 Апреля 2010

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

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

+165

1

$result = mysql_query("SELECT `id` FROM `cats` WHERE `id` = (SELECT `pid` FROM `tovary` WHERE `id` = '".$this->id."' LIMIT 1)");

В один запрос получать id не кошерно.

Было найдено в этом же проекте - http://www.govnokod.ru/2897

Rpsl, 02 Апреля 2010

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

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

+166

1

$limit = (isset($this->params['pass'][1]) == 'news')?null:5;

балина, и это работает :D

lener, 02 Апреля 2010

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

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

+162.6

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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78

<?php


if($_GET['id'] == 10 ){

$location = "/articles/lyustry/lustra-na-kuhne/";
}elseif($_GET['id'] == 13 and $_GET['id_type'] == 17){
$location = "/market/interernye_svetilniki/lyustry/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 22 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bolshie_lyustry/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 34 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/lyustry/bolee_6_rozhkov/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 2 ){
$location = "/market/interernye_svetilniki/podvesnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 40 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/podvesnye/odinochnyj_podves/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 41 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/podvesnye/trojnoj_i_bolee/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 5 ){
$location = "/market/interernye_svetilniki/nastenno-potolochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 35 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/potolochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 52 ){
$location = "/market/interernye_svetilniki/vlagozawitnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 51 ){
$location = "/market/interernye_svetilniki/podsvetka_kartin/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 7 ){
$location = "/market/interernye_svetilniki/bra/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 37 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/odnorozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 38 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/dvuhrozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 39 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/treh_i_bolee_rozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 1 ){
$location = "/market/interernye_svetilniki/tochechnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 42 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/odinochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 43 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/dvojnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 44 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/tri_i_bolee/";
}
elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 8){
$location = "/market/interernye_svetilniki/nastolnye_lampy/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 46 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/nastolnye_lampy/ofisnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 47 and $_GET['act'] == 'sub_
type'){
$location = "/market/interernye_svetilniki/nastolnye_lampy/prikrovatnye_domashnie_ili_interernye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 6 ){
$location = "/market/interernye_svetilniki/torshery/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 9 ){
$location = "/market/ulichnye_svetilniki/";
}...
И так далее..

Клиент поставил нашу CMS, на старой были ссылки типа /?id=10 и т.д. Сделал ему для примера скрипт редиректа с использованием switch case и вот что я увидел через две недели 0_0 Неприкрытый говнокод.

VeroLom, 01 Апреля 2010

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

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

+159.4

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

//возвращает массив без повторяющихся элементов (обязательно должен быть элемент [id])
function arrayUnique($arr)
	{	
		$aResult = array();
		$aIDs = array();
		foreach($arr as $val)
			if (!in_array($val['id'], $aIDs))
			{
				array_push($aIDs, $val['id']);
				array_push($aResult, $val);
			}
		return $aResult;
	}

На входе массив массивов.
Массивы равны, если все элементы одного массива равны элементам другого.
На выходе должен быть массив уникальных массивов.
По моему это попахивает.

nechin, 31 Марта 2010

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

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

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

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

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

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

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

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

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

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

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

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