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

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

+7

01
02
03
04
05
06
07
08
09
10
11

editfobjectobject($fobject_id,$fobject_object_id,
$object_type_id,$fobject_status_id,$all_sq,
$live_sq,$kitchen_sq,$floor,$balcony_id,
$su_id,$windows_id,$repair_id,
$number,$plain,$tplains,
$fcomment,$cost,$all_cost,
$manager_id,$delplain,$dic_room_type_id,
$dic_gazplit_id,$dic_phone_id,
$dic_window_type_id,$dic_pstate_id,$dic_dfloor_id,
$dic_internet_id,$dic_park_id,$dic_secur_id,
$dic_door_id,$dic_lift_id,$full_cost,$is_torg,$task_shema_id);

а до конца самой функции я тогда даже недоскроллил)

dehrom, 18 Сентября 2015

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

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

+2

1

https://github.com/imarketua/dump

Поработал на одном проекте, не оплатили. Предыстория там же, в ридми. Код знатный, подходит по тематике данного сайта, смотрим по ссылке)

HiNeX, 16 Сентября 2015

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

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

+11

1
2
3
4

for ($i = 0; $i < 11; $i++) {
    if ($i < 5) continue;
    //остальной код
}

первые 5 не нужны

kropotor, 16 Сентября 2015

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

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

+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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48

class Order_OrderProcessorBuilder
{
    /**
     * @param $processorName
     * @return Order_Abstract_OrderAbstract
     */
    private static function createOrderProcessor($processorName)
    {
        $processorClassName = 'Order_Types_'.ucfirst($processorName);
        return new $processorClassName;
    }

    /**
     * @param $builderName
     * @param Order_Abstract_OrderAbstract $orderProcessor
     * @return Order_Abstract_BuilderAbstract
     */
    private static function createOrderBuilder($builderName, Order_Abstract_OrderAbstract $orderProcessor)
    {
        $builderClassName = 'Order_Builders_'.ucfirst($builderName);
        return new $builderClassName($orderProcessor);
    }

    /**
     * @param Order_Abstract_BuilderAbstract $orderBuilder
     * @return Order_Director
     */
    private static function createDirectorClass(Order_Abstract_BuilderAbstract $orderBuilder)
    {
        return new Order_Director($orderBuilder);
    }

    /**
     * @param $orderTypeName
     * @return Order_Abstract_OrderAbstract
     */
    public static function build($orderTypeName)
    {
        /** @var Order_Abstract_OrderAbstract $orderProcessor */
        $orderProcessor = self::createOrderProcessor($orderTypeName);
        /** @var Order_Abstract_BuilderAbstract $orderBuilder */
        $orderBuilder = self::createOrderBuilder($orderTypeName, $orderProcessor);
        $director = self::createDirectorClass($orderBuilder);

        $director->construct();
        return $orderBuilder->getResult();
    }
}

Вот такая у нас есть прослойка, которая работает с паттероном "строитель", честно говоря недавно понял зачем она нужна, не знаю.. Как вам решение? Код не мой

proweber1, 16 Сентября 2015

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

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

+6

1
2
3
4
5
6
7

if (strlen($fio) < 2) {
    $error = 1; $error_text .= "Вы не представились<br>";
}

if (strlen($fio) > 5) {
    $error = 1; $error_text .= "Такого имени не бывает<br>";
}

Только избранный пройдет валидацию

develop_id666, 15 Сентября 2015

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

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

+2

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

<?php
$connect=mysql_connect('localhost', 'gvm.12', '963741258') or die(mysql_error());
mysql_select_db('gvm.12');
if(isset($_POST['enter']))
{
$login_=$_POST['login_'];
$password_=md5($_POST['password_']);

$query=mysql_query("SELECT * FROM  users WHERE login='$login_'");
$user_data=mysql_fetch_array($query);
if($user_data['password']==$password_)
{
echo "You have succesfully enter";
$check=true;
}
else 
{
echo "wrong login or password";
}
}
?>

Просто пиздец!

webalex127, 15 Сентября 2015

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

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

+2

1
2
3
4

public function defaultAction() {

        set_time_limit(1200);
        ini_set('memory_limit','4096M');	// 5 Gb

Сунь-цзы -- искусство военного обмана

cephuo, 11 Сентября 2015

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

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

+7

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

$cachefile = __DIR__. "/cache/".hash("md5", json_encode($_GET));

if (file_exists($cachefile))
{
  include($cachefile);
}
else
{
ob_start();

do_something();

$fp = fopen($cachefile,"w");
fwrite($fp, ob_get_contents());
fclose($fp);
ob_end_flush();
}

Кэширование для инвалидов, применяется во внешнем API.

nihirash, 10 Сентября 2015

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

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

+3

1
2
3

public function findByEventId($eventId) {
	return $this->findByEventId($eventId);
}

Я просто оставлю это здесь...

bit0rez, 10 Сентября 2015

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

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

+1

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

$tel = Number::clearPhone(trim($info['telephone']));
            $allowed_chars = '1234567890';
            if (!Number::clearPhone($tel)) {
                $errors['error'] = "Введите телефонный номер";
                $errors[] = 'telephone';
            }
            for ($i = 0; $i < mb_strlen($tel); $i++) {
                if (mb_strpos($allowed_chars, mb_strtolower($tel[$i])) === false) {
                    $errors['error'] = "В номере должны быть только числа";
                    $errors[] = 'telephone';
                }
            }

ну вот можно же было просто как-то так написать:

preg_match('^((8|\+7)[\- ]?)?($?\d{3}$?[\- ]?)?[\d\- ]{7,10}$', $tel)

NotFake, 09 Сентября 2015