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

    +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
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    class MysqlResultSet implements Iterator{
  private $strSQL;
  private $databasename;
  private $connection;
  private $result;
  private $valid;
  private $currentrow;
  private $key;
  const INDETERMINATE_TOTAL_NUMBER = 5001;
  const UNNECESSARY_SQL_CALC_FOUND_ROWS = 5002;
  const NOT_SELECT_QUERY = 5003;
  public function __construct( $strSQL, $databasename, $connection ){
    $this->strSQL = $strSQL;
    $this->connection = $connection;
    $this->databasename = $databasename;
    if(!mysql_selectdb($databasename, $connection)){
      throw new MySQLException(mysql_error(), mysql_errno());
    }
    if(!$this->result = mysql_query($strSQL, $connection)){
      throw new MySQLException(mysql_error(), mysql_errno());
    }
    if (stristr($strSQL,"SQL_CALC_FOUND_ROWS")){
      $msg = "No need to use SQL_CALC_FOUND_ROWS.";
      throw new MySQLException($msg, self::UNNECESSARY_SQL_CALC_FOUND_ROWS);
    }
    $this->rewind();
  }
  public function __destruct(){
    $this->close();
  }
  public function __call($name, $args){  
    $args = null;
    $name = "mysql_". $name;
    if(function_exists($name)){
      return call_user_func_array($name, $args);      
    }  
  }
  # Ну и так далее...

    31-38 строки: Не понятно, зачем нужно было вообще этот класс писать?!
    32-ая строка вообще сногсшибательна... (P.S. Из одной book по ООП в PHP5)

    Запостил: dwinner, 15 Ноября 2010

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

    • ava istem 16.11.2010 00:42 # 0

      Так этож не продакшн...
      Или Вы хотели сразу, любой код из книжки - в готовый сайт?
      Ответить

      • ava dwinner 16.11.2010 07:52 # 0

        Нет, конечно, просто глупость писать wrapper'ы подобным образом (на мой взгляд), да ещё и обнуляя аргументы, чтобы вообще всё стало бесполезно, в реале я пользуюсь вещами куда сложнее...
        Ответить

        • ava istem 16.11.2010 23:35 # +3

          я так понимаю - это пример. А пример может быть и говном тоже, дабы донести мысль внятно...
          Ответить

    • ava dwinner 17.11.2010 08:35 # 0

      Если только мысль как НЕ следует применять magic __call, то да!
      Ответить

    • ava japan007 20.11.2010 03:34 # −3

      бред сивой кобылы
      Ответить

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