felsök min gästbok (Page 1 of 2)

åh, ibland bli blind för sina egna missar, någon som känner igen sig? :-P

Hmm. Du använder gethostbyname() vilken gör en vanlig dns lookup. Det behöver du inte. Bara spara $REMOTE_ADDR direkt i filen istället. Varför det alltid blir ditt IP har jag ingen aning om. Det måste vara det faktum att du använder gethostbyname() som ställer till det. Om du vill kan du använda min funktion för att försöka få tag på rätt IP:

function getRemoteAddress()
{
    if( isset( $_SERVER['HTTP_CLIENT_IP'] ) )
        return $_SERVER["HTTP_CLIENT_IP"];
    else if( isset( $_SERVER['HTTP_X_FORWARDED_FOR'] ) && strlen( $_SERVER['HTTP_X_FORWARDED_FOR'] ) <= 15 )
        return $_SERVER['HTTP_X_FORWARDED_FOR'];
    else
        return $_SERVER["REMOTE_ADDR"];
}

hmm nu skriver den inte ut IP:t alls
har ändat gb_send.php till detta:

<?
function getRemoteAddress()
{
    if( isset( $_SERVER['HTTP_CLIENT_IP'] ) )
        return $_SERVER["HTTP_CLIENT_IP"];
    else if( isset( $_SERVER['HTTP_X_FORWARDED_FOR'] ) && strlen( $_SERVER['HTTP_X_FORWARDED_FOR'] ) <= 15 )
        return $_SERVER['HTTP_X_FORWARDED_FOR'];
    else
        return $_SERVER["REMOTE_ADDR"];
}

$namn = $_POST[namn];
$msg = $_POST[msg];
$mail = $_POST[mail];
$ip = $getRemoteAddress; 
$postat = date("d/m-y  H:i:s");                            
$cfile = "namn.txt"; 

$fp = fopen($cfile, "a"); 
fwrite($fp, "$namn | $msg | $postat | $ip |$mail|\n"); 
fclose($fp); 

#header("Location: index.php?content=gb");
Echo("<center>
<font face=verdana size=3><b> Ditt gästboksinlägg skickades felfritt ;-) </b><br>
<font size=1>Du skickas nu vidare till gästboken, om din browser inte skickar dig vidare eller om du inte orkar vänta klicka <a href=index.php?contens=gb> här</a></font>");
?>

kollade serialize() i manulen, fattar inte vad det ska vara bra till :/

Ändra:

$ip = $getRemoteAddress;

till

$ip = getRemoteAddress();

Angående serialize(). Denna funktion använder man för att omvandla en array till en sträng som man t.ex. kan skriva till en fil eller sätta i en cookie osv. I ditt fall skulle du t.ex. kunna sätta in namn, e-mail, ip osv i en array och sen använda serialize() för att göra detta till en sträng och sen skriva denna sträng till filen. När du sen skulle visa ett inlägg skulle du läsa strängen från filen och köra unserialize() på den som omvandlar tillbaka den till en array. En av fördelarna skulle t.ex. vara att du hur enkelt som helst skulle kunna lägga till nya poster i din form och bara smälla in dem i arrayen innan du kör serialize. Det skulle också tillåta namn och e-mail att innehålla vilka tecken som helst. Om man t.ex. skrev "| bajs |" som meddelande i din gästbok skulle det väl bli lite tokigt när du skriver den till filen med tanke på att du använder | som en avgränsare mellan data. Med serialize skulle detta inte vara ett problem.

allright det låter ju sunt... :-)

whoopido fixat...

bara en sak som stör mig nuv hur fn får jag nya inläggen att visas högs upp... :/

varför inte vacker?
edit: funkar inte alls med den count metoden jag använder nu..

den räknade som vanligt, det inlägget som ligger närmast toppen fick #1 och sedan räknades det uppoy ju långre ner man kom i listan...
nummerna var altså oberone av inläggen...

Förresten, väldigt leet signatur :-)

Tja, array_reverse() kommer vara en ganska tung operation för PHP när du har en massa inlägg. En lite snyggare lösning hade varit att inte appenda nya inlägg utan att på något sätt lägga dem först i filen istället. Det kräver dock lite mer kod.

jo jag provade först genom att bara ändra vilkoren (kallas det det?) för fopen. men det finna ingen som sätter "file pointern" i början av filen utan att skriva över det som finns bakom...

och något mer avancerat kan jag inte komma på...