[MySQL][PHP] link referujacy Opcje

[Lethys]

Witam,

Chcę zrobić system polecania gry.

Problem polega na tym, że mimo iż mimo że zmianna $ref pobiera z linku polecajacego to nie jest on dodawany do bazy :/

Na tym linku można zobaczyć że gracz jest sczytywany http://www.westernlife.pl/poleca/lethys/

Obecnie wygląda to tak:

[PHP] pobierz, plaintext 
 
<?php
$ref = mysql_fetch_array(mysql_query("select * from players where user='$ref'"));
?>
 
 
<form method=post action=rejestracja.php?action=register>
 
<table border="0" cellpadding="0" cellspacing="0" style="border-collapse: collapse" width="500" align="center" height="200">
<tr>
<td width="100%" align="center">
<table border="0">
<tr><td>Użytkownik:</td><td><input type="text" name="user" maxlength="10"></td></tr>
<tr><td>Email:</td><td><input type="text" name="email" maxlength="40"></td></tr>
<tr><td>Powtórz mail:</td><td><input type="text" name="vemail" maxlength="40"></td></tr>
<tr><td>Hasło:</td><td><input type="password" name="pass" maxlength="15"></td></tr>
<tr><td>Polecający:</td><td><?php echo("<input type=\"text\" name=\"referujacy\" value=\"$ref[user]\" disabled=\"disabled\" />"); ?></td></tr>
<tr><td></td><td><br><br><input type="submit" value="Rejestracja" name="submit"></td></tr>
</table>
</form>
<br>Rejestrując się akceptujesz <a href=regulamin.php>regulamin</a>
</td>
</tr>
</table>
 
 
 
