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

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

+55

1
2
3
4
5
6
7

if($contact == null){
   $msg = JText::_('INSERT_CONTACT_DETAILS');
   if($contact == null){
    $mainframe->redirect('index.php?option=com_xxx&act=Agents',Text::_($msg).$user->name);
   }
   return;
 }

Правильно грят: бережённого бог бережёт :)

virtual_cia, 06 Августа 2012

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

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

+64

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

<?php
/* File: config.php */
$ea_hot_color = "#F06000";
$ea_fulllocation = "1";
$ea_curconverter = "1";
$ea_show_hits = "1";
$ea_show_created_date = "1";
//и т.д.~300 строк

// ============================
/* File: config.class.php */
class EAConf{
/*
...
*/
  function EAConf(){
   require('config.php');
   $allvars=get_defined_vars();
   $names=array_keys($allvars);
   foreach($names as $name){
    if(substr($name,0,2)=="ea") $this->$name=$allvars[$name];
   }
  }

/*
...
*/
}

Какая глубина мысли! Смело, нетривиально!

virtual_cia, 06 Августа 2012

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

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

+58

1
2
3

if( $vv || $vv === '0' ){
    $param = htmlspecialchars( $vv, ENT_QUOTES );
}

htmlspecialchars, видимо, творит с нулями что-то особенное

somnambulism, 06 Августа 2012

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

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

+47

1

$cc = str_replace('`','',str_replace("'","",str_replace('"','',strip_tags($_REQUEST['c_code']))));

Регулярные выражения? Не, не слышал

Tairesh, 06 Августа 2012

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

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

+59

1

if ( (is_file($filename) == true) == true)

MaxSvargal, 04 Августа 2012

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

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

+56

01
02
03
04
05
06
07
08
09
10
11

$cur_url=$_SERVER['REQUEST_URI'];
if ($cur_url=='/') { 
	$cur_url=$_SERVER['REQUEST_URI']; 
	if ($cur_url=='/') { // Точно-точно адрес не равен слэшу
		echo '';// После всех проверок можно с уверенностью вывести пустую строку
	}
}
$cur_url2=$_SERVER['REQUEST_URI'];// И ещё разок
if ($cur_url2!='/') { 
	echo '';
}

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

RedMonkey, 03 Августа 2012

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

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

+65

1
2
3
4
5
6
7

function draw_text() {  
        // ....
        /* remove background color */
        imagecolortransparent($im_text, $bg_color);
        return $im_text;
        imagedestroy($im_text);
}

Функция вывода текста CAPTCHA в modx Evolution.

MaXL, 02 Августа 2012

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

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

+56

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

if(isset($_POST['btnsubmitup']))
		{
			for ($i = "0"; Arr::get($_POST, 'id'.$i, ''); $i++) {
			if (Arr::get($_POST, 'up'.$i, '') == '1') {
				$p1=-1;
	//			$uploaddir = '/img/brands/';
				$a = Arr::get($_POST, 'id'.$i, '');
	//			$p1 = Upload::save($_FILES['photo'.$i], $uploaddir.$a.'.jpg', './', 0777);

					$rand=rand(1000000,9999999);
				$uploaddir = '/img_carpets/collection/';
				$uploaddir2 = 'img_carpets/collection/';
				$p1 = Upload::save($_FILES['file1'.$i], $uploaddir.'ID-'.$rand.'-1.jpg', './', 0777);
				$p2 = Upload::save($_FILES['file2'.$i], $uploaddir.'ID-'.$rand.'-2.jpg', './', 0777);
				$p3 = Upload::save($_FILES['file3'.$i], $uploaddir.'ID-'.$rand.'-3.jpg', './', 0777);
				$p4 = Upload::save($_FILES['file4'.$i], $uploaddir.'ID-'.$rand.'-4.jpg', './', 0777);
	//			if ($p1!="0") { $p1=$rand; }
	//			if ($p2!="0") { $p2=$rand; }
	//			if ($p3!="0") { $p3=$rand; }
	//			if ($p4!="0") { $p4=$rand; }
					$im2=Image::factory($uploaddir2.'back.png');

// -> и так далее

Начал разбирать библиотеку (фреймворк скорее - kohanaframework) одного сайта, дабы сделать нормальную админку
Дошел до процедуры сохранения картинок. Я посмотрел, по какому же алгоритму сохраняются картинки (формирование имени файла)
И опупел!
** $rand=rand(1000000,9999999); **
В базе поле для сохранения имени картинки - не уникально.
Т.е., разраб решил поиграть в рулетку, анука генератор чисел выберет еще раз одно и то же число, и перезапишет картинку у товара. ))))
А оператор админки будет чесать репу - тут же работало а тут и нет )

topilnik, 01 Августа 2012

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

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

+42

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

foreach ($balancedSlots as $cluster) {
                if ($cluster['type']=='cluster'){
                        foreach ($cluster['servers'] as $ip=>$server) {
                                if ($GLOBALS['mode']!='test'){
                                        $GLOBALS['dataVar'][$i] = array();
                                        if (isset($server['snmp_server'])){
                                                $p = preg_split('/,/', $server['snmp_server']);
                                                $GLOBALS['dataVar'][$i]['proc_handle'] = popen(dirname(__FILE__).'/snmp_server.sh '.$p[0].' '.$p[1], 'r');
                                                $GLOBALS['dataVar'][$i]['snmp_params'] = $p;
                                                $GLOBALS['dataVar'][$i]['snmp_result'] = array('ifName'=>array(), 'counter'=>array());
                                        }
                                        else {
                                                //print "proc_handle: ".$server['snmp_cmd']."\n";
                                                $GLOBALS['dataVar'][$i]['proc_handle'] = popen($server['snmp_cmd'], 'r');
                                        }
                                        $GLOBALS['dataVar'][$i]['server']=$server;
                                        $GLOBALS['dataVar'][$i]['server']['ip']=$ip;
                                        $GLOBALS['dataVar'][$i]['eb'] = event_buffer_new($GLOBALS['dataVar'][$i]['proc_handle'], "read_result", NULL, "error_func", $i);
                                        event_buffer_timeout_set($GLOBALS['dataVar'][$i]['eb'], 4, 60);
                                        event_buffer_base_set($GLOBALS['dataVar'][$i]['eb'], $GLOBALS['base']);
                                        event_buffer_enable($GLOBALS['dataVar'][$i]['eb'], EV_READ);
                                }
                                elseif ($GLOBALS['mode']=='test')
                                {
                                        if (!array_key_exists($ip, $testLastServers)) {
                                                $testLastServers[$ip]=0;
                                        }
                                        if ($testServersData[$ip][$testServersIteration]!==false){
                                                $GLOBALS['result'][$ip] = new ArrayObject(array('counter64'=>$testLastServers[$ip]+$testServersData[$ip][$testServersIteration], 'ts'=>$GLOBALS['mesure_time']));
                                                $testLastServers[$ip] = $GLOBALS['result'][$ip]['counter64'];
                                        }
                                }
                                $i++;
                        }
                }
        }

Попытка одного товарища использовать библиотеку libevent. Видно щедрое использование глобальных переменных.

Bdfy, 31 Июля 2012

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

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