[PHP]Problem ze skryptem głosowania, jw Opcje

[miras]

Witam, napisałem sobie taki skrypt głosowania:

[PHP] pobierz, plaintext 
 
	if (($_GET['glos']==1) AND (!empty($_GET['id']))) {
			$ip = $_SERVER['REMOTE_ADDR'];
			$spr = file("inc/glosy.txt");
			$linijka=count($spr);
 
 
			if ($ip!=$spr[$linijka]-1) {
 
			$dane = $ip."\n";
			$file = "inc/glosy.txt";
			$fp = fopen($file, "a");
			flock($fp, 2);
			fwrite($fp, $dane);
			flock($fp, 3);
			fclose($fp); 
			$info="Twój głos został dodany!";
			} else { $info="Głosowałeś niedawno!"; }
			echo '<div class="prze_info">'.$info.'</div>';
		}
[PHP] pobierz, plaintext 

IP mi się zapisują elegancko do pliku, ale coś jest nie tak ze sprawdzaniem chyba czy obecne ip=ostatnio dodane ip.. bo można cały czas spamować i głosować..

[nospor]

Skup wieksza uwage na tym kawalku
$spr[$linijka]-1

[miras]

hmm, kurcze nie mogę nadal zdiagnozować problemu, wiem, że to już duża podpowiedź, ale można jaśniej ?

[com]

pytanie poco CI tam to -1 po $spr[$linijka] ?

[fiszol]

Na 100% ma pustą linie na końcu pliku i nie wie jak się z nią rozprawić ;-p

[miras]

Zgadza się, mam pusta linie, z -1 czy bez tak czy siak nie dziala.. nawet -2 probowalem..

[com]

bo dajesz nie tam gdzie trzeba a zmienna dodatkowo możesz wrzucić w http://php.net/manual/en/function.trim.php i Ci ją usunie jak już to dajesz

[PHP] pobierz, plaintext 
$spr[$linijka-1]
[PHP] pobierz, plaintext 

bo tak jak masz to tak jakby od pobranego ip byś chciał odjąć 1 co i tak chyba się nie wykona bo brakuje nawiasu widzę za dużo c ostatnio albo pochodnych

Ten post edytował com 13.09.2013, 18:26:29

[miras]

racja.. matko, ale błąd.. ale niestety nadal mi to nie chce działać... obecny kod:

[PHP] pobierz, plaintext 
		if (($_GET['glos']==1) AND (!empty($_GET['id']))) {
			$ip = $_SERVER['REMOTE_ADDR'];
			$spr = file("inc/glosy.txt");
			$linijka=count($spr);
 
 
			if ($ip!=$spr[$linijka]) {
 
			$dane = $ip."\n";
			$file = "inc/glosy.txt";
			$fp = fopen($file, "a");
			flock($fp, 2);
			fwrite($fp, $dane);
			flock($fp, 3);
			fclose($fp); 
			$info="Twój głos został dodany!";
			} else { $info="Głosowałeś niedawno!"; }
			echo '<div class="prze_info">'.$info.'</div>';
		}
[PHP] pobierz, plaintext 

[com]

no ale dałem Ci przecież, że jak tak chcesz to dajesz $spr[$linijka-1] lub

[PHP] pobierz, plaintext 
 
    	if (($_GET['glos']==1) AND (!empty($_GET['id']))) {
    $ip = $_SERVER['REMOTE_ADDR'];
    $spr = file("inc/glosy.txt");
    $linijka=count(trim($spr));
 
 
    if ($ip!=$spr[$linijka]) {
 
    $dane = $ip."\n";
    $file = "inc/glosy.txt";
    $fp = fopen($file, "a");
    flock($fp, 2);
    fwrite($fp, $dane);
    flock($fp, 3);
    fclose($fp);
    $info="Twój głos został dodany!";
    } else { $info="Głosowałeś niedawno!"; }
    echo '<div class="prze_info">'.$info.'</div>';
    }
[PHP] pobierz, plaintext 

[fiszol]

Ale kombinujecie.

[PHP] pobierz, plaintext 
<?php
if(in_array($ip,$spr)) {
    //do stuff
}
else {
    //do stuff
}
?>
[PHP] pobierz, plaintext 

[miras]

tfu, zły kod wkleiłem ani z trim'em ani z -1 nie działa.. chyba jednak gdzieś jeszcze jest problem..

[kropamk]

Inaczej, tak to sobie wymyśliłem, pewnie trochę na około ale działa

[PHP] pobierz, plaintext 
$ip = $_SERVER['REMOTE_ADDR'];
$fname = "glosy.txt";
if (filesize($fname) == 0) {
 // jeżeli plik jest pusty to dodaję ip
$dane = $ip . "\n";
$file = "glosy.txt";
$fp = fopen($file, "a");
flock($fp, 2);
fwrite($fp, $dane);
flock($fp, 3);
fclose($fp);
} else {
$file = fopen('glosy.txt', 'r') or die("");
$data = fread($file, filesize('glosy.txt'));
$items = explode("\n", $data);
$search = array_search($ip, $items);
 
if ($search === FALSE) {
                $dane = $ip . "\n";
                $file = "glosy.txt";
                $fp = fopen($file, "a");
                flock($fp, 2);
                fwrite($fp, $dane);
                flock($fp, 3);
                fclose($fp);
                $info = "Twój głos został dodany!";
            } else {
                $info = "Głosowałeś niedawno!";
            }
            echo '<div class="prze_info">' . $info . '</div>';
        }
[PHP] pobierz, plaintext 

Ten post edytował kropamk 13.09.2013, 19:59:21