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

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

+2

1

https://toster.ru/q/274179

Ученики Mr.Shit (беднягу зачмырили и он сменил домен на php. webtm.ru) продолжают радовать

loki90, 11 Декабря 2015

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

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

+5

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

$scales = db_select('_scale', 'q')
        ->condition('q.gender', $gender)
        ->fields('q', array('id'))
        ->execute();

    $i = 1;
    while ($scale = $scales->fetchObject()) {
        $raw_score = 0;

        $query = db_select('_scale_question', 'q')
            ->condition('q.scale_id', $scale->id)
            ->fields('q')
            ->execute();

        while ($item = $query->fetchObject()) {
            $user_answer = db_select('_user_answer', 'q')
                ->condition('q.question_id', $item->question_id)
                ->condition('q.result_id', $result_id)
                ->fields('q', array('answer'))
                ->execute()
                ->fetchField();

            if ($user_answer == $item->expected_answer) {
                $raw_score++;
            }
        }


        $raw_scores_item['scale_id'] = $scale->id;
        $raw_scores_item['score'] = $raw_score;

        $raw_scores[$i] = $raw_scores_item;
        $i++;
    }

Drupal 7, SQL-запросы с DB API Drupal в двойном цикле(!).
...А заменить можно 1 нормальным SQL-запросом.

skigf, 10 Декабря 2015

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

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

+1

1
2
3

If (!isset($_GET['mode']) || !ereg("^[a-z0-9_-]{1,15}$",$_GET['mode'])) $mode='read'; else $mode=$_GET['mode'];

switch ($mode){

Как же я люблю в говнодвигах ковырятся Гг

Vocaloid_IA, 09 Декабря 2015

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

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

+1

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

<? 

      $login= _string($_COOKIE['login']);
      $password = _string($_COOKIE['password']);

  if($login && $password) { 
     
  $q = mysql_query('SELECT * FROM `users` WHERE `login` = "'.$login.'" AND `password` = "'.$password.'"');
  $user = mysql_fetch_array($q); 

  if(!$user) { 
     
  setCookie('login', ''); 

  setCookie('password', ''); 
   
  } 

      mysql_query('UPDATE `users` SET `online` = "'.time().'",
                                          `ip` = "'.$_SERVER['REMOTE_ADDR'].'",
                                          `ua` = "'.$_SERVER['HTTP_USER_AGENT'].'",
                                        `self` = "'.$_SERVER['PHP_SELF'].'" WHERE `id` = "'.$user['id'].'"');
?> 
<? 
} 
?>

Нашел на бумасе Хд

Vocaloid_IA, 09 Декабря 2015

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

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

+4

1
2
3
4
5
6
7
8

public function validate($value) {
    try {
        $date = new \DateTime($value);
        return true;
    } catch (\Exception $e) {
        return false;
    }
}

DateTime Validator

bykovski, 08 Декабря 2015

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

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

+3

1

$this->pass_change_token = md5($this->id . "sdkfjgllfdlk" . time() . $this->domain_id . rand(1000, 9000) . $this->email);

Генерируем токен для смены пароля пользователя.

jbot, 07 Декабря 2015

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

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

+5

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

if ($template)             // BAD
if (isset($template))      // GOOD
if ($template !== NULL))   // GOOD
if ($template !== ''))     // GOOD

if (strlen($template) > 0) // BAD! strlen("-1") is greater than 0
if (is_string($template) && strlen($template) > 0) // BETTER

if ($foo == $bar)          // BAD, avoid truthy comparisons
if ($foo != $bar)          // BAD, avoid falsy comparisons
if ($foo === $bar))        // GOOD
if ($foo !== $bar))        // GOOD

 // We only allow valid persons
if (is_object($p) && strlen($p->lastN) > 0 && $p->hidden === FALSE && $this->environment->moonPhase === MOON_LIB::CRESCENT) {
 $xmM = $thd;
}

if ($this->isValidPerson($person) {
  $xmM = $thd;
}

Решил погуглить гайдлайны по PHP: http://flowframework.readthedocs.org/en/stable/TheDefinitiveGuide/PartV/CodingGuideLines/PHP.html

HiNeX, 07 Декабря 2015

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

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

+5

1
2
3
4

$sql = "SELECT * FROM users WHERE id='" . $id . "'";
                $query = db_assoc( $sql );
                unset( $query['id'], $r['openid'], $query['password'], $query['refer'] );
                $_SESSION['inform'] = $query;

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

drop_database, 06 Декабря 2015

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

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

0

1
2
3
4
5

$a=array();
for($i=0;$i<100;$i++){
 $t=rand(1,100);
 $a[$t]=$t;
}

Решение задачи из теста на пригодность в джуны.
Сама задача была поставлена так:

"Нужно объявить массив, в цикле от 1 до 100 заполнить его случайными цифрами (тоже от 1го до 100), и в итоге получить массив с уникальными значениями, реализовать минимальным кол-вом строк. "

Шедевральная наркомания.

Twissel, 04 Декабря 2015

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

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