12. juni 2003 - 20:36
Der er
22 kommentarer og 1 løsning
Jeg har stirret mig blind!
Jeg har vist stirret mig blind på dette script! jeg er helt sikker på det går galt i php som omdøber og flytter billedet!? Det kommer ikke med nogen fejl andet end den ikke indsætter noget i DB'n! Er der en som gidder tjekke det igennem <?php $HTTP_GET_VARS = $_GET; $HTTP_POST_VARS = $_POST; $HTTP_FILES_VARS = $_FILES; $HTTP_SERVER_VARS = $_SERVER; extract($_GET); extract($_POST); extract($_FILES); extract($_SERVER); if (!empty($_FILES['upfile']['name'])) { if(is_uploaded_file($HTTP_POST_FILES['upfile']['tmp_name'])) { $dist = $HTTP_SERVER_VARS['DOCUMENT_ROOT']."/user_system/img/"; $billed = $HTTP_POST_FILES['upfile']['name']; $user_group = $_POST['user_group']; $user_name = $_POST['user_name']; $password = $_POST['password']; $first_name = $_POST['first_name']; $last_name = $_POST['last_name']; $email = $_POST['email']; $filnavn = $_FILES['upfile']['name']; $temparr = explode(".", $filnavn); $temparr[count($temparr)-2] = $user_name; $filnavn = implode(".", $temparr); mysql_connect("","",""); mysql_select_db(""); $tjek = mysql_query("SELECT * FROM user_system WHERE user_name= '$_POST[user_name]'"); if (mysql_num_rows($tjek)) { echo "user_name: <b>$user_name</b>, findes allerede!"; exit; }else{ mysql_query("INSERT INTO user_system (user_group, user_name, password, first_name, last_name, email, picture) VALUES ('$user_group', '$user_name', '$password', '$first_name', '$last_name', '$email', '$picture')") or die(mysql_error()); move_uploaded_file($HTTP_POST_FILES['upfile']['tmp_name'], $dist.$filnavn); $besked = "bruger blev tilføjet!"; }}else{ $besked = "Det gik helt galt!"; } }else{ $besked = "Tilføj ny bruger"; } ?>
Annonceindlæg tema
Cloud & AI
I dette særtema ser vi på, hvordan cloud og AI bliver fundamentet for virksomhedernes digitale forretning, og hvordan de nye muligheder for automatisering og forretningsværdi kan udnyttes uden at miste overblik, sikkerhed og menneskelig kontrol.
Hvad er... move_uploaded_file
eller? if(is_uploaded_file($HTTP_POST_FILES['upfile']['tmp_name'])) { is_uploaded_file
det er meningen at den skal omdøbe filnavnet til user_name!
Ser forkert ud... move_uploaded_file og is_uploaded_file
okay fandt en fejl selv: $billed = $HTTP_POST_FILES['upfile']['name']; er ændret til $picture = $HTTP_POST_FILES['upfile']['name']; men den omdøber ikke filen???
Den skal overskrive - ikk?
move_uploaded_file($HTTP_POST_FILES['upfile']['tmp_name'], $dist.$filnavn); skal måske være move_uploaded_file($HTTP_POST_FILES['upfile']['name'], $dist.$filnavn);
er ikke helt sikker på hvad du mener?
12. juni 2003 - 20:50
#10
mener du skal bruge copy() istedet for move_uploaded_file()... mjl => is_uploaded_file findes, den tjekker om det er en uploaded fil, så evt hackerangab kan overleves Morten
12. juni 2003 - 20:51
#11
ok
12. juni 2003 - 20:52
#12
Det er forøvrigt nogle lidt grimme høkerløsninger du bruger med dine variable i starten af scriptet.
12. juni 2003 - 20:53
#13
Jeg er en krejler...! ;0P
12. juni 2003 - 20:59
#15
12. juni 2003 - 21:05
#16
he he, jeg tager kritikken til mig som en mand... ;)
12. juni 2003 - 21:33
#17
_darkstar_ => god og gennemtænkt artikel... ting til eftertanke (især magic_numbers... også meget fedt ex) Morten
12. juni 2003 - 22:01
#18
Jeg er selv noget af en stil-fascist, når det gælder kode. For mig er det svært at leve uden.
12. juni 2003 - 22:07
#19
if(22 == $num) { start_thermonuclearwar(); } // overdrevent? pjat... ;) Morten
12. juni 2003 - 22:24
#20
Jeg har designet betalingssystemer. I den sammenhæng er det bestemt ikke noget pjat :)
12. juni 2003 - 22:26
#21
if($return_code = 632321320) { send_pengene();} if($return_code = 632321230) { ring_til_politiet_han_snyder(); }
12. juni 2003 - 22:27
#22
det var til start_thermonuclearwar(); hvad med bare something();... men nej ;) den kolde krig var ikke forgæves Morten
12. juni 2003 - 22:35
#23
something() ikke alle til at fange pointen. Desværre. Overdrivelse fremmer forståelsen.
Vi tilbyder markedets bedste kurser inden for webudvikling