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

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

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

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

−3

if ($Module == 'edit' and $_POST['enter']) {	
ULogin(1);
$_POST['name'] = FormChars($_POST['name']);
$_POST['femeli'] = FormChars($_POST['femeli']);
$_POST['den'] = FormChars($_POST['den']);
$_POST['month'] = FormChars($_POST['month']);
$_POST['year'] = FormChars($_POST['year']);
$_POST['pol'] = FormChars($_POST['pol']);
$_POST['country'] = FormChars($_POST['country']);
$_POST['region'] = FormChars($_POST['region']);
$_POST['city'] = FormChars($_POST['city']);


if ($_POST['name'] != $_SESSION['USER_NAME']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `name` = '$_POST[name]' WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_NAME'] = $_POST['name'];
}

if ($_POST['femeli'] != $_SESSION['USER_FEMELI']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `femeli` = '$_POST[femeli]' WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_FEMELI'] = $_POST['femeli'];
}

if ($_POST['den'] != $_SESSION['USER_DEN']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `den` = $_POST[den] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_DEN'] = $_POST['den'];
}

if ($_POST['month'] != $_SESSION['USER_MONTH']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `month` = $_POST[month] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_MONTH'] = $_POST['month'];
}


if ($_POST['year'] != $_SESSION['USER_YEAR']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `year` = $_POST[year] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_YEAR'] = $_POST['year'];
}

if ($_POST['pol'] != $_SESSION['USER_POL']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `pol` = $_POST[pol] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_POL'] = $_POST['pol'];
}



if ($_POST['country'] != $_SESSION['USER_COUNTRY']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `country` = $_POST[country] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_COUNTRY'] = $_POST['country'];
}


if ($_POST['region'] != $_SESSION['USER_REGION']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `region` = $_POST[region] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_REGION'] = $_POST['region'];
}

if ($_POST['city'] != $_SESSION['USER_CITY']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `city` = $_POST[city] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_CITY'] = $_POST['city'];
}


MessageSend(3, 'Данные изменены!');
}

никто не подскажет как сократить вот эту шнягу

Запостил:

viktor, 29 Января 2016

Комментарии (26) RSS

viktor 29.01.2016 07:24 # 0

вот это можно записать в переменную WHERE `id` = $_SESSION[USER_ID]

а оброщения к базе mysqli_query($CONNECT, "UPDATE `users` SET `region` = $_POST[region]
не могу сообразить

Ответить

IvanGL 29.01.2016 07:54 # 0

if ($Module == 'edit' and $_POST['enter']) {
  ULogin(1);
  $fields = array('name', 'femeli', 'den', 'month', 'year', 'pol', 'country', 'region', 'city');
  $updates = array();

  foreach($fields as $pole) {
    $sesson_pole = 'USER_' . strtoupper($pole);
    $value = FormChars($_POST[$pole]);
    if ($value != $_SESSION[$session_pole]) {
      $updates[$pole] = $value;
      $_SESSION[$session_pole] = $value;
    }
  }
  if ($updates) {
    $sql = "UPDATE `users` SET "; 
    $sql .= implode(', ', array_map(function($k,$v){ return "`$k` = $v"}, array_keys($updates), $updates));
    $sql .= " WHERE `id` = $_SESSION[USER_ID]");
    mysqli_query($CONNECT, $sql);
  }
  MessageSend(3, 'Данные изменены!');
}

Ответить

someone 29.01.2016 08:09 # 0

femeli

