Mubeen Mubeen Rasheed Ответов: 3

Ошибка синтаксического анализа: синтаксическая ошибка, неожиданный '$SQL' (T_VARIABLE) в D:\xmp\htdocs\student\delete.php на линии 12

<?php
         
		 
		 
		 $con =mysqli_connect ("localhost", "root", "" );
         $db= mysqli_select_db ($con, "student");
		 
		
		
		 $id = (isset($_POST['id']);
		 
		 $sql = "DELETE *FROM user  WHERE 'id' = .$id";
		 $result = mysqli_query($con, $sql);
		   if  ($result)
			  
			 {
				  
				 echo "Data delete";
			  }
			  else
			  {
			  echo "Data not delete";
				  
			  }


?>


Что я уже пробовал:

таки не могу решить эту ошибку


Parse error: syntax error, unexpected '$sql' (T_VARIABLE) in D:\xmp\htdocs\student\delete.php on line 12

OriginalGriff

Удивительно трудно диагностировать синтаксические ошибки, не имея возможности видеть код ...

Отредактируйте свой вопрос (используйте виджет "улучшить вопрос") и добавьте соответствующий фрагмент кода:саму строку плюс дюжину строк выше и ниже, указывающих, на какой строке система находит проблему ... мы не можем видеть ваш экран, получить доступ к вашему жесткому диску или прочитать ваши мысли!

Mubeen Mubeen Rasheed









это мой Кодекс братана

3 Ответов

Рейтинг:
24

OriginalGriff

Скобки должны совпадать...

$id = (isset($_POST['id']);
Попробуй:
$id = (isset($_POST['id']));


Но... есть еще две вещи, на которые вам следует обратить внимание.
Один тривиальный: Сделайте правильный отступ в коде и избавьтесь от ложных разрывов строк! Это делает его намного легче читать, если вы это сделаете:
 	   if  ($result)
			  
			 {
				  
				 echo "Data delete";
			  }
			  else
			  {
			  echo "Data not delete";
				  
			  }
Против
 	   if  ($result)
			  {
				 echo "Data delete";
			  }
	   else
			  {
			  echo "Data not delete";
			  }

И еще один куда более серьезный: Никогда не объединяйте строки для построения SQL-команды. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого всегда используйте параметризованные запросы.

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:
SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

Mubeen Mubeen Rasheed

ОШИБКА
ОШИБКА
Ошибка синтаксического анализа: синтаксическая ошибка, неожиданный 'MyTable' (T_STRING) в C:\xampp\htdocs\student\delete.php на линии 12

OriginalGriff

Вы действительно читали то, что я сказал, или копировали и вставляли строку инъекции в свое приложение?

Richard Deeming

Если посмотреть на ответ в решении 4, то это будет "нет" для обоих. :)

OriginalGriff

О, Фер ... :до:

Рейтинг:
2

Fexxi

Да, нам нужно увидеть ваш код. Однако, похоже, вы пропустили букву"; " в строке 11.

Рейтинг:
2

Mubeen Mubeen Rasheed 

<?php
         
		 
		 
		 $con =mysqli_connect ("localhost", "root", "" );
         $db= mysqli_select_db ($con, "student");
		 
		
		$id = (isset($_POST['id']));

		
	     SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
		 $result = mysqli_query($con, $sql);
		    if  ($result)
			  {
				 echo "Data delete";
			  }
	   else
			  {
			  echo "Data not delete";
			  }


?>