1. PHP / Говнокод #5338

    +159

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    <?
    mysql_connect("$db_host", "$db_user", "$db_pass") or die(mysql_error());
    mysql_selectdb($db) or die(mysql_error());
    $q = "SELECT `login`, `email`, `name`, `second_name`, `age` FROM $db.accounts WHERE login='".$_SESSION['login']."' ";
    $res = mysql_query($q) or die(mysql_error());
    $row=mysql_fetch_array($res);?>
    ...
    <?
    $name = $_POST['name'];
    $email = $_POST['email'];
    $log = $_SESSION['name'];
    $second_name = $_POST['second_name'];
    $age = $_POST['age'];
    mysql_connect("$db_host", "$db_user", "$db_pass") or die(mysql_error());
    mysql_selectdb($db) or die(mysql_error());
    $query = "UPDATE accounts SET name ='$name', second_name='$second_name', email='$email', age='$age' WHERE login = '$log'";
    mysql_query($query) or die(mysql_error());
    ?>

    http://www.php.ru/forum/viewtopic.php?t=30226
    Туча ошибок и проблемы с безопасностью

    Запостил: Devzirom, 20 Января 2011

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

    • христоматийное говно
      Ответить
    • or die(mysql_error()); после каждого запроса радует.
      Пациенты идут на поправку.
      Ответить
      • вот я всегда сомневаюсь, а не говно ли убивать скрипт? на мой взгляд, нужно обрабатывать ошибку дальше и выводить красивую страничку
        Ответить
        • для этого придумали эксепшены
          Ответить
        • > выводить красивую страничку...
          [...] catch (Exeption $e) {
          echo "<p class="very-nice-paragraph">В Вашем скрипте <b>" . __FILE__ . "</b>, в строке <b>" . __LINE__ . "</b>, возникла страшная ошибка!</p>";
          }
          Ответить
    • уныло
      Ответить

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