Just nu i M3-nätverket
Jump to content

Funktions problem


sjohan

Recommended Posts

Hejsan.

I vanliga fall så är jag ganska så duktig på funktioner men i detta fallet så får jag det inte att fungera.. Jag vill lägga in följande i en funktion men jag lyckas inte. Någon som kan säga hur man ska göra?

 

<?

IF($_POST['ladda_upp']){

if($_FILES['filen'] != "" && $_FILES['filen'] != "none"){

$path = "images/album/".$_POST['kat']."/".$_FILES['filen']['name'];

copy($_FILES['filen']['tmp_name'],"$path");

echo "<p>Du laddade upp filen <b>".$_FILES['filen']['name']."</b>.<br>";

echo "Den är <b>".round(($_FILES['filen']['size']/1024),1)."</b> kB stor.";

}

}

?>

 

Tacksam för all hjälp och snäll med poängen

//Johan

 

Link to comment
Share on other sites

Du använder funktionen copy vilket du inte bör göra. Använd istället funktionen move_uploaded_file.

 

Detta bör fungera.

 

Själv hade jag heller inte skrivit ut någonting innuti funktionen eftersom det då känns lite dumt att använda en funktion över huvud taget. Returnera istället data som säger om det gick bra eller dåligt (1 för att det fungerade eller 0 om det inte fungerade till exempel). Alternativt att du returnerar den data som du nu skriver ut:

 

if ($_POST['ladda_upp']) {
echo upload();
} else {
echo "Vi kommer inte från formuläret! Ladda inte upp!";
}

function upload() {
$uploaddir = "files/";

if ($_FILES['userfile'] != "" && $_FILES['userfile'] != "none"){
	move_uploaded_file($_FILES['userfile']['tmp_name'], $uploaddir . $_FILES['userfile']['name']);
	$returnMsg = "Du laddade upp filen <b>".$_FILES['userfile']['name']."</b>.<br>";
	$returnMsg .= "Den är <b>".round(($_FILES['userfile']['size']/1024),1)."</b> kB stor.";
} else {
	$returnMsg = "Filen fanns inte?";
}
return $returnMsg;
}

[redigerat]Ojdå, man kanske måste returnera värdet från funktionen oxå...[/redigerat]

 

--

Some say I am crazy.. but the Voices in my head disagree...

 

[inlägget ändrat 2003-04-15 13:17:55 av Bj0rN]

Link to comment
Share on other sites

Archived

This topic is now archived and is closed to further replies.



×
×
  • Create New...