PHPer

Starohosting · 23 Mei 2012

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');
?>

Toon Meer

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

jopitan · 23 Mei 2012

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'); 
?>

Toon Meer

NielsB · 24 Mei 2012

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 · 24 Mei 2012

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.

DNS update Directadmin

PC probleem

Let op Vimexx klanten

wie kan mij helpen met Security aanscherpen voor nmijn website

Ik ben op zoek naar een goede hosting

App bouwer gezocht

4G met USB Dongle - windows 11 Chrome of Firefox - browser probleem

Hosting en websites op maat

Xenforo buckaroo plugin

300+ Nieuwe domeinnamen Maart 2024

Actieve crime

PHPer

Participate now!

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Het Belang van Standaardisatie

De Revolutionaire Rol van IT in Alzheimeronderzoek

Functioneel ontwerp

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

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

Delen

Gebruikers die dit topic bekijken