`id` = ' - ты зачем последнюю кавычку экранизируешь? Я что-то тебя не понимаю, у тебя вообще она работает?!

Метод работать не будет, так как аргументом mysql_real_escape_string() должно быть значение, а не запрос.
В результате выполнения приведенного кода ты просто получишь SELECT `login` FROM `user` WHERE `id` = \'что там лежит в parent::$id\', и возникнет ошибка SQL.

Итого имеем

mysql_real_escape_string($query);

где $query - "SELECT `login` FROM `user` WHERE `id` = '". parent::$id ."'"
И тогда получаем

mysql_real_escape_string("SELECT `login` FROM `user` WHERE `id` = '". parent::$id ."'");

Эээээээ кажется это не будет работать :)

Ах, да, загнал я с mysql_real_escape_string

Invis1ble
как в кохане, получить ас. массив, а потом разобрать на одни большой через foreach

Походу вам только запросы новичков исправлять, как по делу так молчат все, ну лады

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

---

Спустя 1 минута, 39 секунд Invis1ble написал(а):

---

покажи текущий вариант кода с учетом исправлений

public static function select($query)
	{
		$result = mysql_query($query) or die($logs->write(mysql_error()));
		while($row = mysql_fetch_assoc($result))
			$data[] = $row;

		return $data;

	}

if(isset(parent::$id))
    {
	$data['user_name'] = mysql :: select("SELECT `login` FROM `user` WHERE `id` = '". parent::$id ."'");
	echo $data['user_name'];
    }

Типа так...

Invis1ble
самое банальное :rolleyes:

public static function select($query) {
  $result = mysql_query($query) or die($logs->write(mysql_error()));
  if ( !mysql_num_rows($result) ) {
    return false;
  }
  while($row = mysql_fetch_assoc($result)) {
    $rows[] = $row;
  }
  return $rows;
} 

ну нормально
только

Winston
Invis1ble
Спасибо за подсказки! Получилась мини кохана:
classes_mysql.php

public static function select($query)
	{
		$result = mysql_query($query) or die(mysql_error());
		
		while($row = mysql_fetch_assoc($result))
		{
		$rows[] = $row;
		}
		return $rows;
	} 

controller_user.php

public function show()
	{
		$data = array();
		if(isset(parent::$id))
		{
			$data['user_name'] = mysql :: select("SELECT `login` FROM `user` WHERE `id` = '". parent::$id ."'");
		}
		View::Factory('user_show', 'Просмотр ', $data);
	}

view_user_show.php

Просмотр юзера
<?php
foreach($user_name as $view)
{
echo $view['login'];
}
?>

в итоге по запросу http://mvc/user/show/1 выводит Просмотр юзера johniek_comp :)

Invis1ble
инициализировал, в случае отсутствии выборки возвращаю null, но вылазит нотис на foreach, мол ничего обрабатывать ему, а лишней проверки на наличие результата выборки делать нихачю!

я ж написал выше, возвращай пустой массив, тогда с foreach проблем не будет

Invis1ble
Решил все так:

return isset($rows) ? $rows : $array = array();

ни нотисов, ничего, как и хотел :)

вот получше вариант, без доп. проверки

$rows = array();

while ($row = mysql_fetch_assoc($result))
  $rows[] = $row;

return $rows;

Invis1ble
Этот способ медленнее работает

медленнее чем с проверкой? ты замерял?

да эт я не глянул на знак(-), твой быстрее

хотя не, подожди, я чего-то торможу:
твой: -1333184323.54
мой: -1333184266.13

Как мне кажется использовать пдо было бы лучше. Как убедился вчера - кода меньше ( 2 строчки )

m4a1fox
Угу, после твоей темы пошел почитал что такое pdo, вот хочу такой же класс как у тебя написать, но пока совмещу с остальным это все дело.
Invis1ble
забей

johniek_comp