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

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

+72

1
2
3
4
5
6
7

<?php if ($content): ?>
    <?php if($is_page): ?>  
        <?php print $content; ?>  
    <?php else: ?>  
        <?php print $content; ?>  
    <?php endif?>
<?php endif?>

Где-то, в глубоких кодах проекта (писанном на Drupal 7)

Edd, 07 Августа 2012

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

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

+47

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

if ($num_rows2>0) {
	$raw2=mysql_fetch_array($sth2, MYSQL_ASSOC);
	$desc=$raw2['desc'];
	$sector =  "$desc";//O-o-o!
} 
//далее $desc нигде не используется, равно как и $raw2
//и ещё:
	$sid=$raw['id'];
	$sector=$raw['sector'];
	$selind='';
	if ($sector==$sid) { $selind='selected'; }
	echo "<option value='$sid' $selind >$sector</option>";

//и так по всему коду
//А ЭТО вместо темплэйта от 30 до 150 include за один раз с огрызками HTML и PHP
<?php 
require_once('includes/core.php');      // connection, config
$is_index_page = true; // add this variable to check that we are on default index page (#2907)
require_once('header.php');
require_once('menu.php');
include('homepage.php');
. . .
. . .
require_once('footer.php');

?>

Реальный кусок Хадж-кода. В комплекте более 3500 файлов перекрёстно инклюдящих друг друга. Действительно "Индусский код" оплачивается построчно..., дёшево, долго, сердито.

virtual_cia, 06 Августа 2012

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

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 / Говнокод #11550

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

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

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

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

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

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

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

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

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