<?php
if (isset ($_POST['submit'])){//isset wyslij
 
    include("config.php");
 
 
		if (!$user || !$email || !$vemail|| !$pass) {
			$problem = TRUE;
            echo("<p id='uwaga'><h2>Musisz wypełnić wszystkie pola.</h2></p>");
		}		
		if (10 < strlen($_POST['user'])) {
            echo("<p id='uwaga'><h2>Zbyt długa nazwa loginu. Maksymalnie <b>10</b> znaków.</h2></p>");
			$problem = TRUE;
        }
        if (strlen($_POST['email']) != strlen($_POST['vemail'])) {
            echo("<p id='uwaga'><h2>Podane przez Ciebie emaile się nie zgadzają. Potwierdzony email musi być taki sam.</h2></p>");
			$problem = TRUE;
        }
		$test1 = mysql_num_rows(mysql_query("select * from players where user='".$_POST['user']."'"));
			if ($test1 > 0) {
            echo("<p id='uwaga'><h2>Ktoś już użył takiej nazwy użytkownika, musisz wybrać inną.</h2></p>");
			$problem = TRUE;
	    }
		$test2 = mysql_num_rows(mysql_query("select * from players where email='".$_POST['email']."'"));
		if ($test2 > 0) {
            echo("<p id='uwaga'><h2>Ktoś już użył takiego adresu email. Musisz użyć innego adresu.</h2></p>");
			$problem = TRUE;
	    }
		$test3 = mysql_num_rows(mysql_query("select * from gracze where user='".$_POST['user']."'"));
			if ($test3 > 0) {
            echo("<p id='uwaga'><h2>Ktoś już użył takiej nazwy użytkownika, musisz wybrać inną.</h2></p>");
			$problem = TRUE;
	    }
		$test4 = mysql_num_rows(mysql_query("select * from gracze where email='".$_POST['email']."'"));
		if ($test4 > 0) {
            echo("<p id='uwaga'><h2>Ktoś już użył takiego adresu email. Musisz użyć innego adresu.</h2></p>");
			$problem = TRUE;
	    }
        if (15 < strlen($_POST['pass'])) {
            echo("<p id='uwaga'><h2>Twoje hasło jest za długie. Maksymalnie możesz użyć <b>15</b> znaków.</h2></p>");
			$problem = TRUE;
        }
		if(!preg_match('/^([a-ż0-9_]+)$/i', $_POST['user'])){
		$problem = TRUE;
        echo("<p id='uwaga'><h2>Użyto niedozwolonych znaków w nazwie użytkownika.</h2></p>");
		}
		if(!preg_match('/^([a-ż0-9_]+)$/i', $_POST['pass'])){
		$problem = TRUE;
        echo("<p id='uwaga'><h2>Użyto niedozwolonych znaków w haśle.</h2></p>");
		}
 
 
        if (!$problem){
 
 
 
		$data = date("d.m.y");
		$haslo = md5($pass);
		$kod = rand(100000000,999999999);
		global $referujacy;
 
	mysql_query("insert into gracze (user, email, pass, data, kod, ref) values ('$user','$email','$haslo','$data','$kod','$referujacy')") or die("Nie można zarejestować.");
 
 
/// blabal dalej kod
[PHP] pobierz, plaintext 

Ten post edytował Lethys 13.04.2011, 16:05:08

[pmir13]

W linii 87 definiujesz zmienną globalną $referujacy i próbujesz wstawić do bazy nie przypisując jej żadnej wartości.
Nie ma ona nic wspólnego ze zmienną $ref, do której na początku coś tam wpisujesz.

[Lethys]

Tej globalnej nie powinno być, stary kod mi sie wrzucił.

No ale bez globalnej też nie działa.

Czemu takie zmiennie jak: $user czy $pass są dodawane do bazy skoro też są z formularza?

Jak dodać do bazy zmienną $referujacy

[nospor]

stary kod mi sie wrzucił.

To może pokaż właściwy kod. Włącz wyświetlanie wszystkich błędów.

[hermo767]

17 linijce dajesz value=$ref[user] a skad skrypt to ma wziac, skoro nie ma? wywal to value i bedzie ok

[Hpsi]

[PHP] pobierz, plaintext 
<?php
 $ref = mysql_fetch_array(mysql_query("select * from players where user='$ref'"));
?>
[PHP] pobierz, plaintext 

a masz wartośc $ref ? bo rozumiem ze pobierasz z adresu nazwe referujacego czyli np twojastrona.pl?ref=hpsi+
wtedy $ref = $_GET['ref']; wg mnie powinno to mniej więcej śmigać, ale dołączam się do proźby nospora

[Lethys]

Kod jest taki sam, tzn w starej wersji nie było tej globalnej, to jedyna różnica.

Włączyłem wyświetlanie błędów:

Wyskoczyło

Notice: Undefined variable: problem in /srv/www/vhosts/westernlife.pl/httpdocs/rejestracja.php on line 85 Notice: Undefined variable: referujacy in /srv/www/vhosts/westernlife.pl/httpdocs/rejestracja.php on line 93

linijka 85:

[PHP] pobierz, plaintext 
if (!$problem){
[PHP] pobierz, plaintext 

linijka 93:

[PHP] pobierz, plaintext 
mysql_query("insert into gracze (user, email, pass, data, kod, ref) values('$user','$email','$haslo','$data','$kod','$referujacy')") or die(mysql_error());
[PHP] pobierz, plaintext 

[nospor]

Pola z atrybutem disabled nie są wysyłane razem z formularzem

[pmir13]

Z jaką wartością startuje $problem?
Dodaj na początku $problem = FALSE;
Podobnie tam gdzie byl ten global wrzuć: $referujacy = $_POST['referujacy'];
Zmień w formularzu input type="hidden", wywal disabled.
Sprawdź dokładnie co jest w $referujacy, nawet jak nie jest wyświetlany na stronie i tak można tam wszystko wpisać.

Ten post edytował pmir13 13.04.2011, 16:47:53

[Lethys]

@ nospor

To jak mogę rozwiązać ten problem?

Nie chciałbym dać normalnego pola bo użytkownicy będą mogli gmerać przy refie.

@pmir13

Zrobiłem jak napisałeśaleś ale to nic nie dało. Nadal nie dodaje refera do bazy

[pmir13]

W zasadzie linki referujące mają w urlu albo id osoby referującej albo nazwę.
Rozumiem, że nie pokazałeś tego fragmentu kodu, który wcześniej pobiera refa z $_GET
Później w pierwszych linijkach tego co widzimy sprawdzasz, czy jest taki user w bazie, ale co jeśli go nie ma?
Używasz konstrukcji $ref = costam( $ref ), czyli pozbywasz się starej wartości i jeśli nie dostaniesz nowej prawidłowej to nie masz nic.
Nie używasz mysql_error() więc nie widzisz nawet czy nie było problemów z wykonaniem zapytania.
Powinieneś używać co najmniej konstrukcji $res = mysql_query( $query ) or die ( mysql_error() );
albo najlepiej napisać sobie własną funkcję z lepszą kontrolą błędów.
No i oczywiście zawartość $_GET też trzeba dokładnie sprawdzić zanim złożysz z tego zapytanie do bazy.

[Lethys]

Id osoby polecającej pobiera ta zmienna:

[PHP] pobierz, plaintext 
$ref = mysql_fetch_array(mysql_query("select * from players where id='$ref'"));
[PHP] pobierz, plaintext 

z linku rejestracja.php?ref=1

$ref[id] będzie wynosiło 1



Cały kod wygląda tak:


Przy tym kodzie wypada błąd:

Notice: Undefined index: referujacy in /srv/www/vhosts/westernlife.pl/httpdocs/rejestracja.php on line 95

[PHP] pobierz, plaintext 
<?php 
error_reporting(E_ALL);
ini_set('display_errors', 1);
$problem = FALSE;
include "header.php";
include "left_sidebar.php";
$ref = mysql_fetch_array(mysql_query("select * from players where id='$ref'"));
?>
					<!-- end #left-sidebar -->
					<div id="content">
						<div class="box">
							<h2 class="title">Rejestracja</h2>
							<div class="content"> 
 
<form method=post action=rejestracja.php?action=register>
 
<table border="0" cellpadding="0" cellspacing="0" style="border-collapse: collapse" width="500" align="center" height="200">
<tr>
<td width="100%" align="center">
<table border="0">
<tr><td>Użytkownik:</td><td><input type="text" name="user" maxlength="10"></td></tr>
<tr><td>Email:</td><td><input type="text" name="email" maxlength="40"></td></tr>
<tr><td>Powtórz mail:</td><td><input type="text" name="vemail" maxlength="40"></td></tr>
<tr><td>Hasło:</td><td><input type="password" name="pass" maxlength="15"></td></tr>
<tr><td>Polecający ID:</td><td><?php echo("<input type=\"text\" name=\"referujacy\" value=\"$ref[id]\" disabled=\"disabled\" />"); ?></td></tr>
<tr><td></td><td><br><br><input type="submit" value="Rejestracja" name="submit"></td></tr>
</table>
</form>
<br>Rejestrując się akceptujesz <a href=regulamin.php>regulamin</a>
</td>
</tr>
</table>
 
<?php
if (isset ($_POST['submit'])){//isset wyslij
 
    include("config.php");
 
 
 
		if (!$user || !$email || !$vemail|| !$pass) {
			$problem = TRUE;
            echo("<p id='uwaga'><h2>Musisz wypełnić wszystkie pola.</h2></p>");
		}		
		if (10 < strlen($_POST['user'])) {
            echo("<p id='uwaga'><h2>Zbyt długa nazwa loginu. Maksymalnie <b>10</b> znaków.</h2></p>");
			$problem = TRUE;
        }
        if (strlen($_POST['email']) != strlen($_POST['vemail'])) {
            echo("<p id='uwaga'><h2>Podane przez Ciebie emaile się nie zgadzają. Potwierdzony email musi być taki sam.</h2></p>");
			$problem = TRUE;
        }
		$test1 = mysql_num_rows(mysql_query("select * from players where user='".$_POST['user']."'"));
			if ($test1 > 0) {
            echo("<p id='uwaga'><h2>Ktoś już użył takiej nazwy użytkownika, musisz wybrać inną.</h2></p>");
			$problem = TRUE;
	    }
		$test2 = mysql_num_rows(mysql_query("select * from players where email='".$_POST['email']."'"));
		if ($test2 > 0) {
            echo("<p id='uwaga'><h2>Ktoś już użył takiego adresu email. Musisz użyć innego adresu.</h2></p>");
			$problem = TRUE;
	    }
		$test3 = mysql_num_rows(mysql_query("select * from gracze where user='".$_POST['user']."'"));
			if ($test3 > 0) {
            echo("<p id='uwaga'><h2>Ktoś już użył takiej nazwy użytkownika, musisz wybrać inną.</h2></p>");
			$problem = TRUE;
	    }
		$test4 = mysql_num_rows(mysql_query("select * from gracze where email='".$_POST['email']."'"));
		if ($test4 > 0) {
            echo("<p id='uwaga'><h2>Ktoś już użył takiego adresu email. Musisz użyć innego adresu.</h2></p>");
			$problem = TRUE;
	    }
        if (15 < strlen($_POST['pass'])) {
            echo("<p id='uwaga'><h2>Twoje hasło jest za długie. Maksymalnie możesz użyć <b>15</b> znaków.</h2></p>");
			$problem = TRUE;
        }
		if(!preg_match('/^([a-ż0-9_]+)$/i', $_POST['user'])){
		$problem = TRUE;
        echo("<p id='uwaga'><h2>Użyto niedozwolonych znaków w nazwie użytkownika.</h2></p>");
		}
		if(!preg_match('/^([a-ż0-9_]+)$/i', $_POST['pass'])){
		$problem = TRUE;
        echo("<p id='uwaga'><h2>Użyto niedozwolonych znaków w haśle.</h2></p>");
		}
 
 
        if (!$problem){
 
 
 
		$data = date("d.m.y");
		$haslo = md5($pass);
		$kod = rand(100000000,999999999);
		$ref = intval($_POST['referujacy']);
 
 
 
	mysql_query("insert into gracze (user, email, pass, data, kod, ref) values('$user','$email','$haslo','$data','$kod','$ref')") or die(mysql_error());
 
 
 
                         echo("<br><p><h2>Zostałeś zarejestrowany, odbierz pocztę i aktywuj konto.</h2></p>");
 
        }else{
              echo("<br><p>Wróć do <a href=rejestracja.php>poprzedniej strony</a> i spróbuj ponownie.</p>");
             }                    
    }
 
?>
[PHP] pobierz, plaintext 

Nie mam pojęcia co już tutaj poprawić, może mi ktoś pomóc ogarnąć to ?

[nospor]

Nie chciałbym dać normalnego pola bo użytkownicy będą mogli gmerać przy refie.

Przecież disabled też bez problemu można zmienić
Daj hidden i po sprawie.

[pmir13]

Masz na serwerze ustawione register_globals on, więc do zmiennych możesz dostać się bezpośrednio, niezależnie czy jest to get, post czy inne, a to mocno utrudnia połapanie się w kodzie. Dużo lepiej byłoby gdybyś uczył się bez takich "ułatwień".
Zmienne podanych w URLu są w tablicy $_GET, czyli na przykład jeśli masz adres
http://www.westernlife.pl/rejestracja.php?ref=12573
to powinieneś dostać wartość 12573 w $_GET['ref']

[PHP] pobierz, plaintext 
$problem = 0;
if( !isset($_GET['ref'] ) )
  $_GET['ref'] = 0;
$referujacy = $_GET['ref'];
[PHP] pobierz, plaintext 

Następnie sprawdzasz wartość $referujacy pod kątem możliwości grzebania, przepuszczając wyłącznie 1 do 6 cyfr z zakresu 0-9, chyba nie planujesz więcej niż milion graczy.

[PHP] pobierz, plaintext 
if( !preg_match( "/^([0-9]){1,6}$/", $referujacy ) )
  $referujacy = 0;
[PHP] pobierz, plaintext 

Następnie sprawdzasz czy taki user jest w bazie danych

[PHP] pobierz, plaintext 
$query = "SELECT user FROM players WHERE id=$referujacy";
$res = mysql_query( $query ) or die( mysql_error() );
$player = mysql_fetch_row( $res ) ;
if( $player )
    $ref_username = $player[0];
else
{
    $ref_username = "";
    $referujacy = 0;
}
[PHP] pobierz, plaintext 

Do bazy bedziesz wrzucac jego id, czyli $referujacy, a $ref_username może zostać do wyswietlenia w formularzu w polu disabled jeśli bardzo tego pragniesz.
Następnie sprawdzamy co trzeba zrobić, czy wyświetlić formularz czy może od razu rejestrować.

[PHP] pobierz, plaintext 
if( !isset ($_GET['action'] ) )
  $_GET['action'] = 0;
$action = $_GET['action'];
if( $action == "register" )
{
  // sprawdzamy poprawność wprowadzonych pól, ustawiamy $problem jeśli coś nie tak
  // wyświetlamy odpowiedni komunikat lub rejestrujemy
}
if( $problem || $action != "register" )
{
  // wyświetlamy formularz do rejestracji
}
[PHP] pobierz, plaintext 

[kill15]

Przeczytaj co Ci tu napisze, trochę nie kontaktuje ale może Cię dobrze zrozumiałem

[PHP] pobierz, plaintext 
<?php
$lol = $_GET[raz];
 
$spr = mysql_query("SELECT id FROM players WHERE user = '$lol' ");
$spr1 = mysql_fetch_assoc($spr);
$spr2 = $spr1[id];
 
if($spr2 = FALSE)  //
{
$referujacy = "0";
}
else
{
$referujacy = $lol;
}
 
?>
<form action="mama_mnie_bije_bo_jest_koksem.mamusia"> 
 
<input type="text" name="krolik" value="<?php echo $lol; ?>" />
</form> 
[PHP] pobierz, plaintext 

Dlaczego ukrywasz tutaj polecającego po co
i pod linkiem:

www.mojamamamniebije.pl/bije.php?raz=IMIEMOJEJMAMY

oczywiście:
mojamamamniebije - adres strony
bije - plik twój
IMIEMOJEJMAMY - to będzie oczywiście polecajacy
Pokładaj sobie to i zmodyfikuj. No i powinno Ci działać ;>
Jeżeli disabled nie wyślesz a nie chcesz żeby ktoś się przy tym polu bawił to może usuń to pole i wyślij ze zmiennej. albo

[PHP] pobierz, plaintext 
<?php
if(!isset($lol) AND empty($lol) AND $lol != "")
{
?>
<input type="text" name="krolik" value="" > // Chyba jak nie ma z linku to chodzi o to żeby wpisać nieprawdaż /?
<?php
$referujacy = $_POST[krolik];
}
else
{
$referujacy = $lol;
}
?>
[PHP] pobierz, plaintext 

Ten post edytował kill15 13.04.2011, 22:39:29