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

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

+155

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

//users.php

public function search_()
    {
// --- Поиск по пользователям
      $sql_search = mysql_query($this->sql);
      $folder = $this->folder;
      if (mysql_num_rows($sql_search)>0)
      {
        $row_search = mysql_fetch_assoc($sql_search);
        do
        {
          $user_id = $row_search["id"];
          $sql = mysql_query("SELECT SOCIAL_fotos.picture_mini FROM SOCIAL_fotos WHERE fotoalbum_id = '0' and user_id = '$user_id' LIMIT 1");
          if (mysql_num_rows($sql)>0)
          {
            $row_sql = mysql_fetch_assoc($sql);
            $picture_mini = $row_sql["picture_mini"];
          }
          else
            $picture_mini = "picture/empty_mini.jpg";
          printf("<p><a href=\"%s\" target=\"_blank\"><img src=\"$folder/%s\">%s</a><br>%s", $row_search["alias"], $picture_mini, $row_search["lfm"], $row_search["city"]);
        }
        while($row_search = mysql_fetch_assoc($sql_search));
      }
    }
//funct_prover.php


function prover($con)
  {
    $element = array("'");
    $con = str_replace($element, ";appost;", $con);
    return htmlspecialchars(trim($con));
  }


//conf_connect.php

$server_con = 'localhost'; // Адрес сервера mysql
  $username_con = ''; // Имя пользователя
  $password_con = ''; // Пароль
  $dbname_con = 'socialdb';

  $url = $_SERVER["HTTP_HOST"];
  $site1 = 'panzins.ru';
  $site2 = 'www.panzins.ru';
  if ($url != $site1 and $url != 'localhost' and $url != $site2)
  exit();
  $url_path = $_SERVER["REQUEST_URI"];
  if ($url == $site2)
  {
    header("Location: http://panzins.ru$url_path");
    exit();
  }

  mysql_connect($server_con, $username_con, $password_con) or die("No connection");
  mysql_query('SET NAMES utf8') or die("Set names error");
  mysql_select_db($dbname_con) or die("No database");
  header('Content-Type:text/html; charset=utf-8');
  $table_log = "SOCIAL_log";
  $table_alias = "SOCIAL_alias";
  $table_user = "SOCIAL_user";

// И еще много няшек

MVC, PDO, Framework, для лохов ибо там избыточный код. Написать свою соц сеть за 30 дней легко. Знакомьтесь, Сергей Панзин, скромный адепт Жопова. http://rutracker.org/forum/viewtopic.php?t=4619804

Сам шидевр. http://panzins.ru/

Keeper, 23 Декабря 2013

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

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

+155

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

$s='';
		$kfg=new Konfig('');
		$kfg('self');
		{
			$kfg->k1='v1';
			$s.=$kfg->k1."\n";
		}
		{
			$kfg->k2->k='v2';
			$s.=$kfg->k2->k."\n";
		}
		{
			$kfg->{'k3.k'}='v3';
			$s.=$kfg->k3->k."\n";
		}
		{
			$kfg->k4='v4';
			$kfg->k4->k='v4';
			$s.=$kfg->k4."\n";
			$s.=$kfg->k4->k."\n";
		}
		{
			$s.=$kfg->strstr('ab','b');
		}
		$this->response->body('<pre>'.$s."\n".Debug::dump((string)$kfg)."\n".Debug::dump($kfg).'</pre>');

TDD in Action, Вот так я проверяю работоспособность волшебной во всех местах конфиги.
из личного сайта http://lure0xaos.url.ph/

Lure Of Chaos, 22 Декабря 2013

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

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

+153

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

/*где-то в начале: error_reporting(0);*/

class PhocaGalleryFile{
	function getCSSFile($id = 0, $fullPath = 0) {
		if ((int)$id > 0) {
			$db = &Factory::getDBO();
			$query = 'SELECT a.filename as filename, a.type as type'
				.' FROM #__styles AS a'
			    .' WHERE a.id = '.(int) $id;
			$db->setQuery($query, 0, 1);
			$filename = $db->loadObject();
			if (isset($filename->filename) && $filename->filename != '') {
				if ($fullPath == 1 && isset($filename->type)) {
					return self::getCSSPath($filename->type). $filename->filename;
				} else {
					return $filename->filename;
				}
			}
		}

		return false;
	}

 . . .
}

