Говнокод

function  get_info($id)  {
					$query = "SELECT  * FROM `static` WHERE `id`='$id'  LIMIT 1";
					$sql= mysql_query($query) or die(mysql_error());
					$row = mysql_fetch_assoc($sql);
					$masiv=array();
					$masiv['keywords']=$row['keywords'];
					$masiv['menu']=$row['menu'];
					$masiv['description']=$row['description'];
					$masiv['title']=$row['title'];
					$masiv['link']=$row['link'];
					$masiv['name']=$row['name'];
					$masiv['name_show']=$row['name_show'];
					$masiv['content']=$row['content'];
					$masiv['id']=$row['id'];
					$masiv['parant_id']=$row['parant_id'];
					$masiv['show']=$row['show'];
					$masiv['addons']=$row['addons'];
					$masiv['redirect']=$row['redirect'];
					$masiv['parant_id']=$row['parant_id'];
					return $masiv;
				}

Занимался переводом сайта в UTF-8, наткнулся на такую вот функцию.

Запостил:

baldrs, 03.05.2012 (Updated 25.03.2018)

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

maslenkov 14 лет ago # 0

>>>$sql = mysql_query($query) or die(mysql_error());

как то тоже использовал такой синтаксис, не помню, как конкретно выглядело в моем случае, но
не вернет ли эта строка булевое значение?
Ответить
- baldrs 14 лет ago # 0 ⇈
  
  она вернет либо результат запроса либо остановит скрипт с ошибкой. а вообще это пример работы с mysql прямо с php.net
  Ответить
- wvxvw 14 лет ago # 0 ⇈
  Тут даже моих очень скромных познаний в SQL хватит, чтобы понять, что
```
"SELECT keywords, menu...parant_id FROM `static` WHERE `id`='$id'  LIMIT 1"
```
  и потом уже не нужно было бы фильтровать массив, можно прямо результат выполнения запроса вернуть.
  
  Кстати, 90% уверенность, что parant_id на самом деле parent_id.
  Ответить
  - baldrs 14 лет ago # 0 ⇈
    а это все поля из базы и есть, поэтому select * остается актуальным. кстати, в базе именно парант_айди и написано =)
    
    а почему вот так нельзя было, я хз )
    
    function get_info($id) { $sql = mysql_query("SELECT * FROM `static` WHERE `id`=".intval($id)); return mysql_fetch_assoc($sql); }
    Ответить
    - Steve_Brown 14 лет ago # 0 ⇈
      
      папонт
      Ответить
      - lucidfoxGovno 14 лет ago # 0 ⇈
        
        childrin
        Ответить
  - eth0 14 лет ago # 0 ⇈
    
    Нет-нет, звёздочка обязательно нужна. Не переписывать же все запросы, если поле какое-то понадобится ещё.
    Ответить
MikleSmart 14 лет ago # 0

parant_id и masiv! фееричнее, только $tt и $k => $v
Ответить