Не могу понять, как код работает (PHP)

Question

[ernestby]

Делаю для себя (практикуюсь) бюро находок, сделал страницу добавления в БД данных
Теперь не могу вывести данные с БД правильно.

Пользователь вводит в полях Имя и Фамилию и категорию
Скрипт в зависимости от того, какая категория выбрана, вытаскиеваеи совпадающую строку
Но у меня сейчас ищет только по загран паспорту, а остальные elseif не работают

if (isset($_POST['select'], $_POST['first_name'], $_POST['last_name'])) {
	$select = $_POST['select'];
	$first_name = $_POST['first_name'];
	$last_name = $_POST['last_name'];
	
	

		if ($select = 'Загран. паспорт') {
				$result= mysql_query("SELECT * FROM zagran WHERE first_name='$first_name' AND last_name='$last_name'");
				$row = mysql_fetch_array($result);
				
				if ($first_name == $row['first_name'] AND $last_name == $row['last_name']) {
												
						echo $first_name = $row['first_name'];
						$last_name = $row['last_name'];
						$founder_name = $row['founder_name'];
						$phone_number = $row['phone_number'];
				}
				else { 
				echo "<center>Совпадений не найдено</center>";
				}
				
				}
		elseif 	($select = 'ID карта') {
				$result= mysql_query("SELECT * FROM idcarta WHERE first_name='$first_name' AND last_name='$last_name'");
				$row = mysql_fetch_array($result);
				
				if ($first_name == $row['first_name'] AND $last_name == $row['last_name']) {
												
						echo $first_name = $row['first_name'];
						$last_name = $row['last_name'];
						$founder_name = $row['founder_name'];
						$phone_number = $row['phone_number'];
				}
				else { 
				echo "<center>Совпадений не найдено</center>";
				}
				
				}	
		
		elseif 	($select = 'Водительское удостоверение') {
				$result= mysql_query("SELECT * FROM vodit WHERE first_name='$first_name' AND last_name='$last_name'");
				$row = mysql_fetch_array($result);
				
				if ($first_name == $row['first_name'] AND $last_name == $row['last_name']) {
												
						$first_name = $row['first_name'];
						$last_name = $row['last_name'];
						$founder_name  = $row['founder_name'];
						$phone_number = $row['phone_number'];
				}
				else { 
				echo "<center>Совпадений не найдено</center>";
				}
				
				}	
				
		elseif 	($select = 'Другой документ') {
				$result= mysql_query("SELECT * FROM other WHERE first_name='$first_name' AND last_name='$last_name'");
				$row = mysql_fetch_array($result);
				
				if ($first_name == $row['first_name'] AND $last_name == $row['last_name']) {
												
						$first_name = $row['first_name'];
						$last_name = $row['last_name'];
						$founder_name = $row['founder_name'];
						$phone_number = $row['phone_number'];
				}
				else { 
				$failed = "<center>Выберите категорию</center>";
				}
				
				}
			
}

Answer 1

[subalex]

Используйте == вместо =

Answer 2

[zenaku]

$select = $_POST['select'];
$first_name = $_POST['first_name'];
$last_name = $_POST['last_name'];

И еще, проверяй переменные во избежание XSS и SQL-инфекций.

Comments

[Константин]

us1.php.net/mysql_real_escape_string

[ernestby]

через isset?

[zenaku]

rutracker.org/forum/viewtopic.php?t=4623969
а если слишком долго и много, то
nnm-club.me/forum/viewtopic.php?t=550443
Но это кратко и понятно.
Также есть php.net/manual/ru/security.php

[zenaku]

@ernestby, нет. isset проверит наличие самого факта существования переменной. Но вот он не проверит содержимое этой переменной. Когда вы предполагаете, что там число, юзер может отправить строку(хотя, все что через формы - уже строка по типу). Или например в БД указано длину ячейки как 3-значное число, а юзер отправит 4-х значное. Но это мелочи, хоть и неприятные. Но вот когда к вам забежит юолее продвинутый и вредный человек, то с помощью POST он может отправить вам SQL код, который либо скопирует БД, или добавит себя в таблицу с правами доступа админа к сайту...и так далее.

[ernestby]

теорию щас читать и вникать долго) а не могли бы в коде мне добавить проверку?)

[zenaku]

Это зависит от того, что должно храниться в этих переменных и где эта переменная используется.
Посмотрите хоть этот курс nnm-club.me/forum/viewtopic.php?t=550443
Он всего 2:45 длится

[Facetrollex]

@Norraxx mysql_"any" - deprecated, сами не используйте и другим не советуйте.

[zenaku]

@Facetrollex ты полностью прав.

[Константин]

@zenaku @Facetrollex Извиняюсь, я уже долгое время как перешёл на другую платформу + на мобильнике был, так не особо вникал: cz1.php.net/mysqli_real_escape_string ;-)