[php] schrijven naar file lukt niet*

[Verwijderd]

Na enig knutselwerk, is er bij mij de volgende code uitgekomen..
Deze code s bedoeld voor de mensen die bij mij op de site komen, de ip, datum, en het adres op te slaan in het ip.txt bestandje.
Hij moet ze telkens onder elkaar zetten in bestandje

voorbeeld

***.***.***.*** 5 mei 2003 blaat.blaat.nl
***.***.**.** 5 mei 2003 jezus.ganou.niet

Zoiets snap je, op de sterretjes moet het ip komen, de datum spreekt vanzelf, en het adres achteraan ook wel denk ik..

code:

<?php 
if( getenv('HTTP_X_FORWARDED_FOR') == '' ){  
        if ( getenv ("REMOTE_ADDR") == '' ){  
            $ip = "127.0.0.1";                  
        } else {  
            $ip = getenv ("REMOTE_ADDR");  
        }                  
    } else {  
        $ip = getenv('HTTP_X_FORWARDED_FOR');  
    }  
      
$file = fopen("ip.txt", "a+"); 
$date = date ("j M Y H:i:s"); 
$content = $ip."      ".$date." \n"; 
$deev = gethostbyaddr($REMOTE_ADDR); 
@fwrite($file , $content , $deev); 

?>

Weet iemand wat er in fout zit want hij doet het niet bij mij...

Greetz

Deev

[whoami]

Wat is niet doen?

[Verwijderd]

Hij schrijft bij mij niet naar het bestand..

[Glimi]

Wat werkt niet, wat doet het niet, wat is de melding, waarom heb je bij het debuggen een @ voor je fwrite staan?
Lijkt me handig om te weten voordat we er iets over kwijt kunnen

edit:
Je hebt de code niet zelf geschreven he? Ga dan eerst eens op zoek naar wat die @ doet

[ Voor 24% gewijzigd door Glimi op 23-05-2003 10:39 ]

[cdgrit]

Haal even de @ voor fwrite weg en post hier even de foutmelding... Zal wel een permission denied ofzo zijn

[Verwijderd]

nix dat is het probleem,

hij doet alsof ie het doet,

maar als ik dan het bestand ga kijken, is er nix bij geschreven...

edit:


Ey glimi, ik heb m wel zelf geschreven, met behulp van tutorials op internet

[ Voor 29% gewijzigd door Verwijderd op 23-05-2003 10:44 ]

[cdgrit]

Ook als je de @ weghaalt krijg je geen melding?
Als je het bestand verwijderd wordt er dan wel opnieuw een bestand aangemaakt?

[TeeDee]

- klopt de locatie van het bestand?
- genoeg rechten?

etc. etc.

[ACM]

Sinds wanneer heeft fwrite een filepointer en twee teksten als parameters?

De tweede tekst wordt geevalueerd naar een intwaarde met de waarde 0 en er wordt vervolgens niks weggeschreven omdat ie 0 tekens mag wegschrijven...

[ Voor 12% gewijzigd door ACM op 23-05-2003 10:47 ]

[Verwijderd]

Nee dan wordt het bestand niet aangemaakt nee,

Ik heb die bestanden en map chmod 777 gedaan (dan weet ik tenminste zeker dat ie het doet )
en de locatie klopt ook

[Glimi]

Verwijderd schreef op 23 May 2003 @ 10:43:

edit:
Ey glimi, ik heb m wel zelf geschreven, met behulp van tutorials op internet

Ik zie dan nog steeds niet in _waarom_ fwrite geprefixed is met een @

[Verwijderd]

had ik al een keer geprobeerd maar dat lukte niet..

[Dope-E]

zie ACM... je geeft de host als laatste parameter mee aan fwrite... da kannie!

[cdgrit]

Wat lukt niet? Gewoon je cursor voor het @ zetten en op de knop delete op je toetsenbord drukken

[Dope-E]

LOL

[Verwijderd]

Hoezo kan dat niet, ik heb die host gedefinieerd by $deev.

Dat is dan tog gewoon een output, die moet weergegeven worden?

[Dope-E]

hmm verdiep je eerst s hoe parameters gebruikt worden en hoe je strings aan elkaar plakt! (very basic php)
Als t dan nog niet lukt, kun je t nog s vragen...

[ACM]

zie de beschrijving van de fwrite functie

die wil maximaal drie parameters
de eerste moet een verwijzing naar een file zijn, geopend met fopen
de tweede de tekst die je wilt schrijven
de derde, optionele, de lengte die je wilt uitschrijven

jij knalt als derde een stuk tekst erin, dat stuk tekst wordt omgezet naar een getal door php en tekst wordt altijd omgezet naar 0 als het niet met een getal begint...

kortom, je schrijft 0 tekens weg...

[Verwijderd]

ok ik ga het thuis even proberen dan...

je hoort nog

Thanx

[Kippenijzer]

Oftwel, maak van je @fwrite($file , $content , $deev);
eens

code:

@fwrite($file , "$content $deev");

Dan evalueert hij de variablen meestal nog wel en werkt het wel

[ACM]

Laat dan gelijk die @ weg, want die is nergens voor nodig

[Verwijderd]

ACM schreef op 23 May 2003 @ 11:06:
Laat dan gelijk die @ weg, want die is nergens voor nodig

Idd, is toch niet zo moeilijk om die @ weg te halen, en kan je meteen een errorfunctie schrijven, dan werk je dat ook netjes weg. En nee niet met "or die()"

[Verwijderd]

ok nou heb ik het zo gedaan

code:

<?php
if( getenv('HTTP_X_FORWARDED_FOR') == '' ){
        if ( getenv ("REMOTE_ADDR") == '' ){
            $ip = "127.0.0.1";
        } else {
            $ip = getenv ("REMOTE_ADDR");
        }
    } else {  
        $ip = getenv('HTTP_X_FORWARDED_FOR');
    }

