[PHP][MySQL]Jak wpisywać ip do bazy danych Opcje

[jahonen]

[PHP] pobierz, plaintext 
if(isset($_GET['ref']))
{
	if(intval($_GET['ref']) !==0)
	{
		$ip = $_SERVER['REMOTE_ADDR'];
	db_connect();
 
$selip = mysql_query("SELECT ip1, ip2, ip3 FROM users WHERE ref = '$_GET[ref]'");
$fetched = mysql_fetch_array($selip);
if($fetched["ip1"] = 0){
 
	$query1 = mysql_query("UPDATE users SET zdolnosc=zdolnosc+1 WHERE ref = '$_GET[ref]'"); 
	$query2 = mysql_query("INSERT INTO users (ip1) VALUES ($ip) WHERE ref = '$_GET[ref]'");
}
elseif($fetched["ip2"] = 0 && $fetched["ip1"] != $ip)
{
	$query1 = mysql_query("UPDATE users SET zdolnosc=zdolnosc+1 WHERE ref = '$_GET[ref]'"); 
	$query2 = mysql_query("INSERT INTO users (ip2) VALUES ($ip) WHERE ref = '$_GET[ref]'");
	}
elseif($fetched["ip3"] = 0 && $fetched["ip2"] != $ip && $fetched["ip1"] != $ip)
{
	$query1 = mysql_query("UPDATE users SET zdolnosc=zdolnosc+1 WHERE ref = '$_GET[ref]'"); 
	$query2 = mysql_query("INSERT INTO users (ip3) VALUES ($ip) WHERE ref = '$_GET[ref]'");
	}
	}
	}
[PHP] pobierz, plaintext 

Napisaem taki kod. Nie mam jak go sprawdzić ponieważ działam na localhost. $_GET["ref"] to tylko cyfry Moje pytania:
Czy kod jest bezpieczny?
Czy ip zosatnie zapisane w bazie danych?
Ip zapisywać varchar'em czy int'em, a może czymś innym?
Czy kod jest poprawnie napisany?

W kodzie chodzi mi o to by inkrementować pole zdolność, kiedy wszystkie warunki zostaną spełnione, czyli ludzie o 3 innych adresach ip wejdą w podany link.
Domyślna wartość pól ip1, ip2, ip3 w bazie wynosi 0.

Z góry dzięki (IMG:style_emoticons/default/wink.gif)

Ten post edytował jahonen 14.05.2014, 12:44:06