function sql($sql){
 return @mysql_query($sql) or die('<h4>Не вышло выполнить <i>SQL</i> запрос <b>[ <font color="red">'.$sql.'</font> ]</b></h4>');
}

alex12060, не в этом дело!

mid

Да? В чем же?

Вот я и спрашиваю.Все запросы кроме этого

SELECT `vizit` 
FROM `online` 
WHERE `iduser`='(int)$id' 
LIMIT `1`

работают!
И без пользовательской пробовал делать этот sql запрос!
Нихрена.А в функции sql и возвращать ничего не надо!

у тебя для mysql_fetch_array функция sql должна вернуть результат, так что ставь return.

alex12060, я уже проверил давно,нихрена не работает не с return ни с чем!!!Я ещё до того как сюда выложить ставил этот return чертов!
И написал что не в этом дело!
Не нужно мне ссылок я и сам прекрасно знаю про пользовательские функции,и return в частности.Ну если без него работает.Тем более что у меня только в одном запросе вернуть надо,который и не работает!Ну и просто если ввести mysql_query(запрос) тоже не рабит!

Конкретна с запросом что-то!

$mid = mysql_fetch_array(mysql_query("SELECT `vizit` FROM `online` WHERE `iduser`='".(int)$id."' LIMIT `1`"));

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in Z:\home\test1.ru\subdomain\functions.php on line 24

Убедились что не функции sql дело?а?и ставил, ещё раз повторюсь, RETURN!

И сразу напишу,а то наверно сидите и думаете что я чайник!База данных есть,подключение есть.Все есть!И по теории должно работать,а так хрен.Хочу заметить что остальные запросы работают!

Окей, все равно у тебя без return работать не будет :)
И не психуй.

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

SELECT `vizit` FROM `online` WHERE `iduser`= ".(int)$id." LIMIT 1

А вообще, я бы сделал твою функцию как миниимум такой:

function sql($sql) {
  if (!empty($sql)) {
     $res = mysql_query($sql);

       if (mysql_num_rows($res) > 0) {
          return $res;
       } else {
          die('We have 0 results');
       }
  } else {
      die('Attempt to execute empty query!');
   }
}

Но а вообще, должно выозвращать при неудаче не строки с умиранием, а false, но это так, для дебажинга.

может быть LIMIT 1, без обратных кавычек? в них обычно указываются название полей и таблиц.

alex12060, я понимаю что ошибка или пустой результат.у меня запросы на удаление,редактирование,добавление..что для них нужно возвращать?а на этот запрос Mysql_query написал!ъ
Все,спс.Говорил же ошибка в запросе.И с моей функцией sql тоже рабит и без return..Но я сейчас для запросов SELECT другую функцию напишу,с RETURN!Всем спс.+1 в карму

mid

А по мне, проще сделать флаг на возвращение результата в одной функции.

function sql($sql, $return = false) {
  # ...
  if ($return)
    return $res;
}

alex12060, я без флажков обошелся:

function sql($sql){
	$type = explode(' ', $sql);
		$type = $type['0'];
		
			if($type == 'SELECT')
				{
				$ok = @mysql_fetch_array(mysql_query($sql)) or die('<h4>Не удалось выполнить <i>SQL</i> запрос [ <font color="red">'.$sql.'</font> ]</h4>');
				return $ok;
				}
			else
				{
				@mysql_query($sql) or die('<h4>Не удалось выполнить <i>SQL</i> запрос [ <font color="red">'.$sql.'</font> ]</h4>');
				}
}

mid
Как ужасно!
Тебе же показали, как правильно... зачем лепить mysql_fetch_array, когда у тебя ошибка может быть на уровне mysql_query() ? И неправильные запросы надо логировать.

inpost,так будет правильно?

<?php
function sql($sql){
	$type = explode(' ', $sql);
		$type = $type['0'];
		
			if($type == 'SELECT')
				{
				$mysql_Q = @mysql_query($sql) or die('<h4>Не удалось выполнить <i>SQL</i> запрос [ <font color="red">'.$sql.'</font> ]</h4>');
				$ok = @mysql_fetch_array($mysql_Q);
				return $ok;
				}
			else
				{
				@mysql_query($sql) or die('<h4>Не удалось выполнить <i>SQL</i> запрос [ <font color="red">'.$sql.'</font> ]</h4>');
				}
}

Гость_mid
Нет, собаки лишние.

А для чего вообще собаки нужны?Я просто увидел у кого-то..У винстона вроде,и начал ставить на все запросы

mid

жаль, не стать тебе хорошим программистом...

alex12060, ну чтож теперь:)
Ты с битвы экстрасенсов?

Guest

А зачем быть экстрасенсом, когда тут все видно как на ладони.

Вместо всего этого чихуа-хуа можно было обойтись очень элегантной функцией, которая не будет грузить PHP зря, лишь для того, чтобы узнать, запрос на выборку или нет.

alex12060, кому как больше нравится,правильно?)
Судить по тому,что кто-то пытается своими знаниями решить проблему мне кажется смешным:)
Ну и ладно:)У каждого свое мнение)

Guest

Кому как больше нравится?
Окей, у меня больше нет вопросов.
Вот когда тебя начнут пинать за тормоза и миллиарды ошибок в проектах, за разработку которых ты будешь получать деньги, тогда вспомнишь меня
удачи.

alex12060
Да ладно тебе,я не упертый баран:)
Я просто не умею работать с флагами,поэтому прислал как я сделал,чтобы тоже проверили.Я не думаю что из-за одной функции у меня все будет тормозить