Avatar billede mikze Nybegynder
06. juni 2004 - 11:52 Der er 8 kommentarer og
1 løsning

Parse error

Hvad er der galt med denne kode?:

<?
include "../inc/db.php";
include "../inc/config.php";
?>
<html>
<head>
<title>Ns Replay</title>
<LINK REL="STYLESHEET" HREF="../inc/style.css" TYPE="text/css">
</head>
<body scroll="auto">
<?

$host="db.mikze.dk";
$user="web13697";
$pw="8fs529dm88";
$db="web13697";

/* connect db */

mysql_connect($host,$user,$pw);
mysql_select_db($db);


$brugernavn = $_POST[brugernavn];
$kode =  $_POST[kode];
$kon = $_POST[kon];
$email = $_POST[email];

if($_POST[submit]) {

  //Tjekker om de 2 koder er ens og at alle felterne er udfyldt
  if(!$brugernavn) {echo "Brugernavn er ikke udfyldt. Gå venligst tilbage <a href='opret.php'>HER</a>, og udfyld alle felterne.";
  exit;
}
  if(!$kode) {echo "Kode er ikke udfyldt. Gå venligst tilbage <a href='opret.php'>HER</a>, og udfyld alle felterne.";
  exit;
}
  if($kon == "Vælg...") {echo "Du har ikke valgt dit køn. Gå venligst tilbage <a href='opret.php'>HER</a>, og udfyld alle felterne.";
  exit;
}
  if(!$email) {echo "Mail er ikke udfyldt. Gå venligst tilbage <a href='opret.php'>HER</a>, og udfyld alle felterne.";
  exit;
}
else{


  //Tjekker om brugernavnet allerede findes
  $result = mysql_query("select brugernavn from brugere where brugernavn = '$_POST[brugernavn]'") or die(mysql_error());
  $number = mysql_num_rows($result);
  if ($number > 0) {
  echo "Desværre brugernavnet er optaget. Gå tilbage <a href='opret.php'>HER</a> og vælg et andet.";
  }
  else{



      $status = "bruger";
      $dato = date('d-m-Y H:i:s');
      $aktiveret = "nej";
      $ip = "$_SERVER[REMOTE_ADDR]";
      mysql_query("INSERT INTO brugere (brugernavn, kode, status, email, ip, aktiveret, oprettet, kon) VALUES ('$brugernavn', '$kode', '$status', '$email', '$ip', '$aktiveret', '$dato', '$kon')") OR DIE(mysql_error());
      echo "Din bruger er nu oprettet. Log ind i bunden af sitet.";}


}
} else {
  echo "Der opstod en fejl, prøv igen!";
}

?>
</body>
</html>






Den melder dette som fejl:


Parse error: parse error, unexpected T_VARIABLE in /home/web/web13697/nsr-paaskeharen2/community/doopret.php on line 52
Avatar billede mikze Nybegynder
06. juni 2004 - 11:54 #1
$number = mysql_num_rows($result);

Det har jeg talt som linje 52 men er ikke sikker ;P
Avatar billede fangel Nybegynder
06. juni 2004 - 12:00 #2
*host* det er pænt dumt at skrive kodeordet til ens webhotel...
Avatar billede fangel Nybegynder
06. juni 2004 - 12:11 #3
hvis det der er al din kode så er den linie du skriver kun linie 49...

og hverken i linie 49 eller 52 synes jeg at kunne se nogle synlige fejl - omend din kodestil er relativ dårlig (tag det ikke ilde op)

et par retningslinier
  -  brug altid string-padding til array-keys fx $array['tekst'], da der ellers kommer NOTICE: undefined constant, assuming "..."

  -  tjek snare om ting er opfyldt, end at de ikke er opfyldt... (dine if(!$brugernavn)), for som det er nu vil det muligvis være muligt at omgå dem...

  -  tjek altid om variabler findes med if(isset($variabel)) istedet for at tjekke om sand/falsk med if($variabel) - ellers vil du få en NOTICE: undefined variable

  -  få mere styr på indent/indrykning - som det er nu er det ret umuligt at se hvilke {'er og }'er der hører sammen...

det lydder nok som noget pedantisk sortsnak, men tro mig, det hjælper - og hvis du skulle få en server der kørte med error_reporting = E_ALL, vil du få en hulens mængde NOTICE's... ikke fedt...

Morten
Avatar billede mikze Nybegynder
07. juni 2004 - 16:22 #4
Øhmn... Det forstod jeg ikke noget af?
Avatar billede mikze Nybegynder
07. juni 2004 - 20:42 #5
Er der ingen :S ?
Avatar billede fangel Nybegynder
07. juni 2004 - 23:08 #6
johh - men at du så ikke forstå, det er en helt anden ting...

Men as said - den linie du skriver er linie 49 - ikke 52 !, og hverken 49 eller 52 ser ud til at have nogle fejl !!

Morten
Avatar billede mikze Nybegynder
08. juni 2004 - 15:05 #7
Okay... Men hvordan gør jeg så sådan at scriptet virker :S ?
Avatar billede fangel Nybegynder
08. juni 2004 - 15:44 #8
tjaa - umidbart kan _jeg_ ikke se nogle fejl...
Avatar billede mikze Nybegynder
08. juni 2004 - 16:32 #9
Ok... Kan jeg heller ikke... Men har fundet ud af det :) Det var i filen som postede havde jeg glemt og rette value til NONE i Vælg... hvis i forstår... Min value var gfx/dreng.gif hvor den skulle have været none :)
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
Vi tilbyder markedets bedste kurser inden for webudvikling

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester