2016-04-04 4 views
-2

Как я могу минимизировать этот PHP-код? Или написать это проще?PHP select и insert

$res = mysqli_query($mysqlc, "SELECT id FROM admin_szallas") or die(mysqli_error($mysqlc)); 
while($row = mysqli_fetch_array($res)) { 

    $szID = $row["id"]; 
    mysqli_query($mysqlc, "INSERT INTO admin_ertesitesek (szallasID, ido, uzenet, allapot, akcio) VALUES('$szID', '$ido', '$cim', '0', '$akcio')") or die(mysqli_error($mysqlc)); 

} 
+3

свести к минимуму, каким образом? –

+0

Я поставил свой код на форум, и они сказали, что это рабочий код, но очень уродливый –

+0

ну, красота в глазах смотрящего, не так ли? ;-) –

ответ

0

Люди могут видеть здесь различные проблемы.

  1. Coding style из представленного кода.
  2. Использование процедурных функций вместо объектно-ориентированных.
  3. Использование die() вместо Exceptions.

Если вы хотите защитить его от SQL Injection check out this topic.