Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Функция и параметры

 PHP.SU

Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


 Страниц (1): [1]   

> Без описания
amf1k
Отправлено: 22 Апреля, 2015 - 06:19:26
Post Id


Гость


Покинул форум
Сообщений всего: 110
Дата рег-ции: Май 2013  


Помог: 0 раз(а)

[+]


Есть 2 страницы: например - kino.php и actor.php
Есть БД, в ней 2 таблицы - фильмы и актеры
Есть функция вывода информации с БД
PHP:
скопировать код в буфер обмена
  1.  
  2. function getInfo($kino, $actor){
  3.     //Если 1-й параметр равен NULL
  4.     if(empty($kino)){
  5.         //Делаем запрос на выборку всех актеров
  6.         $query = 'SELECT * FROM `actor`';
  7.     }
  8.     //Если 2-й параметр равен NULL
  9.     else{
  10.         //Делаем запрос на выборку всех фильмов
  11.         $query = 'SELECT * FROM `kino`';
  12.     }
  13.     $result = mysql_query($query);
  14.     $array = array();
  15.     while($row = mysql_fetch_assoc($result)){
  16.         $array[] = $row;
  17.     }
  18.     //Возвращаем массив актеров, или массив фильмов
  19.     return $array;
  20. }
  21.  

Если страница kino.php, то выводим информацию по фильмах

Если страница actor.php, то выводим информацию по актерах

Вопрос: Можно ли так применять параметры к функции (NULL), если нет, то почему???
Если честно, то код работает, но что то мне подсказывает, что этот код не имеет права на жизнь?
 
 Top
Мелкий Супермодератор
Отправлено: 22 Апреля, 2015 - 08:17:00
Post Id



Активный участник


Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009  
Откуда: Россия, Санкт-Петербург


Помог: 618 раз(а)




Да, не имеет права на жизнь.
Вы пытаетесь в один метод засунуть два не связанных между собой.


-----
PostgreSQL DBA
 
 Top
amf1k
Отправлено: 22 Апреля, 2015 - 08:34:27
Post Id


Гость


Покинул форум
Сообщений всего: 110
Дата рег-ции: Май 2013  


Помог: 0 раз(а)

[+]


Мелкий пишет:
Да, не имеет права на жизнь.
Вы пытаетесь в один метод засунуть два не связанных между собой.

это же для примера, код рабочий!
 
 Top
tato
Отправлено: 22 Апреля, 2015 - 09:32:25
Post Id



Посетитель


Покинул форум
Сообщений всего: 468
Дата рег-ции: Сент. 2011  
Откуда: Владивосток


Помог: 8 раз(а)




должно быть две функции getKino() и getActor() или если не хотите делать две функции, то

PHP:
скопировать код в буфер обмена
  1.  
  2. getInfo($table) {
  3.     $sql = "select * from `$table`";
  4. ...
  5. }
  6.  


Но так делать нельзя, т.к. это не безопасно. А вообще почитайте про ORM тогда поймете как надо.


-----
просто ?: сложно
 
 Top
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Вопросы новичков »


Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.
 



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB