PHP, uppladdningsskript

Jag var en gång i tiden riktigt vass på PHP, dock märkte jag i dag, när jag skulle göra ett uppladdningsskript, hur förbannat snabbt kunskaperna försvinner om man inte sysslar med det regelbundet.

Skulle någon kunna snygga upp den här otroligt fula PHP-koden som jag har knåpat ihop nu under kvällen? Det vore guld värt.

<?php
$upload_dir = 'uploaded_files/';

$chosen_file = $_FILES['chosen_file']['name'];

$file_type = explode('.', $chosen_file);
$file_type = $file_type[count($file_type)-1];
$allowed_file_types = array('png', 'jpg', 'jpeg');

$file_size = $_FILES['chosen_file']['size'];
$max_size = (1024*25000);

$submit = $_POST['upload_submit'];

$new_filename = rand(0, 9) . rand(0, 9) . rand(0, 9) . rand(0, 9) . '.' . $file_type;




if (isset($submit))
{

	if (empty($chosen_file))
	{

		$message = 'Du valde <strong>INTE</strong> en fil.';

	}

	else
	{

		if (empty($chosen_file))
		{

			$message = 'Filens filtyp kunde <strong>INTE</strong> utläsas.';

		}

		else
		{

			if (!in_array($file_type, $allowed_file_types))
			{

				$message = 'Filtypen ' . $file_type . ' är <strong>INTE</strong> tillåten.';

			}

			else
			{


				if ($file_size > $max_size)
				{

					$message = 'Filen är <strong>FÖR STOR</strong>.';

				}

				else
				{

					move_uploaded_file($_FILES['chosen_file']['tmp_name'], $upload_dir.$new_filename);

					$message = 'Filen är uppladdad och finns <a href="uploaded_files/' . $new_filename . '">här</a>!';

				}

			}

		}

	}
}

if (!empty($message))
{

	$message = '<div id="message_area">' . $message . '</div>' . "\n";

}
?>

Funderingar:

• Vad jag minns, från dagarna då jag faktiskt var någorlunda duktig på att koda, är att jag inte alls skrev en massa else-satser efter varandra på det här viset – utan då hade jag någon array, har jag för mig, som fylldes med värden utifall att filtypen inte accepterades, eller dylikt, och i slutändan visades alla meddelanden, ifall arrayen inte var tom givetvis. Någon som förstår vad jag syftar på, och kan demonstrera ett exempel?

• Vad skriver man för att slumpa fram en bokstav mellan a-zA-Z?

EDIT:
För att slumpa fram en bokstav mellan A-Z: chr(rand(65,90))
För att slumpa fram en bokstav mellan a-z: chr(rand(97,122))

Jag vet, därav sitter jag just nu och klurar på hur en while()-sats bör se ut. Den ska alltså kolla det framslumpade filnamnet och se om det redan existerar i mappen "uploaded_files", om ingen träff upptäcks kör den vidare, om den osannolikt nog hittar en träff gör den ett nytt filnamn.

Anledningen till while() istället för if() är ifall det, sjukt osannolikt nog, skulle bli krock två gånger om med filnamnen.

Några tips?

$s = hash('sha256', $filename);
while(file_exists($s)) {
    $s = hash('sha256', $filename);
}

Då folk har en tendens att döpa filer till exempelvis "asdf.png", kommer det väl inte att göra saken bättre för att man hashar det? Då är säkrare att döpa filen till den nuvarande tiden + millisekunder, om man ska vara på det viset. Men jag vill bestämt få utforma mina filnamn på detta viset och "rädda det" genom den här while()-satsen som jag nämnde tidigare – så att krockar och överskrivningar inte sker.

Minns dock inte hur den bör se ut.

$errors = array();
if(empty($variabel)){
$errors[] = 'Denna variabel är tom';
}

if(!empty($errors)){
foreach($errors as $error){
print($error.'<br />');
}

Tack, det där var till stor hjälp!

Det verkar dock som att $errors = array(); inte behöver anges, utan det fungerar väl även utan den raden.

Skulle du kunna förklara, kortfattat, vad skillnaden är mellan print() och echo() – samt varför det behövs en foreach() för att kunna skriva ut innehållet i arrayen på det här viset. Jag förstår mig inte alls på den logiken.

Jag märkte att med print() skriver den ut varje "array-post" efter varandra, medan med echo() skriver den ut hela arrayens innehåll på en och samma gång – vilket då inte gav möjlighet till att skriva ut exempelvis <br /> efter varje array-post. Vad skillnaden i helhet är förstår jag dock inte.

Tack så mycket för förklaringarna!

Är detta användningsområdet foreach()'s egentliga syfte? Det verkar nämligen vara så, efter att ha tagit en snabb titt på php-dokumentationen om det. Jag förstår mig nämligen inte inte riktigt på logiken med namnet "for each".

Fördelen med print, i detta fallet, är att jag, efter varje array-post, kan lägga till <br /> och "\n" om jag behagar. MEN resultatet från print() kan inte sparas i en variabel, utan det skrivs ut på direkten.

Hur ska jag då göra för att lägga till radbrytningar och dylikt efter varje array-port, när jag sätter ihop alla array-poster i en och samma variabel?

Med print() lades <br /> till efter varje array-post. Med echo() slängdes alla array-poster ihop och skrevs ut på en och samma gång, så det gick inte att trycka in <br /> mellan posterna. print() hade därav en fördel, nackdelen var dock att det inte gick att göra typ $hey = print($array . '<br />');, för print skrevs ut på direkten och gick inte att spara i en variabel.

$yo = print($helloworld);

Fördelen med print() var, som sagt, att man kunde trycka in <br /> efter varje array-post, nackdelen var dock att det inte gick att spara resultatet i en variabel då resultatet skrevs ut på direkten. Så hur jag ska göra vet jag inte.