sql запрос в функции

[Winni]

Можно ли делать sql запрос внутри функции. Например:

 function left($n) 
 {
	 echo "<div class=\"news\">";
	 $qvery = mysql_query("SELECT * FROM  left_news WHERE id=$n ",$db);
	 $array = @mysql_fetch_array($qvery);
	 echo $array['id'];
	 echo $array['zag'];
	 echo $array['pic'];
	 echo $array['news'];
	 echo "<div class=\"n_zag\">$zag</div>";

  $n++;
  echo  "</div>";
 };

[Veseloff]

Конечно можно. И в методе класса тоже можно.

[Winni]

а почему у меня выскакивает это сообщение, хотя я функцию даже не запрашиваю

Warning: mysql_query() expects parameter 2 to be resource, null given in Z:\home\see\www\main.php on line 9

девятая линия:

 $qvery = mysql_query("SELECT * FROM  left_news WHERE id=$n ",$db);

[Int]

global $db;

[Winni]

Зачем мне делать её глобальной, $db это идентефикатор, я получил его в этом файле на пять строк выше.

Если делать всё тоже вне функции, то всё работает.

[Searcher]

Зачем мне делать её глобальной, $db это идентефикатор, я получил его в этом файле на пять строк выше.

Какая разница на сколько строк выше вы получили значение $db? Главное, что эта переменная определена вне функции.

И дальше вы сами ответили на свой вопрос, только мозгами надо немного поработать...

Если делать всё тоже вне функции, то всё работает.

Ну? Поняли?

[Winni]

да, так работает, спс.

[keltanas]

да, так работает, спс.

было бы правильнее передавать идентификатор базы в параметре.

но если вы пользуетесь mysql_query(), то второй парамерт можно вообще опустить.

[Winni]

так я и передаю ид базы параметром(а как иначе?)

А насчёт опустить, это лучший вариант, хы и столько проблем новичку из-за необязательного параметра.

[Veseloff]

И вообще, это лишнее, ИМХО. Будет работать и без $db. Маловероятно, что вы производите выборки из разных баз в рамках одного сайта. Ну а если используете, то тогда да.

[keltanas]

так я и передаю ид базы параметром(а как иначе?)

А насчёт опустить, это лучший вариант, хы и столько проблем новичку из-за необязательного параметра.

Я имел ввиду это:

function left( $db, $n )
{
	// ...
}

и вообще, если это метод класса, то его конструктору передается этот самый идентификатор соединения, который он сохраняет у себя

class MyClass
{
	private $db;
	function __construct( $db )
	{
		 $this->db = $db;
	}
}

А потом во всех методах его используем

// class MyClass
	function left( $n )
	{
		 $sql = "...";
		 mysql_query( $sql, $this->db );
		 // ...
	}