Поубивал пустые строки (по 2 штуки между каждой строкой кода) и комменты вида:
/* ===============
== This function is geting a CSSFile fron database .....
===============
*/
Гы! Код для заказчика должен выглядеть наукообразно и непонятно.
И обязательно выдавать Notices и Warnings.

virtual_cia, 19 Декабря 2013

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

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

+156

1
2
3
4
5
6
7
8

/* @component Phoca Component
 * @copyright Copyright (C) Jan Pavelka www.phoca.cz
*/
	protected function getLabel() {
		echo '<div class="clearfix"></div>';
		return parent::getLabel();
		echo '<div class="clearfix"></div>';
	}

Это диагноз...

virtual_cia, 18 Декабря 2013

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

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

+119

01
02
03
04
05
06
07
08
09
10

Well, there were other factors in play there. htmlspecialchars was a
very early function. Back when PHP had less than 100 functions and the
function hashing mechanism was strlen(). In order to get a nice hash
distribution of function names across the various function name lengths
names were picked specifically to make them fit into a specific length
bucket. This was circa late 1994 when PHP was a tool just for my own
personal use and I wasn't too worried about not being able to remember
the few function names.

-Rasmus

http://news.php.net/php.internals/70691

someone, 17 Декабря 2013

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

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

+166

1

if (substr(json_encode($row['list']), 0, 1) == '[') {

Такой вот аналог is_array()

Lowezar, 17 Декабря 2013

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

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

+149

1
2
3
4
5
6

if ($result->fetch()) {
    return $result->get('num_flags');
}
else {
    return 666;
}

Верующий программист :)

xakip, 17 Декабря 2013

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

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

+161

1
2
3
4
5
6
7

//надо показать элемент каталога во всей красе
$_CENTER="show_category_item(".$newParts[0].");";
eval($_CENTER);

// ...
$_LEFT='get_main_category($cat_id, $new_path);';
$_CENTER="show_category_item_list($".'newParts'.");";

Вот с таким адом мне приходится работать.

oooZinka, 16 Декабря 2013

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

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

+164

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

if($_POST['Selling_property_1']!=NULL):update_field('field_51db015800526',  $_POST['Selling_property_1'], $_POST['pid']); endif;
if($_POST['Relocation_budget_1']!=NULL):update_field('field_51db018100527',  $_POST['Relocation_budget_1'], $_POST['pid']); endif;
if($_POST['Selling_property_2']!=NULL):update_field('field_51db01ac00528',  $_POST['Selling_property_2'], $_POST['pid']); endif;
if($_POST['School_1']!=NULL):update_field('field_51db01b900529',  $_POST['School_1'], $_POST['pid']);endif;
if($_POST['homestyling_1']!=NULL):update_field('field_51db01c60052a',  $_POST['homestyling_1'], $_POST['pid']);endif;
if($_POST['Telephony_1']!=NULL):update_field('field_51db02090052b',  $_POST['Telephony_1'], $_POST['pid']);endif;
if($_POST['Broadband_1']!=NULL):update_field('field_51db02130052c',  $_POST['Broadband_1'], $_POST['pid']);endif;
if($_POST['TV_1']!=NULL):update_field('field_51db021c0052d',  $_POST['TV_1'], $_POST['pid']);endif;
if($_POST['Home_alarm_1']!=NULL):update_field('field_51db02280052e',  $_POST['Home_alarm_1'], $_POST['pid']);endif;
if($_POST['Craftsmen_1']!=NULL):update_field('field_51db023a0052f',  $_POST['Craftsmen_1'], $_POST['pid']);endif;
if($_POST['Home_decor_1']!=NULL):update_field('field_51db023e00530',  $_POST['Home_decor_1'], $_POST['pid']);endif;
if($_POST['Bank_1']!=NULL):update_field('field_51db024900531',  $_POST['Bank_1'], $_POST['pid']);endif;
if($_POST['Moving_cmp_1']!=NULL):update_field('field_51db025400532',  $_POST['Moving_cmp_1'], $_POST['pid']);endif;
if($_POST['homestyling_2']!=NULL):update_field('field_51db025f00533',  $_POST['homestyling_2'], $_POST['pid']);endif;
if($_POST['change_1']!=NULL):update_field('field_51db026900534',  $_POST['change_1'], $_POST['pid']);endif;
if($_POST['moving_cmp_2']!=NULL):update_field('field_51db027200535',  $_POST['moving_cmp_2'], $_POST['pid']);endif;
if($_POST['moving_truck_1']!=NULL):update_field('field_51db027e00536',  $_POST['moving_truck_1'], $_POST['pid']);endif;
if($_POST['household_cleaning_1']!=NULL):update_field('field_51db028800537',  $_POST['household_cleaning_1'], $_POST['pid']);endif;
if($_POST['insurance_1']!=NULL):update_field('field_51db029200538',  $_POST['insurance_1'], $_POST['pid']);endif;
if($_POST['cleaning_1']!=NULL):update_field('field_51db029c00539',  $_POST['cleaning_1'], $_POST['pid']);endif;
if($_POST['Home_alarm_1']!=NULL):update_field('field_51db02a70053a',  $_POST['home_alarm_2'], $_POST['pid']);endif;
if($_POST['select_2']!=NULL):update_field('field_51db02b10053b',  $_POST['select_2'], $_POST['pid']);endif;
if($_POST['insurance_2']!=NULL):update_field('field_51db02ca0053c',  $_POST['insurance_2'], $_POST['pid']);endif;
if($_POST['school_2']!=NULL):update_field('field_51db02d40053d',  $_POST['school_2'], $_POST['pid']);endif;
if($_POST['Telephony_2']!=NULL):update_field('field_51db02e10053e',  $_POST['Telephony_2'], $_POST['pid']);endif;
if($_POST['Broadband_2']!=NULL):update_field('field_51db02e50053f',  $_POST['Broadband_2'], $_POST['pid']);endif;
if($_POST['TV_2']!=NULL):update_field('field_51db02f100540',  $_POST['TV_2'], $_POST['pid']);endif;
if($_POST['gasket_1']!=NULL):update_field('field_51db02fc00541',  $_POST['gasket_1'], $_POST['pid']);endif;
if($_POST['electricity_2']!=NULL):update_field('field_51db030400542',  $_POST['electricity_2'], $_POST['pid']);endif;
if($_POST['storage_1']!=NULL):update_field('field_51db031600543',  $_POST['storage_1'], $_POST['pid']);endif;
if($_POST['electricity_1']!=NULL):update_field('field_51db031c00544',  $_POST['electricity_1'], $_POST['pid']);endif;
if($_POST['change_2']!=NULL):update_field('field_51db032700545',  $_POST['change_2'], $_POST['pid']);endif;
if($_POST['garbage_1']!=NULL):update_field('field_51db033100546',  $_POST['garbage_1'], $_POST['pid']);endif;
if($_POST['garbage_2']!=NULL):update_field('field_51db033c00547',  $_POST['garbage_2'], $_POST['pid']);endif;
if($_POST['gasket_2']!=NULL):update_field('field_51db034400548',  $_POST['gasket_2'], $_POST['pid']);endif;
if($_POST['storage_2']!=NULL):update_field('field_51db034e00549',  $_POST['storage_2'], $_POST['pid']);endif;
if($_POST['cleaning_2']!=NULL):update_field('field_51db03570054a',  $_POST['cleaning_2'], $_POST['pid']);endif;
if($_POST['moving_truck_2']!=NULL):update_field('field_51db035f0054b',  $_POST['moving_truck_2'], $_POST['pid']);endif;
if($_POST['inspection_1']!=NULL):update_field('field_51db03690054c',  $_POST['inspection_1'], $_POST['pid']);endif;
if($_POST['craftsmen_2']!=NULL):update_field('field_51db03750054d',  $_POST['craftsmen_2'], $_POST['pid']);endif;
if($_POST['household_cleaning_2']!=NULL):update_field('field_51db037e0054e',  $_POST['household_cleaning_2'], $_POST['pid']);endif;

Обработка формы румынскими колегами.

jenezis, 13 Декабря 2013

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

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

+148

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

/* Почему статические методы — говно */

abstract class BaseNewsPeer {
…
	/**
	 * Method to select one object from the DB.
	 *
	 * @param      Criteria $criteria object used to create the SELECT statement.
	 * @param      PropelPDO $con
	 * @return     News
	 * @throws     PropelException Any exceptions caught during processing will be
	 *		 rethrown wrapped into a PropelException.
	 */
	public static function doSelectOne(Criteria $criteria, PropelPDO $con = null)
	{
		$critcopy = clone $criteria;
		$critcopy->setLimit(1);
		$objects = NewsPeer::doSelect($critcopy, $con);
		if ($objects) {
			return $objects[0];
		}
		return null;
	}
…

class NewsPeer extends BaseNewsPeer
{
	…
}

kix, 13 Декабря 2013

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

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