Не вносит данные в базу?

Question

[invaest]

$sl возвращает false. Почему и как правильно? Сам никак не могу определить ,так как ошибок никаких не показывает при выполнении скрипта.

<?php
error_reporting(-1);
ini_set('display_errors', true);
include('bd.php');
 $data=$_POST['data'];
  $namef=$_POST['namef'];
   $tags=$_POST['tags'];
     $dir='/.upload/';
     @mkdir($dir,0777);
     if(isset($_FILES["upfile"]))
     {
	   $upfile=$_FILES["upfile"]["tmp_name"];
	   $upfile_name=$_FILES["upfile"]["name"];
	  // $upfile_size=$_FILES["upfile"]["size"];
	  // $upfile_type=$_FILES["upfile"]["type"];
	   // $upfile_code=$_FILES["upfile"]["error"];   
    $upfile_name = $dir.$upfile_name;
   copy($upfile,$upfile_name);
   }

$upfile_name=$_FILES["upfile"]["name"];
    $lnk = 'http://'.$_SERVER['HTTP_HOST'].'/.upload/'.$upfile_name; 

     $sl="INSERT INTO `ufiles` (`data`,`namef`,`tags`,`link`) VALUES('$data','$namef','$tags','$lnk')";
      if($sl=='true') 
        {echo "<p>В базу внесено!</p>";}
          else 
           {echo "<p>В базу не внесено!</p>";}
             echo $lnk;
?>

Answer 1

[1Michael1]

потому что:
1) строка "INSERT INTO `ufiles` (`data`,`namef`,`tags`,`link`) VALUES('$data','$namef','$tags','$lnk')" не равняется строке "true" и всегда будет выполнятся блок else
P.S. для того, чтоб проверить выполнение запроса надо сначала соединиться с БД (возможно это делаеться в bd.php) и потом выполнить запрос, а результат уже проверять и выводить все то, что нужно

Comments

[invaest]

спасибо, на будущее буду знать. Часа 4 сидел никак не мог разобраться

[1Michael1]

@invaest
пожалуйста.
небольшой совет:
надо очень усердно поработать над code style (взять, к примеру, psr-x стандарт) и над именованием переменных. так же советую почитать Clean Code Роберта Мартина

[mx2000]

плохой совет. хороший совет: выкинуть PHP и вернуться к изучению любого типизированного языка.

PS. У вас тут 100% SQL injection, а вы про какой-то code style рассуждаете.

[mx2000]

PPS. А еще injection на файловой системе. Просто анекдот.

[1Michael1]

@mx2000
плохому танцору всегда что-то мешает...
проблема не в том, что PHP плох, проблема в том, что его зачастую неумело используют по причине того, что начинают его учить, так как он сам по себе простой.
а дыры - это совсем другое дело. это никак не зависит от выбора языка. на java и c# такого немало встречал
да и скорость разработки на скриптовом языке в разы больше чем на любом типизированом

[mx2000]

@1Michael1 топикстартер не отличает boolean от string и вообще имеет крайне поверхностное представление о том, что он пишет в коде. Это лечится возвращением к азам на Паскале - языку, имеющему проверку типов при компиляции.

Вы, кстати, тоже немного не в теме: типизация и скриптовый язык не являются взаимоисключающими характеристиками. Гуглить, например, Python.

[1Michael1]

@mx2000
(facepalm)
1. я говорил о code style и именовании переменных. это приминимо в ЛЮБОМ языке программирования
2. паскаль давно умер. и паскаль - это не панацея. нормальные IDE говорят еще во время написания кода такие проблемы.
3. я не говорил про то, что типизация и скриптовый язык - две взаимоисключающие вещи. я говорил про скорость разработки.
в любом случае, как в типизированных языках, так и в слабо типизированых (или, как кто-то порой думает НЕтипизированых) можно проверять как с учетом типов данных, так и без.
пример на скриптовых языках:

Python:
print "100" < 2 # False
print 100 < "2" # True

PHP :
print "100" < 2 # False
print 100 < "2" # False

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

P.S. достаточно прочитать только то, что написано, а не додумывать то, чего нет

[invaest]

товарищ mx2000 прав я еще дико не опытен, но надо с чего то начинать а по поводу sql инъекций то передо мной пока не стоит задача писать защищенный код