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

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

+1

01
02
03
04
05
06
07
08
09
10

var cloths = {
  <?php foreach ($clothList as $cloth): ?>
    "<?=$cloth->id()?>": {
      "id": <?=$cloth->id()?>,
      "name": "<?=$cloth->name()?>",
      "public_name": "<?=str_replace('"', '\"', $cloth->public_name())?>"
    },
  <?php endforeach; ?>
  "dummy": {}
};

kgm-rj, 14 Декабря 2015

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

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

+2

1
2
3
4

if (!$subscription) {
    return $this->redirect(['action' => 'cancel_success']);
    throw new BadRequestException();
}

epadqwbd, 11 Декабря 2015

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

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

+2

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

$sql = 'SELECT * FROM `user` WHERE `id`=' . $_SESSION['userId'];
        $result = $this->db->query($sql);
        $data = $result->fetch_assoc();
        $sessionId = $data['sessionID'];
        $session_id_to_destroy = $sessionId;
        $login = 'Login';
        if ($_GET['SID'] != $sessionId) {
            if (file_exists(ROOT . DS . 'app' . DS . 'sessions' . DS . 'sess_' . $sessionId)) {
                unlink((ROOT . DS . 'app' . DS . 'sessions' . DS . 'sess_' . $sessionId));
            }
            session_id($sessionId);
            session_start();
            session_destroy();
            session_start();
            session_regenerate_id($sessionId);
            $login = 'Check';
        }

        $sql = 'UPDATE `user` SET `sessionID`="' . session_id() . '", `lastLogin`=' . time() . ', `everyDayUse`=' . $everyDayUse . ', `lastDayUse`=' . $lastDayUse . ' WHERE `id`=' . $_SESSION['userId'];
        $_SESSION['userInfo']['sessionID'] = session_id(); 
        $this->db->query($sql);

dilana, 07 Декабря 2015

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

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)

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