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

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

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

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

+146

//Функция обработки ошибок PHP
set_error_handler('error_php'); 
function error_php($errno, $errstr, $errfile, $errline) 
{ 
    if (!error_reporting()) 
    { 
        return; 
    } 
     switch ($errno) 
    { 
        case E_WARNING: 
        case E_USER_WARNING: 
            $errfile = str_replace(getcwd(), '', $errfile); 
            require(ROOT_DIR.'/messages/errors/error_php.php'); 
            exit;
        break; 
    }         
} 

class mysql_db {
	
    private $db;
  
  function __construct() { //Метод вызываемый автоматически для присоединения к MySQL и выбора БД.
    $this->db=mysql_pconnect(MySQL_Host, MySQL_User, MySQL_Pass) or mysql_err('Не удалось соединиться с MySql.','Проверьте настройки параметров - MySql_Host, MySql_User, MySql_Pass в файле config.php');
    @mysql_select_db(MySQL_DB,$this->db)  or $this->mysql_err('Не удалось выбрать БД.','#'.mysql_errno().': '.mysql_error());
    @mysql_query('SET NAMES '.$this->str_sql(MySQL_Character), $this->db) or mysql_err('Не удается установить кодировку.','#'.mysql_errno().': '.mysql_error());
  }  
    
  private function str_sql ($sql) {
    return mysql_real_escape_string($sql);
  }

  private function mysql_err($txt,$error) {
    require(ROOT_DIR.'/messages/errors/error_db.php');  
    exit; 
}

  public function query($sql) {
    $result=mysql_query($sql) or $this->mysql_err('Не удается выполнить запрос к БД.','#'.mysql_errno().': '.mysql_error().'<br />--------------------------<br />SQL: '.$sql);
    return $result;
}
  
  public function count_rows($table,$where='') { //Метод подсчета количества строк в таблице.
    if($where!='') { $where=' WHERE '.$where; } 
    $result=$this->query('SELECT COUNT(1) FROM `'.$table.'`'.$where);
    return mysql_result($result,0);
  }
  
  public function inc($table,$data,$where='',$inc=1) {
    if($where!='') { $where=' WHERE '.$where; }
    $inc=(int)$inc;
    $query='`'.$data.'`=`'.$data.'`+'.$inc;
    $query='UPDATE  `'.$table.'` SET '.$query.' '.$where;
    $this->query($query);
    return mysql_affected_rows();   
  }
  
  public function dec($table,$data,$where='',$dec=1) {
    if($where!='') { $where=' WHERE '.$where; }
    $dec=(int)$dec;
    $query='`'.$data.'`=`'.$data.'`-'.$dec;
    $query='UPDATE  `'.$table.'` SET '.$query.' '.$where;
    $this->query($query);
    return mysql_affected_rows();   
  }
  
    public function insert_id() { //ID добавленной записи.
    $int=mysql_result($this->query('SELECT LAST_INSERT_ID()'),0);
    return $int;
  }
  
  public function select($table,$data='*',$where='') { //Метод запроса данных в таблице.
    if($where!='') { $where=' WHERE '.$where; }
    $query='SELECT  '.$data.' FROM `'.$table.'` '.$where;
    $result=$this->query($query);
    return $result;
  }
  
  public function insert($tabl,$data) {
     foreach ($data as $key=>$val) {
        $k[]='`'.$key.'`';
        $v[]='\''.$this->str_sql($val).'\'';
     }
     $k=implode(",",$k);
     $v=implode(",",$v);
     $query='INSERT INTO `'.$tabl.'` ('.$k.') VALUE ('.$v.')';
     $this->query($query);
     return mysql_affected_rows();   
    }
  
  public function update($table,$data,$where='') { //Метод обновления данных в таблице.
    if($where!='') { $where=' WHERE '.$where; }
    
    foreach ($data as $key=>$val) {
        $query[]='`'.$key.'`=\''.$this->str_sql($val).'\'';
    }                                                  
    $query=implode(',',$query);
    $query='UPDATE  `'.$table.'` SET '.$query.' '.$where;
    $this->query($query);

Класс для работы с MySQl

Запостил:

good_web_master, 11 Февраля 2012

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

good_web_master 11.02.2012 15:12 # −2

Писал этот код я, если что не так скажите пожалуйста.
I wrote this code, if that is not so please tell me.
Ответить
- eth0 11.02.2012 15:24 # −1
  
  Ошибся немного с ресурсом.
  Ответить
  - good_web_master 11.02.2012 15:27 # −1
    
    мне кажется почему то что у меня говно код :)
    Ответить
    - Vasiliy 11.02.2012 21:25 # +1
      
      Комплексы
      Ответить
      - SmackMyBitchUp 12.02.2012 10:54 # −1
        
        подростковые
        Ответить
roman-kashitsyn 11.02.2012 15:20 # −1

оцените код? Php Data Base Connectivity
Ответить
good_web_master 11.02.2012 15:37 # −47

показать все, что скрытоАУ
Ответить
- good_web_master 11.02.2012 15:43 # −47
  