$file = fopen("ip.txt", "a+");
$date = date ("j M Y H:i:s");
$content = $ip."      ".$date." \n";
$deev = gethostbyaddr($REMOTE_ADDR);
fwrite($file , "$content $deev");

?>

maar nu geeft hij de volgende fout weer

code:

Warning: Address is not a valid IPv4 or IPv6 address in /usr/local/apache2/htdocs/info/ja.php on line 15

en als ik dan in de ip.txt ga kijken, dan heeft hij het adres, de datum en tijd in het bestand geschreven, maar de host heeft hij niet opgeslagen (dat is dus de fout denk ik die word gegeven)

Weten jullie daariets op:?

Greetz

[ACM]

"$deev = gethostbyaddr($REMOTE_ADDR);"

waar wordt die variabele $REMOTE_ADDR gedefinieerd?
Je moet toch zelf es leren debuggen, want dit soort simpele foutjes spot ik in enkele seconden... En dat is nou niet omdat ik intelligenter ofzo ben (zou ik niet weten of dat zo is ), maar gewoon omdat ik geleerd heb hoe je foutmeldingen moet lezen en de oorzaken eenvoudig kan achterhalen...

[Verwijderd]

ACM schreef op 23 mei 2003 @ 17:54:
"$deev = gethostbyaddr($REMOTE_ADDR);"

waar wordt die variabele $REMOTE_ADDR gedefinieerd?
Je moet toch zelf es leren debuggen, want dit soort simpele foutjes spot ik in enkele seconden... En dat is nou niet omdat ik intelligenter ofzo ben (zou ik niet weten of dat zo is ), maar gewoon omdat ik geleerd heb hoe je foutmeldingen moet lezen en de oorzaken eenvoudig kan achterhalen...

$REMOTE_ADDR is een standaard php variable die het IP van de bezoeker bevat

[Verwijderd]

Verwijderd schreef op 23 mei 2003 @ 17:58:
$REMOTE_ADDR is een standaard php variable die het IP van de bezoeker bevat

Idd, en ik vind dat ACM dat wel had moetten weten

Maar als je dit vanuit een functie doet moet je WEL die variable op GLOBAL moetten hebben, anders kan je daar niet bij in een functie. Als dat niet kan -> Dan neem ik ook terug wat ACM zij.

[ Voor 3% gewijzigd door Verwijderd op 23-05-2003 18:42 . Reden: = als dat niet kan.... ]

[Verwijderd]

en anders gewoon $_SERVER['REMOTE_ADDR']

Overigens kan je beter dit doen:

PHP:

$ip = isset($_SERVER["HTTP_X_FORWARDED_FOR"]) ? $_SERVER["HTTP_X_FORWARDED_FOR"] : $_SERVER["REMOTE_ADDR"];
$host = @gethostbyaddr($ip);

en dan $host wegschrijven.

edit:

lama ... zie dat je die forward al deed

[ Voor 131% gewijzigd door Verwijderd op 23-05-2003 18:29 ]

[Verwijderd]

dus hi is al geefiieerd ,l omdat die standaard is

[ACM]

Verwijderd schreef op 23 May 2003 @ 17:58:
$REMOTE_ADDR is een standaard php variable die het IP van de bezoeker bevat

Hoe zeker ben je er van dat ie idd een waarde bevat

Want dat is de enige plek waar de warning die je toont vandaan kan komen en als je een recente php hebt of bezig bent in functies heeft ie geen waarde.
Verder doe je steeds getenv('...') en dan ineens een variabele aanroep, sowieso niet handig om dat op verschillende manieren te doen...

[ACM]

Verwijderd schreef op 23 May 2003 @ 18:25:

PHP:

$ip = isset($_SERVER["HTTP_X_FORWARDED_FOR"]) ?

Wat jullie overigens allemaal vergeten is dat als je een gewone thuis proxy gebruikt je wel het adres van de proxy moet hebben en niet de forwarded_for ... Want er kan dan heel goed als ip 192.168.0.1 staan en daar heb je vrij weinig aan.

Mooier is wellicht zelfs nog het controleren op 192.168.x.x/10.x.x.x/172.benikvergeten.x.x enzo

Trouwens, handiger is gewoon om beide erin te zetten als het toch om een logfile gaat.

Thx erkens

[ Voor 31% gewijzigd door ACM op 23-05-2003 23:38 ]

[Erkens]

ACM schreef op 23 May 2003 @ 23:34:
Mooier is wellicht zelfs nog het controleren op 192.168.x.x/10.x.x.x/172.benikvergeten.x.x enzo

offtopic:
172.16.x.x-172.32.x.x

[Verwijderd]

pssst
error_reporting(E_ALL);

[Verwijderd]

$_SERVER['REMOTE_ADDR']; is beter...

[Verwijderd]

$REMOTE_ADDR wordt met de nieuwere php versies niet meer standaard gedefinieerd (REGISTER_GLOBALS uit). gebruikt in plaats van $REMOTE_ADDR eens $_SERVER["REMOTE_ADDR"]. die is niet afhankelijk van register_globals, dus werkt altijd. Laat het even weten

[Grijze Vos]

Verwijderd schreef op 23 May 2003 @ 17:58:
[...]


$REMOTE_ADDR is een standaard php variable die het IP van de bezoeker bevat

Alleen als je register globals aan hebt staan, geloof ik.

yup, je wil anders waarschijnlijk $_SERVER['REMOTE_ADDR'] gebruiken.

[ Voor 13% gewijzigd door Grijze Vos op 25-05-2003 01:42 ]