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

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

+159

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

if (!isset($_GET['outbox']) and !isset($_GET['deleted']) and !isset($_GET['verification'])){
				$res2 = mysql_query("SELECT l.* FROM letters l LEFT JOIN `customers` ct ON ct.id=l.from_user WHERE l.deleted<>1 AND ct.`Sex` = 'male' {$addon_sql} ORDER BY l.time_insert DESC");
				for($arr=array(); $t = mysql_fetch_assoc($res2); $arr[]=$t);

				smarty_put_var('zagolovok', 'Входящие письма');
		}

		if (isset($_GET['verification'])) {
		$res2 = mysql_query("SELECT * FROM `letters` where `deleted` != 1 AND `checked` = 'N' {$addon_sql} order by `time_insert` DESC");
		for($arr = array(); $t = mysql_fetch_assoc($res2); $arr[] = $t);

		smarty_put_var('zagolovok', 'Ожидающие проверку');
		}

		if (isset($_GET['outbox'])) {
    	$res2 = mysql_query("SELECT l.* FROM `letters` l
		    	LEFT JOIN `customers` ct ON ct.`id`=l.from_user
		    	WHERE l.deleted != 1 AND ct.`Sex` = 'female' {$addon_sql}
		    	ORDER BY l.time_insert DESC LIMIT 200");
		    	for ($arr = array(); $t = mysql_fetch_assoc($res2); $arr[]=$t);

		    	smarty_put_var('zagolovok', 'Исходящие письма');
		}

		    	if (isset($_GET['deleted'])){
    	$res2 = mysql_query("SELECT * FROM `letters` WHERE `deleted` = 1
		    	{$addon_sql} ORDER BY `time_insert` DESC");
		    	for($arr=array(); $t=mysql_fetch_assoc($res2); $arr[]=$t);

		    	smarty_put_var('deleted', true);
		    	smarty_put_var('zagolovok', 'Удаленные письма');
		    	}

		    	$letters = array_slice($arr, $first, $limit);

		foreach ($letters as &$letter) {
			$letter['from_user_details'] = Model_Table_Customers::model()
				->fetchByPk($letter['from_user']);
			$letter['to_user_details'] = Model_Table_Customers::model()
				->fetchByPk($letter['to_user']);
		}

if (isset($_GET['outbox'])){smarty_put_var('pages', pages_admin("?module=admin_letters&outbox&page=", $arr, $page, $limit)); smarty_put_var('type', 'outbox');}
elseif (isset($_GET['verification'])){smarty_put_var('pages', pages_admin("?module=admin_letters&verification&page=", $arr, $page, $limit)); smarty_put_var('type', 'verification');}
elseif (isset($_GET['deleted'])){smarty_put_var('pages', pages_admin("?module=admin_letters&deleted&page=", $arr, $page, $limit)); smarty_put_var('type', 'deleted');}
else {smarty_put_var('pages', pages_admin("?module=admin_letters&page=", $arr, $page, $limit)); smarty_put_var('type', '');}

...

function pages_admin($link, $arr, $page, $vis)
{
    $radius=4;
    $pages=floor(count($arr)/$vis);
    $return="";
    $str='<table cellpadding="0" cellspacing=3 border="0">
    <tr><td height=7></td></tr>
    <tr>';

    if ($pages > 1) {
        $first = 'First';
        $prev = 'Prev';
        $next = 'Next';
        $last = 'Last';

        if($page<$radius*2){
           .....
      трэшак в том же духе

Очередной шедевр с сохранением авторского стиля и табуляции. Это подготовка для smarty представления таблицы с постраничной разбивкой. Возможны 4 типа писем - 4 вкладки. pages_admin() - возвращает пагинатор в виде готового html кода.

Клиент жаловался, что страница сильно тормозит. Там более 150 000 писем. Я догадывался, что код можно чутка оптимизирвоать, но не думал, что существенно. А когда посмотрел вижу - п-ц ))) Интересно насколько получить ускорить?

stechkh, 01 Марта 2015

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

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

+159

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

public function get_from_text($text){ 
      foreach($text as $x){ 
        $a = array('A', '?', 'A', '?', '?', '?', '?', 'C', 'E', 'E', 'E', 'E', '?', '?', 'I', 'I', '?', '?', '?', '?', 'O', '?', '?', '?', 'U', '?', 'U', 'U', '?', '?', 'à', '?', 'â', '?', '?', '?', '?', 'ç', 'è', 'é', 'ê', 'ë', '?', '?', 'î', 'ï', '?', '?', '?', 'ô', '?', '?', '?', 'ù', '?', 'û', 'ü', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', 'Œ', 'œ', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', 'ƒ', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?', '?'); 
        $b = array('A', 'A', 'A', 'A', 'A', 'A', 'AE', 'C', 'E', 'E', 'E', 'E', 'I', 'I', 'I', 'I', 'D', 'N', 'O', 'O', 'O', 'O', 'O', 'O', 'U', 'U', 'U', 'U', 'Y', 's', 'a', 'a', 'a', 'a', 'a', 'a', 'ae', 'c', 'e', 'e', 'e', 'e', 'i', 'i', 'i', 'i', 'n', 'o', 'o', 'o', 'o', 'o', 'o', 'u', 'u', 'u', 'u', 'y', 'y', 'A', 'a', 'A', 'a', 'A', 'a', 'C', 'c', 'C', 'c', 'C', 'c', 'C', 'c', 'D', 'd', 'D', 'd', 'E', 'e', 'E', 'e', 'E', 'e', 'E', 'e', 'E', 'e', 'G', 'g', 'G', 'g', 'G', 'g', 'G', 'g', 'H', 'h', 'H', 'h', 'I', 'i', 'I', 'i', 'I', 'i', 'I', 'i', 'I', 'i', 'IJ', 'ij', 'J', 'j', 'K', 'k', 'L', 'l', 'L', 'l', 'L', 'l', 'L', 'l', 'l', 'l', 'N', 'n', 'N', 'n', 'N', 'n', 'n', 'O', 'o', 'O', 'o', 'O', 'o', 'OE', 'oe', 'R', 'r', 'R', 'r', 'R', 'r', 'S', 's', 'S', 's', 'S', 's', 'S', 's', 'T', 't', 'T', 't', 'T', 't', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'W', 'w', 'Y', 'y', 'Y', 'Z', 'z', 'Z', 'z', 'Z', 'z', 's', 'f', 'O', 'o', 'U', 'u', 'A', 'a', 'I', 'i', 'O', 'o', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'U', 'u', 'A', 'a', 'AE', 'ae', 'O', 'o'); 
        $x= str_replace($a, $b, $x); 
        $x= str_replace("!", "", $x);$x= str_replace("@", "", $x);$x= str_replace("#", "", $x); 
        $x= str_replace("$", "", $x);$x= str_replace("%", "", $x);$x= str_replace("^", "", $x); 
        $x= str_replace("&", "", $x);$x= str_replace("*", "", $x);$x= str_replace("(", "", $x); 
        $x= str_replace(")", "", $x);$x= str_replace("-", "", $x);$x= str_replace("_", "", $x); 
        $x= str_replace("=", "", $x);$x= str_replace("+", "", $x);$x= str_replace("+", "", $x); 
        $x= str_replace("nbsp;", "", $x);$x= str_replace(";", "", $x);$x= str_replace(".", "", $x); 
        $x= str_replace("{", "", $x);$x= str_replace("[", "", $x);$x= str_replace("]", "", $x); 
        $x= str_replace("}", "", $x);$x= str_replace(":", "", $x);$x= str_replace("'", "", $x); 
        $x= str_replace("\"", "", $x);$x= str_replace(",", "", $x);$x= str_replace("<", "", $x); 
        $x= str_replace("\\", "", $x);$x= str_replace("?", "", $x);$x= str_replace(">", "", $x); 
        $x= str_replace("/0-9/", "", $x);$x= str_replace("?", "", $x);$x= str_replace(">", "", $x); 
        $x=trim(str_replace(range(0,9),'',$x));$x=explode(" ",$x); 
                $vok[]=$x; 
      } 
      return $vok; 
}

http://www.phpclasses.org/package/8932-PHP-Filter-text-to-remove-non-SEO-friendly-characters.html#view_files/files/57793

shilgen, 01 Марта 2015

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

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

+158

1
2

$arr = mysql_fetch_assoc(mysql_query("select * from customers where Email1='".$_POST['email']."'"));
if ($arr and count($arr)!=0) {

Проверка, что массив не пустой

stechkh, 28 Февраля 2015

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

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

+159

1

if (strlen($PARAM)<=0) $PARAM = " class=\"inputselect\" ";

Если длинна строки меньше нуля? Битрикс, ты серьёзно?

Happinness_Cookie, 27 Февраля 2015

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

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

+156

01
02
03
04
05
06
07
08
09
10
11

function GetDataArray($qs)
{
    $res = array();
    $q = mysql_query($qs);
    if ($q) {
        while ($row  = mysql_fetch_array($q)) {
            array_push($res,$row);
        }
    }
    return $res;
}

Других путей получить массив записей наверно для автора нет

stechkh, 26 Февраля 2015

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

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

+158

1
2
3
4
5
6
7
8
9

<?php if( !Yii::app()->user->checkAccess('Manager')){ ?><td class="text-center"><?php  if($project->c1Projects->stat_id==23){
                echo '|';
                }elseif($project->c1Projects->stat_id==496){
                    echo '||';
                }else{
                   echo  '-';
                }

    ?></td><?php } ?>

Вюха одного проекта

Elenberg, 24 Февраля 2015

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

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

+162

1
2
3
4

taras@host ~/host/sites/default $ drush phpsh-console
php> var_dump(eval('array()'));
bool(false)
PHP Parse error:  syntax error, unexpected end of file in /usr/local/lib/python2.7/dist-packages/phpsh/phpsh.php(591) : eval()'d code(1) : eval()'d code on line 1

О мой мозг...! А Python здесь причем?

kissarat, 24 Февраля 2015

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

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

+161

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

there is a reason why opencart is the no.1 most used ecommerce solution in places like china and india, its the easiest code base to understand!

--

many apps servers! what does that mean? you mean different applications running from the same framework?
you build each application starting from the index.php file and include what ever library classes you require.

--

"I agree with you that it's harder to write simple code, because REPEATING CODE IS HARD TO DEBUG HARD TO READ AND TO CORRECT. so it makes you waste a lot of time."

this is what search and replace is for!

Создатель опенкарта (Daniel Kerr) исходит на говно, много мякотки
http://www.techchattr.com/never-use-opencart#comment-1151857248

Fike, 22 Февраля 2015

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

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

+160

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

foreach (scandir(DIR.'app'.SLASH.'lib') as $filename) {
    $path = DIR.'app'.SLASH.'lib'.SLASH.$filename;
    if (is_file($path) && substr($filename,strlen($filename)-4,4)=='.php') {
        require_once($path);
    }
}
foreach (scandir(DIR.'app'.SLASH.'models') as $filename) {
    $path = DIR.'app'.SLASH.'models'.SLASH.$filename;
    if (is_file($path) && substr($filename,strlen($filename)-4,4)=='.php') {
        require_once($path);
    }
}
foreach (scandir(DIR.'app'.SLASH.'ext') as $filename) {
    $path = DIR.'app'.SLASH.'ext'.SLASH.$filename;
    if (is_file($path) && substr($filename,strlen($filename)-4,4)=='.php') {
        require_once($path);
    }
}

Начался разбор проблемы со слов заказчика - "сайт долго грузится"

Crawdaunt, 22 Февраля 2015

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

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

+157

1
2
3
4
5
6

function getMinQueueOrdering()
  {
    $sql="SELECT MAX(ordering)
          FROM priceloaddata_queue";
  .........
  }

нет слов.

Vasiliy, 19 Февраля 2015

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

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

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

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

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

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

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

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

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

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

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

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