Raring Skrevet 22. mai 2004 Del Skrevet 22. mai 2004 (endret) ? <?php include 'leseinniarray.php'; $key = array_search($_SERVER[REMOTE_ADDR], $linje); If ($key != "") { $filename = 'like.html'; $someconten = "$_SERVER[REMOTE_ADDR]"; $somecontent = "$someconten <br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) { exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} $filename = 'datol.html'; $someconten = "date(d)"; $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) {exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} Elseif ($key == "") { $filename = 'ulike.html'; $someconten = "$_SERVER[REMOTE_ADDR]"; $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) { exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} $filename = 'datou.html'; $someconten = "date(d)"; $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) {exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} ?> Kanskje litt uoversiktelig... Her er URL: http://home.no.net/hbhb1/prosjekt/ Endret 22. mai 2004 av Raring Lenke til kommentar
wacie Skrevet 22. mai 2004 Del Skrevet 22. mai 2004 Jeg tok å la på } } på slutten sånn at koden ble sånn: <?php include 'leseinniarray.php'; $key = array_search($_SERVER[REMOTE_ADDR], $linje); If ($key != "") { $filename = 'like.html'; $someconten = "$_SERVER[REMOTE_ADDR]"; $somecontent = "$someconten <br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) { exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} $filename = 'datol.html'; $someconten = "date(d)"; $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) {exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} Elseif ($key == "") { $filename = 'ulike.html'; $someconten = "$_SERVER[REMOTE_ADDR]"; $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) { exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} $filename = 'datou.html'; $someconten = "date(d)"; $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) {exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} } } ?> da fikk jeg bare opp erroren at den ikke kunne finne 'leseinniarray.php' Håper det er riktig det jeg har gjort -wacie Lenke til kommentar
jorgis Skrevet 22. mai 2004 Del Skrevet 22. mai 2004 Meget rotete kode, ja... Skal vi se: <?phpinclude 'leseinniarray.php'; $key = array_search($_SERVER[REMOTE_ADDR], $linje); If ($key != "") { // liten bokstav på if'ene $filename = 'like.html'; $someconten = "$_SERVER[REMOTE_ADDR]"; //du trenger ikke å sette fnutter rundt. Bedre med fnutter inni []-ene $somecontent = "$someconten <br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) { exit;} //du kan godt sette sluttbrakken på en egen linje for økt lesbarhet if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} $filename = 'datol.html'; $someconten = "date(d)"; //det blir feil, iom. at date() er en funksjon, ikke en strengverdi $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) {exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} Elseif ($key == "") { //liten forbokstav igjen, takk $filename = 'ulike.html'; $someconten = "$_SERVER[REMOTE_ADDR]"; //fremdeles ikke nødvendig med fnutter $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) { exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} $filename = 'datou.html'; $someconten = "date(d)"; $somecontent = "$someconten<br>"; if (is_writable($filename)) { if (!$handle = fopen($filename, 'a')) {exit;} if (fwrite($handle, $somecontent) === FALSE) { exit;} fclose($handle);} ?> Var nok kanskje ikke det du hadde håpet på, men den koden der var bare spaghetti. Prøv å gjør den litt mer lesbar, så er det håp for andre å skjønne logikken. Den koden kunne forresten godt ha vært krympet med endel linjer. Lenke til kommentar
???????? Skrevet 22. mai 2004 Del Skrevet 22. mai 2004 $key != "" - det finnes en egen funksjon for dette: empty() - eventuelt !empty() når du skal sjekke om den ikke er tom. Hvor kommer variabelen $key fra? Er det noe som sendes med header? (fra skjema, cookie, adresselinjen eller session) I så fall må du kanskje bruke: $_POST['key'] $_GET['key'] $_COOKIE['key'] $_REQUEST['key'] - henter fra POST, GET og cookie $_SESSION['key'] avhengig av hvor variabelen kommer fra - og om register globals er av eller på. Lenke til kommentar
Raring Skrevet 22. mai 2004 Forfatter Del Skrevet 22. mai 2004 Den koden kunne forresten godt ha vært krympet med endel linjer. Å? Hvordan? Jeg har kopiert koden så jeg skjønner egentelig ikke så mye av den selv. Kunne en av dere prøvd å rydde litt opp i "spagetthien"? Hvor kommer variabelen $key fra? Det står på 3. linje. $key != "" - det finnes en egen funksjon for dette: empty() - eventuelt !empty() når du skal sjekke om den ikke er tom. Er ikke != det motsatte av ==? Sånn at: != = ikke lik == = Lik Lenke til kommentar
jorgis Skrevet 22. mai 2004 Del Skrevet 22. mai 2004 $key != "" - det finnes en egen funksjon for dette: empty() - eventuelt !empty() når du skal sjekke om den ikke er tom. Er ikke != det motsatte av ==? Sånn at: != = ikke lik == = Lik Det stemmer, men hvis du bare skal sjekke om en variabel er tom, kan du heller bruke funksjonen empty(); Hvis du vil vite om den ikke er tom, er det !empty(); ! = not/ikke Lenke til kommentar
Anbefalte innlegg
Opprett en konto eller logg inn for å kommentere
Du må være et medlem for å kunne skrive en kommentar
Opprett konto
Det er enkelt å melde seg inn for å starte en ny konto!
Start en kontoLogg inn
Har du allerede en konto? Logg inn her.
Logg inn nå