PHPer

Starohosting

Beste leden van CP,

Ik heb een opdracht voor school en heb een probleempje met mijn script.
Hij upload het bestand niet naar de map.
En in de database komt array te staan.

DIt is het script:

PHP

<?PHP
include ('header.php');
?>


<?PHP
$tabelnaam = 'gegevens';
if(isset($_POST['verzenden'])) {
$Naam = ($_POST['Naam']);
$Vervoerder = ($_POST['Vervoerder']);
$Bouwjaar = ($_POST['Bouwjaar']);
$Plaats = ($_POST['Plaats']);
$Beschrijving = ($_POST['Beschrijving']);
$Bestand = $_FILES['Bestand'];
$tijdelijk = $Bestand['tmp_name'];
$pad = "../database/";


if(empty($_FILES['Bestand']['name']) )
{
  echo "U heeft geen Foto ingevult.</br>";
}


if(empty($_POST['Naam']) )
{
  echo "U heeft geen Naam ingevult.</br>";
}


if(empty($_POST['Vervoerder']) )
{
  echo "U heeft geen Vervoerder ingevult.</br>";
}


if(empty($_POST['Bouwjaar']) )
{
  echo "U heeft geen Bouwjaar ingevult.</br>";
}


if(empty($_POST['Plaats']) )
{
  echo "U heeft geen Plaats ingevult.</br>";
}


if(empty($_POST['Beschrijving']) )
{
  echo "U heeft geen Beschrijving ingevult.</br>";


  } else {




$result = "INSERT INTO $tabelnaam (Naam, Vervoerder, Bouwjaar, Plaats, Beschrijving,Bestand) values ('$Naam','$Vervoerder','$Bouwjaar','$Plaats', '$Beschrijving', '$Bestand')"; 
if (mysql_query($result)) {
echo mysql_error();
}


if(move_uploaded_file($tijdelijk, $pad.$Naam )){


echo "




		<div id='content'> 
		<div class='titel'>Gegevens geslaagd</div> 
		<p>U heeft een overzichht aangemaakt en is verzonden! <a href='admin_gegevens.php'>Voer nog meer gegevens in</a></p> 
		</div>
		</div>
		";


} else {




echo "
		<div id='content'> 
		<div class='titel'>Gegevens niet verwerkt</div> 
		<p>het aanmaken is mislukt! <a href='gegevens.php'>Ga terug naar de homepagina</a></p> 
		</div>
		</div>
		";


}
}
}
else
{
?> 






<?php if(ingelogd('id', 'wachtwoord', 'hash') == TRUE){ ?>




		<div id='content'> 
		<div class='titel'>Admin Uploader</div> 
		<p>
		
<form name="verwerken" action="admin_gegevens.php" method='post' enctype='multipart/form-data'>
  <p>
  <table>
  <tr>
  <td width=330px;> Bestand: </td>
  <td width=330px;> <label for="Bestand"></label><input type="file" name="Bestand" id="Bestand"></td>
  </tr>


  <tr>
  <td width=330px;> Naam: </td>
  <td width=330px;> <input name='Naam' type='text'> </td>
  </tr>
  
  <tr>
  <td width=330px;> Vervoerder: </td>
  <td width=330px;> <input name='Vervoerder' type='text'> </td>
  </tr>
  
  <tr>
  <td width=330px;> Bouwjaar: </td>
  <td width=330px;> <input name='Bouwjaar' type='text'> </td>
  </tr>
  
  <tr>
  <td width=330px;> Plaats: </td>
  <td width=330px;> <input name='Plaats' type='text'> </td>
  </tr>
  
   
  <tr>
  <td width=330px;> Beschrijving: </td>
  <td width=330px;> <input name='Beschrijving' type='text'> </td>
  </tr>
  
  
  <tr>
  <td width=330px;> Verzenden: </td>
  <td width=330px;> <input name='verzenden' type='submit' value='verzenden'> </td>
  </tr>
  </table>
  </form>      


	
		</p> 
		</div>
		</div>
		
		
		
		
		
		
		
		
		
		
		
		
		
		
		<?PHP } else { ?>


		<div id='content'> 
		<div class='titel'>Niet ingelogd</div> 
		<p>
		
		U dient voor deze pagina ingelogd te zijn.
	
		</p> 
		</div>
		</div>
		<?PHP
		}
		}
		?>


<?php
include ('footer.php');
?>

Meer tonen

Wat doe ik verkeerd het is een klein foutje maar wat kan het zijn?

jopitan

Ik weet niet zeker of dit werkt, heb het even overnieuw geschreven. Heb het niet kunnen testen namelijk

PHP

<?php
	error_reporting(E_ALL);
	ini_set('display_errors', '1');


	include ('header.php'); 
	
	$_table = 'gegevens'; 
	if(isset($_POST['verzenden'])) { 
		isset($_POST['Naam']) ? $naam = $_POST['naam'] : $naam = '';
		isset($_POST['Vervoerder']) ? $vervoerder = $_POST['vervoerder'] : $vervoerder = '';
		isset($_POST['Bouwjaar']) ? $bouwjaar = $_POST['Bouwjaar'] : $bouwjaar = '';
		isset($_POST['Plaats']) ? $plaats = $_POST['Plaats'] : $plaats = '';
		isset($_POST['Beschrijving']) ? $beschrijving = $_POST['Beschrijving'] : $beschrijving = '';
		isset($_FILES['Bestand']) ? $bestand = $_FILES['Bestand'] : $bestand = '';
		$_toegestaneTypes = array("image/gif", "image/jpeg", "image/pjpeg");
		$_pad = "../database/";
		
		if($bestand["error"] > 0) {
			echo 'Error: ' . $bestand["error"];
		} elseif(!in_array($bestand["type"], $_toegestaneTypes)) {
			echo 'Er is een ongeldige type bestand geupload.';
		} elseif(empty($naam)) {
			echo 'U heeft geen naam ingevuld.';
		} elseif(empty($vervoerder)) {
			echo 'U heeft geen vervoerder ingevuld.';
		} elseif(empty($bouwjaar)) {
			echo 'U heeft geen bouwjaar ingevuld.';
		} elseif(empty($plaats)) {
			echo 'U heeft geen plaats ingevuld.';
		} elseif(empty($beschrijving)) {
			echo 'U heeft geen beschrijving ingevuld.';
		} elseif(file_exists($_pad . $bestand["name"])) {
			echo 'Bestand ' . $bestand["name"] . ' bestaat al.';
		} elseif(empty($bestand["tmp_name"])) {
			echo 'Er is een probleem ontstaan tijdens het uploaden van het probleem. Probeer het later nog eens.';
		} else {	
			if(move_uploaded_file($bestand["tmp_name"], $_pad . $bestand["name"])) {
				$_sql = "INSERT INTO `$_table` (`Naam`,`Vervoerder`,`Bouwjaar`,`Plaats`,`Beschrijving`,`Bestand`) values('" . mysql_real_escape_string($naam) . "','" . mysql_real_escape_string($vervoerder) . "','" . mysql_real_escape_string($bouwjaar) . "','" . mysql_real_escape_string($plaats) . "','" . mysql_real_escape_string($beschrijving) . "','" . mysql_real_escape_string($_pad . $bestand["name"]) . "')";
				mysql_query($_sql) or die(mysql_error());
				echo "<div id='content'>  
							<div class='titel'>Gegevens geslaagd</div>  
								<p>U heeft een overzichht aangemaakt en is verzonden! <a href='admin_gegevens.php'>Voer nog meer gegevens in</a></p>  
							</div> 
						</div>"; 
			} else {
				echo "<div id='content'>  
							<div class='titel'>Gegevens niet verwerkt</div>  
								<p>het aanmaken is mislukt! <a href='gegevens.php'>Ga terug naar de homepagina</a></p>  
							</div> 
						</div>"; 
			}
		}
	} else { 
		if(ingelogd('id', 'wachtwoord', 'hash') === true) {
			echo '<div id="content">  
				<div class="titel">Admin Uploader</div>  
				<p> 
					<form name="verwerken" action="admin_gegevens.php" method="post" enctype="multipart/form-data"> 
						<table> 
							<tr> 
								<td width=330px;> Bestand: </td> 
								<td width=330px;> <label for="Bestand"></label><input type="file" name="Bestand" id="Bestand"></td> 
							</tr>
							<tr> 
								<td width=330px;> Naam: </td> 
								<td width=330px;> <input name="Naam" type="text"> </td> 
							</tr> 
							<tr> 
								<td width=330px;> Vervoerder: </td> 
								<td width=330px;> <input name="Vervoerder" type="text"> </td> 
							</tr> 
							<tr> 
								<td width=330px;> Bouwjaar: </td> 
								<td width=330px;> <input name="Bouwjaar" type="text"> </td> 
							</tr> 
							<tr> 
								<td width=330px;> Plaats: </td> 
								<td width=330px;> <input name="Plaats" type="text"> </td> 
							</tr> 
							<tr> 
								<td width=330px;> Beschrijving: </td> 
								<td width=330px;> <input name="Beschrijving" type="text"> </td> 
							</tr> 
							<tr> 
								<td width=330px;> Verzenden: </td> 
								<td width=330px;> <input name="verzenden" type="submit" value="verzenden"> </td> 
							</tr> 
						</table> 
					</form>       
				</p>  
			</div> 
			</div>';
			} else {
				echo '
				<div id="content">  
					<div class="titel">Niet ingelogd</div>  
					<p> 
						U dient voor deze pagina ingelogd te zijn. 
					</p>  
				</div> 
				</div>';
			} 
	} 
	
	include('footer.php'); 
?>

Meer tonen

NielsB

Jopitan, dit stukje code vind ik beetje overbodig:

PHP

isset($_FILES['Bestand']) ? $bestand = $_FILES['Bestand'] : $bestand = '';

Schrijf het dan teminste om het manier dat je maar 1 keer variabel aanmaak en dan vult met waarde1 of waarde2

Dus:

PHP

$bestand = isset($_FILES['Bestand']) ? $_FILES['Bestand'] : '';

En zo geldt dat ook voor die andere waardes.

Voor de rest je kijkt of die gevuld is en zo ja laat die hem staan en anders vul je hem met niks. Kan je beter daar gelijk een if en else constructie om heen zetten, aangezien je nu dubbele check doet.

PHP

if(empty($naam) || !isset($naam) {

In je database zet je een tmp_name in en niet een name en een pad van het bestand, hierdoor krijg je de verkeerde waardes te zien.

Voorbeeld voor het uploaden:

PHP

$uploads_dir = '/uploads';
foreach ($_FILES["Bestand"]["error"] as $key => $error) {
    if ($error == UPLOAD_ERR_OK) {
        $tmp_name = $_FILES["Bestand"]["tmp_name"][$key];
        $name = $_FILES["Bestand"]["name"][$key];
        move_uploaded_file($tmp_name, "$uploads_dir/$name");
    }
}

Eerst wordt heel het bestand geopend en doorgelopen, hij zoekt hier naar een error, wanneer deze ok staat zal hij hem uploaden naar de map met het bestand. Je kan dit ook testen zonder je database te gebruiken, zodat je weet of het werkt.

Jopitan geeft ook slim in zijn code de mysql pas vrij als er geupload is, dit is dan ook zeker een tip om te gebruiken .

jopitan

Dat laatste stukje is alleen voor als je meerdere bestanden gaat uploaden, wat hier niet het geval is. Als je toch meerdere bestanden zou willen uploaden dan moet je ook in de foreach een query uitvoeren met het toevoegen van een rij.
Of je moet de bestanden in een losse tabel zetten die gekoppeld staan met de rij met de namen enzov. Dit voorkomt dan dubbele rijen met dezelfde naam enzovoort.

PHP

if(empty($naam) || !isset($naam) {

kan gewoon

PHP

if(empty($naam)) {

zijn aangezien je op het begin al checkt of hij wordt geset of niet. Als dit niet zo is wordt hij standaard empty ingevuld.

ICTscripters

Dé plek voor IT

Dé plek voor IT

[Beta] Ottomaan: gratis browserspel met een echte, levende stad. Feedback gezocht!

Port Father: beta van een browsergame in Antwerpen 1931, feedback welkom

Maffia Hotel: een browsergame waarin je van lobbygast naar spil van het hotel groeit

Crime Club volledig herschreven vanaf nul

Malavita (nieuwe maffia game, van 0)

Herkent deze Notion-structuurscan echte problemen?

🚀 WE ZOEKEN TESTERS! 🚀

Luchtvaartsoftware: Hoe technologie de moderne luchtvaart verandert

[Aanbod] Stadskaart voor je browsergame: van menu naar klikbare stad, vanaf €200

[Verkoop] G4G (Gangsters 4 Gangsters): volledige broncode, nieuw gebouwd in PHP 8.3 | € 30 per domein

[TE KOOP] Malavita.be [Maffiaspel, gebaseerd op omicidio.nl, unieke code]

PHPer

Registratie

PHP waarschuwt voor twee ernstige bugs die makkelijk uit te buiten lijken

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Gebruikers die dit topic bekijken