  показать все, что скрытоАУ
  Ответить
bober_maniac 11.02.2012 15:44 # −1

Год - полное говно в силу естественных причин.
Ответить
- good_web_master 11.02.2012 15:44 # −2
  
  Год ?
  Ответить
  - bober_maniac 11.02.2012 15:48 # 0
    
    И год тоже.
    Ответить
    - good_web_master 11.02.2012 15:49 # −2
      
      WTF?
      Ответить
      - bober_maniac 11.02.2012 15:49 # −3
        
        Вам не понять, увы.
        Ответить
- good_web_master 11.02.2012 15:45 # −1
  
  каких причин?
  Ответить
  - bober_maniac 11.02.2012 15:48 # −1
    
    ООП-обертки над структурным кодом редко поражают красотой и изяществом.
    Ответить
    - good_web_master 11.02.2012 15:53 # −1
      
      Хотите сказать что на хрен тут нужно ООП?
      Ответить
      - bober_maniac 11.02.2012 15:54 # 0
        
        Хочу сказать, что один тот факт, что вы пишете на похапе очередной класс-обертку для работы с мускулем уже делает то, что вы пишете говнокодом.
        
        Вы еще не строчки не написали - а уже получился говнокод.
        
        И что бы вы не делали, как бы не меняли полученный класс - все равно выйдет говнокод.
        Ответить
        
        good_web_master 11.02.2012 16:00 # 0
        
        Что мне делать?
        Ответить
        
        guest 11.02.2012 16:05 # +1
        
        не вестись на дешевые провокации местных недотроллей
        Ответить
        
        bober_maniac 11.02.2012 16:05 # 0
        
        Погуглить и оценить глубину морального падения.
        Ответить
        
        good_web_master 11.02.2012 16:10 # −1
        
        Это как?
        http://phpprogs.ru/article/mysql-sozdanie-klassa-na-php-dlya-raboty-s-mysql
        Ответить
        
        Говногость 13.02.2012 21:37 # 0
        
        sozdanie-klassa-na-php-dlya-bydla
        Ответить
        
        guest 14.02.2012 09:58 # 0
        
        semki-est?
        Ответить
        
        guest 11.02.2012 16:04 # 0
        
        дооо посаны из моего класса сказали что нужно ебошить по хардкору на асме, все остальное говнокод!!!!
        Ответить
        
        good_web_master 11.02.2012 16:05 # −1
        
        :-)
        Ответить
        
        bober_maniac 11.02.2012 16:05 # −2
        
        Респект и уважуха посанам из твоего класса!
        Ответить
roman-kashitsyn 11.02.2012 16:16 # +4

даже я знаю, что в php есть PDO.
Ответить
- good_web_master 11.02.2012 16:21 # −4
  
  Не знал! Спасибо! Будем изучать......
  Ответить
- eth0 11.02.2012 17:51 # −1
  
  При этом каждый должен сделать свой велосипед.
  Ответить
  - roman-kashitsyn 11.02.2012 17:56 # −1
    
    Ну не знаю, я свою orm не писал и не собираюсь.
    А в обёртывании (говяшками) api mysql смысла особого не вижу.
    Ответить
    - eth0 11.02.2012 19:10 # −1
      
      Теоретически, там можно сделать забавные вещи. А за счёт mysql_fetch_object и заполнять объекты сразу. Правда, беда в том, что если заполнять через магические __set, то крайне медленно. А сконструировать объект с заранее нужным набором приватных/публичных полей без eval-ов не так просто. Я пробовал всё, сериализацию, ещё какую-то хитрость с приведением объекта к массиву, где для приватного поля использовалось имя с \0 в начале. В итоге, ничего не вышло, плюнул, бросил. Хотя, конечно, задаток интересный.
      Ответить
    - bugmenot 11.02.2012 23:30 # −1
      
      пхпешники всегда пишут свои велофреймворки, потому что сходи на кузнецкий - у всех свои фреймворки
      Ответить
    - roman-kashitsyn 13.02.2012 10:14 # 0
      
      Минусаторам-велосипедистам могу лишь сказать, что читал Фаулера и при необходимости могу сваять ORM. Просто необходимости как таковой нет.
      Ответить
      - eth0 13.02.2012 10:24 # 0
        
        А я вообще за велосипеды. о_О
        Необъяснимая просто отака.
        Ответить
        
        bugmenot 13.02.2012 18:25 # 0
        
        братья наши меньшие ужасно примитивны - пожрать, посрать, испытать баттхёрт
        Ответить
        
        eth0 13.02.2012 20:12 # 0
        
        Интересно, а можно приручить рачка-клована?
        Ответить
- Vasiliy 11.02.2012 21:33 # −1
  
  Самое печальное не все хостеры это знают или знают но как то урезано только для мускула. Есть еще ниша для оберток. Тут главное без фанатизма а то можно на лепить уровней абстракции и потом аргументы по ним гонять.
  Ответить
guest8 09.04.2019 12:40 # −999

показать все, что скрытоvanished
Ответить

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

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

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

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