Вывод данных MySQL по имени. Как реализовать?

Question

[Nokira Nokawa]

Проблема вот в чем, мне надо вывести гильдию, в которой состоит игрок. Она искаться по имени игрока

if($_GET['user']){
	$result = mysql_query ('SELECT * FROM authme WHERE id='.$_GET['user'].'');
	$myrow = mysql_fetch_array ($result);
	$guild = mysql_query ('SELECT * FROM clan_members WHERE name='.$myrow['username'].'');
	$c_myrow = mysql_fetch_array ($guild);
	$tpl->get_tpl('template/Default/user.tpl');
	$tpl->set_value('USERNAME', $myrow['realname']);
	$tpl->set_value('WORLD', $myrow['world']);
	if($myrow['isLogged'] == 1){
		$tpl->set_value('ONLINE', '<font color="green">В игре!</font>');
	}else{
		$tpl->set_value('ONLINE', '<font color="red">Вне игры!</font>');
	}
	if($myrow['access'] == 1){
		$tpl->set_value('ACCESS', '<font color="red">Администратор</font>');
	}else{
		$tpl->set_value('ACCESS', '<font color="gray">Игрок</font>');
	}
	if($c_myrow['clan']){
		$tpl->set_value('GUILD', $c_myrow['clan']);
	}else{
		$tpl->set_value('GUILD', 'Не состоит в гильдии');
	}
}else{
	echo 'Пользователь не выбран';
}

В таблице гильдий есть запись:

Вот таблица с пользователями:

Мне надо вывести гильдию, где name = имени пользователя, т.е. $myrow['username'];
но мне выдает ошибку
Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in M:\OpenServer\OpenServer\domains\vergal.ru\index.php on line 10

А если добавить || die(mysql_error())
То ошибка следующая
Unknown column 'nokira' in 'where clause'
Не могу понять, что не так. Прошу вашей помощи

Comments

[krypt3r]

А вы в курсе, что mysql_query() может вернуть FALSE (для select-запросов) и ваш дальнейший код бессмысленен? Учитесь обрабатывать результаты, которые возвращают функции.
А в курсе вы того, что переменные $_GET['user'] и $myrow['username'] должны содержать целочисленные значения?
А знаете ли вы о существовании prepared statements и PDO или mysqli?

[Nokira Nokawa]

Нет :D

[Nokira Nokawa]

А как тогда мне получить содержимое по имени?

[krypt3r]

Вы дали структуру только одной таблицы

[Nokira Nokawa]

krypt3r: дополнил

[krypt3r]

Внимание, вопрос! А зачем вы запрашиваете две таблицы, если имя пользователя имеется в обеих? Если в $_GET['user'] у вас имя пользователя, то делайте запрос сразу ко второй таблице

[Nokira Nokawa]

krypt3r: там идет по id запрос

[krypt3r]

Если я правильно понял, попробуйте так:

try {
    // ...
    if (!is_numeric($_GET['user'])) {
        throw new ErrorException('Неверные входные данные');
    }
    $result = mysql_query("SELECT * FROM authme WHERE username = '" . (int) $_GET['user'] . "'");
    if ($result === false) {
        throw new ErrorException(mysql_error());
    }
    $myrow = mysql_fetch_array($result);
    if ($row === false) {
        throw new ErrorException('Ошибка получения данных');
    }
    $guild = mysql_query("SELECT * FROM clan_members WHERE name='" . mysql_real_escape_string($myrow['username']) . "'");
    // ...
    // либо оба запроса можно объединить в один через JOIN
} catch (ErrorException $e) {
    die('Error: ' . $e->getMessage());
}

Answer 1

[Павел Беляев]

Во первых не забывайте про фильтрацию поступаемых данных из GET, ну и строковые значения в кавычки засуньте, а названия полей в косые.

$guild = mysql_query ('SELECT * FROM clan_members WHERE name='.$myrow['username'].'');

типа так

$guild = mysql_query ("SELECT * FROM clan_members WHERE `name`='".$myrow['username']."'");

Во вторых, ну

//тут надо обрезать $_GET['user'], оставив только [0-9]+

//нужно еще проверить существование ячейки в массиве
if(isset($_GET['user']) && $_GET['user']>=0){
 

  $sql = "SELECT 
  `authme`.`realname`, 
  `authme`.`world`, 
  `authme`.`isLogged`,
  `authme`.`access`,
  `clan_members`.`clan`
  FROM `authme`
  LEFT JOIN `clan_members` ON `clan_members`.`name`=`authme`.`username`
  WHERE `authme`.`id`=".$_GET['user'];

  $result = mysql_query($sql);
  $myrow = mysql_fetch_array ($result);

  unset($result,$sql); //подчищаем лишнее



  $tpl->get_tpl('template/Default/user.tpl');
  $tpl->set_value('USERNAME', $myrow['realname']);
  $tpl->set_value('WORLD', $myrow['world']);
  if($myrow['isLogged'] == 1){
    $tpl->set_value('ONLINE', '<font color="green">В игре!</font>');
  }else{
    $tpl->set_value('ONLINE', '<font color="red">Вне игры!</font>');
  }
  if($myrow['access'] == 1){
    $tpl->set_value('ACCESS', '<font color="red">Администратор</font>');
  }else{
    $tpl->set_value('ACCESS', '<font color="gray">Игрок</font>');
  }
  if($myrow['clan']){
    $tpl->set_value('GUILD', $myrow['clan']);
  }else{
    $tpl->set_value('GUILD', 'Не состоит в гильдии');
  }
}else{
  echo 'Пользователь не выбран';
}

Можно еще конечно спорить про структуру базы, про текстовые ключи, которые лучше на числовые индексы заменить.

Answer 2

[Rsa97]

mysql_fetch_array() expects parameter 1 to be resource, boolean given
Значит в $result у вас false, то есть предыдущая команда mysql_ завершилась с ошибкой.

А вообще, не используйте функции mysql_, начиная с PHP7.0 они не поддерживаются. Переходите на PDO или mysqli_ с параметризованными запросами.