Another Simple thing

[bluewalrus]

I'm trying to make another simple thing like the simple mailer but a simple page counter. I want the counter to write the number with the users ip address and computer information. I have this code but for some reason it has "Resource ID #3." I dont know where it's getting the Resource ID # from or the 3 the file was set to 0. This is my code thanks for any help you can offer here.

<?php 
$user =  $_SERVER['REMOTE_ADDR'];
$computer = $_SERVER["HTTP_USER_AGENT"];
$count = "hit.txt";
$value = fopen($count, "r+");
$entervalue = fread($value, 999999);
$entervalue = entervalue + 1;
fwrite($value, $value);
fclose($value);
$hitsfile = "log.txt";
$gethits = fopen($hitsfile, "r+");
$page = $value . ". " . $user . " " . $computer . "\n";
echo $page;
fwrite($gethits, $page);
fclose($gethits);
?>

[Master_script_maker]

try changing this: fwrite($value, $value); to this:fwrite($value, $entervalue); and this:$page = $value . ". " . $user . " " . $computer . "\n"; to this:$page = $entervalue . ". " . $user . " " . $computer . "\n";

[Nile]

If this helps you can base if off these:
http://www.dynamicdrive.com/forums/s...59&postcount=2
http://www.dynamicdrive.com/forums/s...16&postcount=7

[bluewalrus]

Okay I got it working now but decided to try and make it work better again hah. So I added in an attempt to see if a user has been to the site before but now it thinks the user has been there every time. Thanks for any further help you can offer with this

<?php 
$user =  $_SERVER['REMOTE_ADDR'];
$computer = $_SERVER["HTTP_USER_AGENT"];
//To Change the date formate from MM/DD/YY HH:MM go here http://www.php.net/date
$date = date('m/j/y H:i');
$count = "hit.txt";
$value2 = fopen($count, "r+");
$entervalue = fread($value2, 999999999);
$entervalue = $entervalue + 1;
fclose($value2);
$value = fopen($count, "w+");
fwrite($value, $entervalue);
fclose($value);
$hitsfile = "log.txt";
$gethits = fopen($hitsfile, "a+");
$check = fread($gethits, 9999999999);
$page = $entervalue . ". " . $date . " " . $user . " " . $computer . "\n";
//REMOVE ALL OF THIS IF YOU DON'T CARE IF A USER HAS BEEN HERE MORE THAN ONCE IN A DAY... REMOVE THAT . " "  IF YOU WANT TO BE MORE SPECIFIC TO IF THEY HAVE EVER BEEN THERE.
$beenhere = str_replace($check, "", $date . " " . $user);
if ($date . " " . $user === $beenhere) {
$entervalue = $entervalue - 1;
die ("You've been here today.");
} 
else {
//REMOVE UNTIL HERE
fwrite($gethits, $page);
fclose($gethits);
}
?>

[Master_script_maker]

try:

<?php  
$user =  $_SERVER['REMOTE_ADDR']; 
$computer = $_SERVER["HTTP_USER_AGENT"]; 
//To Change the date formate from MM/DD/YY HH:MM go here http://www.php.net/date 
$date = date('m/j/y H:i'); 
$count = "hit.txt"; 
$value2 = fopen($count, "r+"); 
$entervalue = fread($value2, 999999999); 
$entervalue = $entervalue + 1; 
fclose($value2); 
$value = fopen($count, "w+"); 
fwrite($value, $entervalue); 
fclose($value); 
$hitsfile = "log.txt"; 
$gethits = fopen($hitsfile, "a+"); 
$check = fread($gethits, 9999999999); 
$page = $entervalue . ". " . $date . " " . $user . " " . $computer . "\n"; 
//REMOVE ALL OF THIS IF YOU DON'T CARE IF A USER HAS BEEN HERE MORE THAN ONCE IN A DAY... REMOVE THAT . " "  IF YOU WANT TO BE MORE SPECIFIC TO IF THEY HAVE EVER BEEN THERE. 
$beenhere = preg_match("/".$user."/", $check); 
if ($beenhere) { 
$entervalue = $entervalue - 1; 
die ("You've been here today."); 
}  
else { 
//REMOVE UNTIL HERE 
fwrite($gethits, $page); 
fclose($gethits); 
} 
?>

[bluewalrus]

Yup that did it thanks.

If anyone comes here looking for this the whole code is below. I had to add in the rewriting to the text file the count for the if the user had been there before. So it is as follows.

<?php  
$user =  $_SERVER['REMOTE_ADDR']; 
$computer = $_SERVER["HTTP_USER_AGENT"]; 
//To Change the date formate from MM/DD/YY HH:MM go here http://www.php.net/date 
$date = date('m/j/y H:i'); 
$count = "hit.txt"; 
$value2 = fopen($count, "r+"); 
$entervalue = fread($value2, 999999999); 
$entervalue = $entervalue + 1; 
fclose($value2); 
$value = fopen($count, "w+"); 
fwrite($value, $entervalue); 
fclose($value); 
$hitsfile = "log.txt"; 
$gethits = fopen($hitsfile, "a+"); 
$check = fread($gethits, 9999999999); 
$page = $entervalue . ". " . $date . " " . $user . " " . $computer . "\n"; 
//REMOVE ALL OF THIS IF YOU DON'T CARE IF A USER HAS BEEN HERE MORE THAN ONCE IN A DAY... REMOVE THAT . " "  IF YOU WANT TO BE MORE SPECIFIC TO IF THEY HAVE EVER BEEN THERE. 
$beenhere = preg_match("/".$user."/", $check); 
if ($beenhere) { 
$entervalue = $entervalue - 1; 
$value = fopen($count, "w+"); 
fwrite($value, $entervalue); 
fclose($value); 
die ("You've been here today."); 
}  
else { 
//REMOVE UNTIL HERE 
fwrite($gethits, $page); 
fclose($gethits); 
} 
?>

[Master_script_maker]

glad i could help

[Twey]

Nice how you can disable the IP addresses of people you don't like by putting them in your user-agent string and then accessing the page And that IP addresses that contain other IP addresses will be considered the same address (e.g. 123.123.123.123 and 3.123.123.1). Also note how the script tries to read from the hits file 999999990 bytes more than are ever likely to be used, how if two users access the page at the same time it's likely that only one will be recorded, how regular expressions are used for a simple substring match, and how instead of just checking whether the user has been there before updating the hit counter, the script always updates the hit counter but then goes back and decreases it again...

Also, don't use locale-specific date formats in code, especially not the crazy American one — use ISO-8601 standard date format, DD-MM-YYYY (or 'N-m-Y').

[bluewalrus]

Yea I was considering going back through and reverse it so that it checked first to see if the user had been there then functioned the idea for that came one I first got it working. What do you mean by disabling the ip addresses with the users-agent string?

Also I noticed the checker I had isn't checking the date and user it's just checking the user so I just tried chaning to to this

$beenhere = preg_match("/".$date." ".$user."/", $check); 

but I got this:

Warning: preg_match(): Unknown modifier '1' in /hsphere/local/home/crazychr/bluewalrus.net/hitter/hit.php on line 19

Is it because of the " "?

Is there an infinite function so it will read the whole file no matter the size?

[Master_script_maker]

I've rewritten your code using the function file_get_contents() which retrieves the whole file.

<?php  
$user =  $_SERVER['REMOTE_ADDR']; 
$computer = $_SERVER["HTTP_USER_AGENT"]; 
$date = date('N-m-Y'); 
$counter = "hit.txt"; 
$cData = file_get_contents($counter);

$logfile = "log.txt"; 
$log = file_get_contents($logfile);  
$page = $cData . ". " . $date . " " . $user . " " . $computer . "\n"; 
$beenhere = str_replace($date." ".$user, '', $log); 
if ($beenhere===$log) { 
    $cData = $cData + 1;
    file_put_contents($logfile, $log.$page); 
    file_put_contents($counter, $cData);
} else { 
    file_put_contents($counter, $cData);
    die ("You've been here today.");
} 
?>