Raring Skrevet 22. mai 2004 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
wacie Skrevet 22. mai 2004 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
jorgis Skrevet 22. mai 2004 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.
???????? Skrevet 22. mai 2004 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å.
Raring Skrevet 22. mai 2004 Forfatter 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
jorgis Skrevet 22. mai 2004 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
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å