[php] Zabezpieczenie modułu przez "nabijaniem" Opcje

[fredzio90]

mam takówy problem.. robie moduł do systemu CMS podobnego do Programu Partnerskiego na stronach komercyjnych.. i mam problem...
mam 2 tabele do jednej zapisuje się zarejestrowanego usera, do drugiej zapisywane są adresy ip które odwiedziły linki PP ( domyślacie się, linki aby zbierać punkty ) i moj problem polega na zabezpieczeniu przez nabijaniem punktów przez tą samą osobe, przez te same IP.... napisałem taką funkcje:

[PHP] pobierz, plaintext 
<?php
function pp_view() {
	global $pp_tbl, $pp_conf_tbl, $goto, $PHP_SELF, $REMOTE_ADDR;
 
 
$sql = mysql_query("SELECT * FROM $pp_tbl WHERE pp_nr='{$_GET['url']}'");
while($r = mysql_fetch_array($sql)) {
	$points = $r['points'];
	$nr_pp = $r['pp_nr'];
	$ppnick = $r['ident']; 
 
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip = $ccc['ip'];
}
 
if($ip=='$REMOTE_ADDR') {
header("Location: $PHP_SELF");
} else {
$sqlUp = mysql_query("UPDATE $pp_tbl SET points=points+1 WHERE ident='$ppnick'");
$sqlZab = mysql_query("INSERT INTO $pp_conf_tbl SET ip='$REMOTE_ADDR', data=now()");
header("Location: $PHP_SELF");
}
}
}
?>
[PHP] pobierz, plaintext 

i niewiem jak tą funkcje przekrztałcić aby:

podczas wchodzenia w link, sprawdzane było w tabeli $pp_conf_tbl czy ip usera ktory kliknow w link PP nie znajduje się w tabeli.. jeżeli tak to żeby pokazało $PHP_SELF, a jeżeli nie to żeby dodało jego ip do bazy $pp_conf_tbl i zaaktualizowało tabele $pp_tbl a dokladnie jego pole points+1...

jak kilkam w odnosnik kierujacy do dodania punktów, dodaje się do bazy ip i aktualizuje punkty usera, lecz powinno wykrywać także czy ip usera nie jest w tabeli i uniemożliwić "nalicznie" punnktów...

Ten post edytował fredzio90 28.03.2007, 19:33:54

[MacPain]

Pobierasz nr ip od kolesia sprawdzasz czy istnije w tabeli, jeżeli nie to dodajesz ip + punkty, jeżeli tak to nie dodajesz nic dobrze rozumiem (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?

Jeżeli tak to spróbuj tak

[PHP] pobierz, plaintext 
<?php
 
//...łączysz sie z baza
 
//...pobierasz ip
 
$zapytanie = mysql_query("SELECT * FROM numery_ip WHERE nr_ip = '$ip'");
   if(mysql_num_rows($zapytanie) > 0){
	  echo "Nr. ip istnieje już w bazie";
	  }else{
		 $dodaj = mysql_query("INSERT INTO numery_ip VALUES(0,'$ip','punkty')");
		 echo "Głos został oddany dziękuje";
		 }
?>
[PHP] pobierz, plaintext 

Ten post edytował dr_bonzo 30.03.2007, 16:51:09

[fredzio90]

TAK (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

to co podales to musi byc w tym stylu.. tutaj sie nei lkacze z baza.. chodiz mi o ten fragment kodu:

[PHP] pobierz, plaintext 
<?php
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip = $ccc['ip'];
}
 
if($ip=='$REMOTE_ADDR') {
header("Location: $PHP_SELF");
} else {
$sqlUp = mysql_query("UPDATE $pp_tbl SET points=points+1 WHERE ident='$ppnick'");
$sqlZab = mysql_query("INSERT INTO $pp_conf_tbl SET ip='$REMOTE_ADDR', data=now()");
header("Location: $PHP_SELF");
}
}
?>
[PHP] pobierz, plaintext 

wszystko dodaje, ip usera do bazy pp_conf... ale dodaje rowniez za kazdym razem punkty.. ja chce zeby z 1 IP mozna bylo 1 glos walnac....

i chodzi mi tutaj czy pętelke while() jest dobrze w kodzie powyżej.. i niewiem ale nie działa opcja ta:

[PHP] pobierz, plaintext 
<?php
if($ip=='$REMOTE_ADDR') {
?>
[PHP] pobierz, plaintext 

pomoze ktos (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

Ten post edytował fredzio90 28.03.2007, 21:58:09

[Kicok]

1. http://pl.php.net/manual/pl/language.types.string.php
Tutaj znajdziesz odpowiedź, dlaczego kod:

[PHP] pobierz, plaintext 
<?php
if($ip == '$REMOTE_ADDR')
?>
[PHP] pobierz, plaintext 

nie działa tak jak sobie tego życzysz.


2. Powinno się stosować zapis:

[PHP] pobierz, plaintext 
<?php
$_SERVER['PHP_SELF'];
$_SERVER['REMOTE_ADDR'];
?>
[PHP] pobierz, plaintext 

zamiast:

[PHP] pobierz, plaintext 
<?php
$PHP_SELF;
$REMOTE_ADDR;
?>
[PHP] pobierz, plaintext 

Jeśli na serwerze, na którym umieścisz swój skrypt register_globals będzie wyłączone, to nic ci nie będzie działać.


3.

[PHP] pobierz, plaintext 
<?php
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip = $ccc['ip'];
}
?>
[PHP] pobierz, plaintext 

Ta pętla odczyta ci tylko ostatni adres IP z tabeli $pp_conf_tbl. Na pewno o to ci chodziło? Może przerób zmienną $ip na tablicę, a później sprawdzaj adresy IP funkcją in_array" title="Zobacz w manualu PHP" target="_manual

[fredzio90]

oprubowałe m zmienić i wyszło mi tkaie coś:

[PHP] pobierz, plaintext 
<?php
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip = array('$ccc['ip']');
}
 
if (in_array("$_SERVER['REMOTE_ADDR']", $cc)) {
header("Location: $PHP_SELF");
} else {
$sqlUp = mysql_query("UPDATE $pp_tbl SET points=points+1 WHERE ident='$ppnick'");
$sqlZab = mysql_query("INSERT INTO $pp_conf_tbl SET ip='$REMOTE_ADDR', data=now()");
header("Location: $PHP_SELF");
echo mysql_error();
}
?>
[PHP] pobierz, plaintext 

i niestety błąd:

Parse error: syntax error, unexpected T_STRING, expecting ')'

w tej linijce:

$ip = array('$ccc['ip']');

[UDAT]

[PHP] pobierz, plaintext 
<?php
$ip = array(		  '$ccc['				  ip				  ']'			  );
?>
[PHP] pobierz, plaintext 

A co to za dziwny string ip?? (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)



Daj raczej tak:

[PHP] pobierz, plaintext 
<?php
$ip =array ();
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip[] = $ccc['ip'];
}
 
if (in_array("$_SERVER['REMOTE_ADDR']", $cc)) {
header("Location: $PHP_SELF");
} else {
$sqlUp = mysql_query("UPDATE $pp_tbl SET points=points+1 WHERE ident='$ppnick'");
$sqlZab = mysql_query("INSERT INTO $pp_conf_tbl SET ip='$REMOTE_ADDR', data=now()");
header("Location: $PHP_SELF");
echo mysql_error();
}
?>
[PHP] pobierz, plaintext 

[fredzio90]

kodzik:

[PHP] pobierz, plaintext 
<?php
$ip =array ();
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip[] = $ccc['ip'];
}
 
if (in_array("{$SERVER['REMOTE_ADDR']}", $ccc)) {
header("Location: $PHP_SELF");
} else {
$sqlUp = mysql_query("UPDATE $pp_tbl SET points=points+1 WHERE ident='$ppnick'");
$sqlZab = mysql_query("INSERT INTO $pp_conf_tbl SET ip='$REMOTE_ADDR', data=now()");
header("Location: $PHP_SELF");
echo mysql_error();
}
?>
[PHP] pobierz, plaintext 

i wciąż dodaje punkty ( points ) do bazy jeżeli ip usera odwiedzjaego jest w bazie....

[pbnan]

[PHP] pobierz, plaintext 
<?php
$ip =array ();
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip[] = $ccc['ip'];
}
 
if (in_array("{$SERVER['REMOTE_ADDR']}", $ip)) {
header("Location: $PHP_SELF");
} else {
$sqlUp = mysql_query("UPDATE $pp_tbl SET points=points+1 WHERE ident='$ppnick'");
$sqlZab = mysql_query("INSERT INTO $pp_conf_tbl SET ip='$REMOTE_ADDR', data=now()");
header("Location: $PHP_SELF");
echo mysql_error();
}
?>
[PHP] pobierz, plaintext 

Źle sprawdzasz warunek (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) (chyba... (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) )

[fredzio90]

wciąz zle (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif)

[pbnan]

[PHP] pobierz, plaintext 
<?php
$ip =array ();
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip[] = $ccc['ip'];
}
 
if (in_array($_SERVER['REMOTE_ADDR'], $ip)) {
header("Location: $PHP_SELF");
} else {
$sqlUp = mysql_query("UPDATE $pp_tbl SET points=points+1 WHERE ident='$ppnick'");
$sqlZab = mysql_query("INSERT INTO $pp_conf_tbl SET ip='$REMOTE_ADDR', data=now()");
header("Location: $PHP_SELF");
echo mysql_error();
}
?>
[PHP] pobierz, plaintext 

Teraz działać powinno bardziej, niż poprzednio (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Używaj poprawnego bbcode!! Poprawiam (sab)
//dobra, tylko jeszcze się muszę nauczyć (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował pbnan 31.03.2007, 14:17:52

[fredzio90]

dalej to samo! wciąż dodaje do bazy

da sie to jhakoś zrobić ?

zrobiłem tak:

[PHP] pobierz, plaintext 
<?php
$check = mysql_query("SELECT * FROM $pp_conf_tbl");
while($ccc = mysql_fetch_array($check)) {
$ip = $ccc['ip'];
}
 
if (!$ip ==$REMOTE_ADDR) {
$sqlUp = mysql_query("UPDATE $pp_tbl SET points=points+1 WHERE ident='$ppnick'");
$sqlZab = mysql_query("INSERT INTO $pp_conf_tbl SET ip='$REMOTE_ADDR', data=now()");
header("Location: $PHP_SELF");
}
?>
[PHP] pobierz, plaintext 

i działa