FEMELI
Ответить
- Vindicar 29.01.2016 09:55 # 0
  
  foreach($fields as $pole)
  
  Рунглиш FTW
  Ответить
  - IvanGL 29.01.2016 10:01 # +2
    
    У них так поля называются: USER_DEN, но USER_MONTH; USER_POL, но USER_COUNTRY. Очевидно у них такие coding guidelines. Я лишь поддерживаю заданный стиль.
    Ответить
    - viktor 29.01.2016 10:03 # 0
      
      USER_DEN , USER_MONTH и так далее - это сессии
      Ответить
      - IvanGL 29.01.2016 10:05 # 0
        
        Это разные сессии? Или одна и та же? Это очень важно!
        Ответить
        
        viktor 29.01.2016 10:10 # 0
        
        Это разные сессии
        у каждого поля $_POST[''] после обновления в базе создается сесия
        
        'name', 'femeli', 'den', 'month', 'year', 'pol', 'country', 'region', 'city'
        
        $_SESSION[USER_NAME] , $_SESSION[USER_FEMELI], $_SESSION[USER_DEN] и так далее
        Ответить
        
        IvanGL 29.01.2016 10:15 # 0
        
        Раз это разные сессии это же совсем меняет дело! Тут надо использовать рекурсию. Возможно даже транзакции. А в каком браузере пробуешь? Попробуй в Opera. Мне нравится Opera. Она помогает более 350 миллионам пользователей находить всё самое лучшее и интересное в Интернете и получать доступ к данным с любого устройства, из любой сети и в любой точке мира. Попробуй и ты!
        Ответить
        
        viktor 29.01.2016 10:19 # 0
        
        у меня Opera другие браузеры не увожаю слишком много рекламы и вирусов
        Ответить
        
        IvanGL 29.01.2016 10:28 # 0
        
        Это замечательно! Ты на полпути к успеху. Осталось найти в моем коде пару досадных опечаток и всё заработает как надо! Или как не надо. Но обязательно заработает.
        Ты главное не опускай руки. Глаза боятся, а руки делают.
        Ответить
        
        viktor 29.01.2016 10:29 # 0
        
        глаза боятся а руки уже трисутся)
        Ответить
        
        viktor 29.01.2016 10:47 # 0
        
        1 ошибку врод6е нашел
        $sesson_pole буквы i не хватает
        может вторая в этой строке
        $sql .= implode(', ', array_map(function($k,$v){ return "`$k` = $v"}, array_keys($updates), $updates));
        в скобках $sql .= implode(', ',
        Ответить
        
        IvanGL 29.01.2016 11:13 # 0
        
        Молодец! Так держать! Еще чуть-чуть.
        Ответить
        
        tucvbif 30.01.2016 07:49 # 0
        
        Ска-а-а-а-а-а-йп!
        Ответить
- viktor 29.01.2016 12:34 # 0
  
  femeli ?
  
  нихрена понять не могу
  Ответить
  - IvanGL 29.01.2016 13:16 # +1
    
    Фамилия, наверно. name - имя, femeli - фамилия. Такой у них рунглиш, чо...
    Ответить
viktor 29.01.2016 10:00 # 0

не работает(
Ответить
- IvanGL 29.01.2016 10:08 # 0
  
  Отрицательный результат - тоже результат! Не отчаивайся! Попробуй перезагрузить компьютер и запустить еще раз.
  Ответить
  - viktor 29.01.2016 11:00 # 0
    
    не работает(
    Ответить
  - viktor 29.01.2016 12:01 # 0
    
    хоть убей не могу наити ошибку да и после того как я поставил твой код у меня кнопка выход перестала работать , у меня в этом же фаиле находится обработчик от кнопки выход . код убераю и кнопка работает.
    это все что я смог наити)
    Ответить
    - HiewMorjowie 29.01.2016 15:02 # +4
      
      По-моему viktor уже заслужил пачку дошика и приз зрительских симпатий.
      Ответить
tucvbif 30.01.2016 00:47 # 0

$session_pole это ближе к telegraph pole, или к north pole?
Ответить
- bormand 30.01.2016 02:09 # 0
  
  Dance pole
  Ответить
guest 23.02.2016 22:04 # +1

Ctrl+A, Delete
Ответить
paulpritula 27.04.2016 17:44 # 0

последний комментарий убил :D :D :D
Ответить

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

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

Комментарии (26) RSS

Добавить комментарий