php - SQL syntax - Udvikleren.dk - Programmering, webdesign og grafik

Tags: mysql php

<< < 1 2 3 > >>

Bruger #17534 @ 30.11.13 18:48

Jeg er i gang med et nyhedssystem, hvor det er muligt for brugeren at kunne loade nogle billeder med. Problemet er bare, at den ikke smider billederne ind i databasen eller folderen, men den skriver fint nyheden ind.

Der hvor jeg er kommet frem til at fejlen må være er her (fejl fundet ved hjælp af die(print_r($files)):

PHP kode

 $query = ("INSERT INTO news (headline, bodytext, date) VALUES ('{$headline}', '{$bodytext}', now())") or die(mysql_error());
                         $result = mysql_query($query, $connection);
                             if ($result) {
                                     // Gem id på nyhed i en variabel til brug senere
                                     $newsid = mysql_insert_id();
                                     // Indsæt filerne nu hvor nyheden er oprettet
                                     foreach($files as $file);
                                     {
                                         mysql_query("INSERT INTO newsimg (newsID, images) VALUES ('{$newsid}', '{$file}'") or die(mysql_error());
                                     }
                                     // Done
                                     echo "Nyhed oprettet.";
                             }
                             else {

Jeg modtager denne "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1"

Jeg har nu ledt og ledt og ledt. Men jeg kan simpelthen ikke finde ud af, hvad det er der er galt. Nogen der kan hjælpe? :s

26 svar postet i denne tråd vises herunder
2 indlæg har modtaget i alt 8 karma

Sorter efter stemmer Sorter efter dato

Bruger #17534 @ 02.12.13 14:46

-1

Utroligt nok, virker det endnu ikke.
Den loader endnu ikke billederne op i newsmig folderen på serveren, og den skriver hellere ikke billede navn/adresse ind i tabellen.

På nuværende tidspunkt ser scriptet således ud.

PHP kode

 <?php
     $headline = $_POST['headline']; 
     $bodytext = $_POST['bodytext'];
     $archive = $_POST['archive'];
     $files = array(); 
     $error = ""; 
      
     if(isset($_POST['submit']))
     { 
             if($headline == ""){
                     $error .="Du skal angrive en overskift.<br />";
             } 
             if($bodytext == ""){
                     $error .="Du kan ikke indsende en tom nyhed.<br />";
             }
      
             // Er alt ok fra formularen?
             if ($error == "")
             {
                     // Da vi uploader flere end én fil bliver vi nødt til at lave en løkke for at køre dem alle igennem
                     // $i indikerer hvilken fil vi arbejder med
                     $antal_i_input_file_array = 5;
                     for($i = 0; $i < 5; $i++)
                     {
                         // Er der en fil uploadet? 
                         if (! file_exists($_FILES["file"]["tmp_name"][$i])) {
                         // Fortsæt til næste løkke 
                         continue; 
                         }
                             $filenameArray = explode(".", $_FILES['file']["name"][$i]);
                  
                     // Hiv fat i sidste ende af filnavnet (feks png)
                     $extension = end($filenameArray);
                  
                     // Angiv nyt unikt filnavn så filerne ikke overskriver hinanden
                     $newfile = uniqid() . "." . $extension;
                  
                     // Mappen filen skal ligge i
                     $folder = 'newsimg/';
                  
                     // Tilladte filtyper
                     $allowedTypes = array('image/gif', 'image/jpeg', 'image/jpg', 'image/pjpeg', 'image/x-png', 'image/png');
                  
                     // Er filen et billede? (baseret på filtyperne foroven)
                     if ( ! in_array($_FILES['file']["type"][$i], $allowedTypes))
                     {
                         $error = "Du må kun uploade billeder";
                     }
                     else
                     {
                         // Er filen for stor? Maks 2MB
                         if (($_FILES['file']["size"][$i] > 5097152))
                         {
                             $error = "Filen er for stor. Maks 5MB";
                         }
                         else
                         {
                             // Er der en fejl ved upload?
                             if ($_FILES["file"]["error"][$i] > 0)
                             {
                                 $error = "Fejl ved upload: " . $_FILES["file"]["error"][$i];
                             }
                             else
                             {
                                 // Kan vi gemme den nye fil? (@ gør at funktionen ikke udskriver en fejl på siden, hvis der er en fejl)
                                 if (@move_uploaded_file($_FILES["file"]["tmp_name"][$i], $folder . $newfile) )
                                 {
                                     $files[] = $newfile;
                                 }
                                 else
                                 {
                                     $error = "Kunne ikke uploade filen. Kontakt admin";
                                     break; // Stop løkken
                                 }
                  
                             }
                         }
                     }
                     }
      
                     // Er alt ok ved filerne?
                     if ($error == "")
                     {
                         $query = "INSERT INTO news (userID, headline, bodytext, archive, date) VALUES ('{$_SESSION['user_id']}', '{$headline}', '{$bodytext}', '{$archive}', NOW())";
 $result = mysql_query($query, $connection) or die(mysql_error()); 
                             if ($result) {
                                     // Gem id på nyhed i en variabel til brug senere
                                     $newsid = mysql_insert_id();
                                     // Indsæt filerne nu hvor nyheden er oprettet
                                     foreach($files as $file) die(print_r($files));
                                     {
                                         $query2 = "INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file')";
                                         mysql_query($query2, $connection) or die(mysql_error());
                                         }
                                     // Done
                                     echo "Nyhed oprettet.";
                             }
                             else {
                                     $error = "Noget gik galt, nyhed ikke oprettet. Prøv igen.";
                                     $error .= "<br />" . mysql_error();
                             }
                     }
             }
     }
     if (!empty($error)) {echo "<p>" . $error . "</p>";}
      
     ?>

Bruger #16075 @ 02.12.13 17:28

1.223

se din form sådan ud ?? (det skal den)

Kode

<form action="" method="post" enctype="multipart/form-data">
    Overskift:<br/>
    <input type="text" name="headline" maxlength="30" value="<?php echo $headline; ?>" /><br/>
    Tekst:<br/>
    <textarea name="bodytext" rows="9" cols="40"><?php echo $bodytext; ?></textarea><br/>
    Billeder:<br/>
    <input name="file[]" type="file" /><br />
    <input name="file[]" type="file" /><br />
    <input name="file[]" type="file" /><br />
    <input name="file[]" type="file" /><br />
    <input name="file[]" type="file" /><br />
    <input type="submit" name="submit" value="Opret nyhed" />
    </form>

læg mærke til name="file[] det er det du spørger på i $_FILES["file"]

hvor er input til archive, skal også ligge i din form, den har du lagt på efterfølgende

hvorfor har du die på linje 90 ??, så vil du aldrig komme vidre
foreach($files as $file) die(print_r($files));
skal se sådan ud
foreach($files as $file)

Bruger #17534 @ 02.12.13 20:33

-1

Formen ser sådan ud ja.

die(print_r($files)); har jeg taget med, for at se hvorfor den ikke loader billederne op.

Bruger #10216 @ 02.12.13 20:46

4.283

Jeg tror din move_uploaded_file() fejler.

Du har lidt for mange niveauer af betingelser. Det gør det svært at læse og fejlsøge. Der er også tilføjet lidt ekstra error handling. Prøv at se om du kommer videre fra denne.

PHP kode

 <?php
 $headline = $_POST['headline'];
 $bodytext = $_POST['bodytext'];
 $archive = $_POST['archive'];
 $files = array();
 $error = "";
 
 if (isset($_POST['submit'])) {
     if ($headline == "") {
         $error .= "Du skal angrive en overskift.<br />";
     }
     if ($bodytext == "") {
         $error .= "Du kan ikke indsende en tom nyhed.<br />";
     }
     
     // Er alt ok fra formularen?
     if ($error == "") {
         // Da vi uploader flere end én fil bliver vi nødt til at lave en løkke
         // for at køre dem alle igennem
         // $i indikerer hvilken fil vi arbejder med
         $antal_i_input_file_array = 5;
         for($i = 0; $i < 5; $i++) {
             // Er der en fil uploadet?
             if (! file_exists($_FILES["file"]["tmp_name"][$i])) {
                 // Fortsæt til næste løkke
                 continue;
             }
             $filenameArray = explode(".", $_FILES['file']["name"][$i]);
             
             // Hiv fat i sidste ende af filnavnet (feks png)
             $extension = end($filenameArray);
             
             // Angiv nyt unikt filnavn så filerne ikke overskriver hinanden
             $newfile = uniqid() . "." . $extension;
             
             // Mappen filen skal ligge i
             $folder = 'newsimg/';
             
             // Tilladte filtyper
             $allowedTypes = array('image/gif', 'image/jpeg', 'image/jpg', 'image/pjpeg', 'image/x-png', 'image/png');
             
             if(!is_dir($folder)) {
                 $error = '"'.$folder.'" er eksisterer ikke - prøv den absolutte sti sti';
             }
             
             // Er filen et billede? (baseret på filtyperne foroven)
             if (! in_array($_FILES['file']["type"][$i], $allowedTypes)) {
                 $error = "Du må kun uploade billeder";
             } elseif (($_FILES['file']["size"][$i] > 5097152)) { // Er filen for stor? Maks 2MB
                 $error = "Filen er for stor. Maks 5MB";
             } elseif ($_FILES["file"]["error"][$i] > 0) { // Er der en fejl ved upload?
                 $error = "Fejl ved upload: " . $_FILES["file"]["error"][$i];
             } else if (@move_uploaded_file($_FILES["file"]["tmp_name"][$i], $folder . $newfile)) { // Kan vi gemme den nye fil? (@ gør at funktionen ikke udskriver en fejl på siden, hvis der er en fejl)
                 $files[] = $newfile;
             } else {
                 $error = "Kunne ikke uploade filen. Kontakt admin";
                 break; // Stop løkken
             }
         }
         
         if(empty($files)) {
             $error = 'Ingen filer';
         }
         
         // Er alt ok ved filerne?
         if ($error == "") {
             $query = "INSERT INTO news (userID, headline, bodytext, archive, date) VALUES ('{$_SESSION['user_id']}', '{$headline}', '{$bodytext}', '{$archive}', NOW())";
             $result = mysql_query($query, $connection) or die(mysql_error());
             if ($result) {
                 // Gem id på nyhed i en variabel til brug senere
                 $newsid = mysql_insert_id();
                 // Indsæt filerne nu hvor nyheden er oprettet
                 foreach($files as $file) {
                     $query = "INSERT INTO newsimg (newsID, images) VALUES ('$newsid', '$file')";
                     mysql_query($query, $connection) or die(mysql_error());
                 }
                 // Done
                 echo "Nyhed oprettet.";
             } else {
                 $error = "Noget gik galt, nyhed ikke oprettet. Prøv igen.";
                 $error .= "<br />" . mysql_error();
             }
         }
     }
 }
 if (! empty($error)) {
     echo "<p>" . $error . "</p>";
 }
 
 ?>

Bruger #17534 @ 10.12.13 20:24

-1

Jeg prøver

Men venter lige til jeg har styr på mysqli nu, har jeg besluttet.

Bruger #16075 @ 15.12.13 10:57

1.223

udemærket beslutning
og slå så lige din fejl visning til, og fjerne din undertrykning af fejl, fjern @ i
if (@move_uploaded_file($_FILES[

kontroller display_error og error_reporting er sat til i php.ini
display_errors = On
error_reporting = E_ALL | E_STRICT

enten passer din mappe structur ikke med det du angiver eller din sql passer ikke med dine tabeller

<< < 1 2 3 > >>

SQL syntax

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler