Witam wszystkich.
Otóż jestem w trakcie pisania prostej strony, która pozwala użytkownikowi zaadoptować zwierzątko i rozwijać je.

Nie mam kompletnie pomysłu jak miałby wyglądać skrypt adopcji. Chodzi o to, że użytkownik zarejestrowany oczywiście może zadoptować maxymalnie 3 zwierzątką.

np.

1x Amstaff
1x York
1x Ratlerek

Każdy nowo zadoptowany zwierzaczek ma domyślny poziom 1.
Potem napiszę skrypt, który umożliwi dokarmianie ich.

I tu pojawia się moje pytanie jak najlepiej wykonać taki skrypt adopcji.

Jakie pola do bazy, nowa tabelka?
Od czego zacząć ten skrypt - od sprawdzenia czy jest zalogowany to na pewno.

90% pracy za tobą....

Tabele:
user
zwierzak
user_zwierzak - ta tabela będzie łączyć usera z adoptowanym zwierzakiem. Ma miec pola:
ID_USER
ID_ ZWIERZAK

No to jedziemy z kolejnymi - pewnie dla Was banalnymi pytaniami.

Tabela users:
ID, Login, Hasło, Email.

Tabela adoptions:
login, zwierzak, id_login, id_zwierzak

Mniej więcej tak to miałoby wyglądać?
User u Ciebie to login u mnie.

ID_Login oraz ID_Zwierzak mają się automatycznie dodawać?

W tabeli adoptions te dwa pola:
login, zwierzak,
są totalnie zbędne.

No Twój skrypt ma je tworzyć gdy ktos będzie adoptował zwierzaka. Wówczas ty masz wygenerować rekord w tabeli adoptions

[PHP] pobierz, plaintext 
CREATE TABLE `adoptions` (
  `id_login` mediumint(8) NOT NULL auto_increment,
  `id_pet` mediumint(8) NOT NULL auto_increment,
  `pet_name` varchar(25) NOT NULL,
  PRIMARY KEY  (`id_pet`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin2 AUTO_INCREMENT=2 ;
[PHP] pobierz, plaintext 

id_login musi być takie same jak id usera z tabelki users.
id_pet będzie tworzyło się automatycznie gdy użytkownik stworzy nowego zwierzaczka.
pet_name użytkownik będzie za pomocą formularzu nadawał imię zwierzaczkowi. Max 25 znaków.

Myślę, że dobrze zrozumiałem?

źle...
w tabeli adoptions nie ma być żadnych autoincrement. Wartości tych pól będziesz nadawał Ty.
Kluczem głównym ma być klucz złozony (id_login, id_pet)

pet_name - ok

Sądziłem, że zwierzaki już będziesz miał w bazie. Czyli masz np. w tabeli PET zwierzaki:
tygrys,
małpa
wąz
.....

A koleś wybiera z nich co adoptuje

[PHP] pobierz, plaintext 
CREATE TABLE `adoptions` (
  `id_login` mediumint(8) NOT NULL,
  `id_pet` mediumint(8) NOT NULL,
  `pet_name` varchar(25) NOT NULL,
) ENGINE=MyISAM  DEFAULT CHARSET=latin2 AUTO_INCREMENT=2 ;
[PHP] pobierz, plaintext 

Czy po prostu z varcharem mam to zrobić - co jak co, ale w zapytaniach jestem cieńki.



Hmm tym systemem staram wzorować się na skrypcie Smoki.Nightwood.net po wejściu w adopt.php użytkownik widzi 3 pola.

- Wąż
- Tygrys
- Żółw

No i wybiera sobie np. Węża i klika dalej, a potem ma puste pole, w ktorym może wpisać pet_name - czyli nazwę zwierzątka.
No i oczywiście na jednego użytkownika przypada max 3 zwierzątka.

Ok, czyli nie mamy do wyboru zwierzaków jako takich tylko rodzaj zwierzaka.
Tabela adoptions:
ID - klucz główy, autoincrement
ID_LOGIN
ID_PET_TYPE - czyli typ zwierzaka. ID z tabeli PET_TYPE
PET_NAME

[PHP] pobierz, plaintext 
CREATE TABLE `adoptions` (
  `id` mediumint(8) NOT NULL auto_increment,
  `id_login` varchar(25) NOT NULL,
  `pet_type` varchar(25) NOT NULL,
  `pet_name` varchar(25) NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin2 AUTO_INCREMENT=2 ;
[PHP] pobierz, plaintext 

Nie jestem pewny czy tak - zaraz otworzę manual'a mysql..

id_login
pet_type

To mają być pola liczbowe a nie varchar. Te pola to ID z odpowiedających im tabel. Nie kombinuj, tylko pisz poprawnie.

I przestan dla pól liczbowych używać (8). To niczemu nie służy.

Aa, faktycznie.

[PHP] pobierz, plaintext 
CREATE TABLE `adoptions` (
  `id` mediumint(8) NOT NULL auto_increment,
  `id_login` INT (25) NOT NULL,
  `pet_type` INT (25) NOT NULL,
  `pet_name` varchar(25) NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin2 AUTO_INCREMENT=2 ;
[PHP] pobierz, plaintext 

Tzn. 8 jest zbędna?

INT (25)
(25) Tez jest zbędne. Każdla liczba w nawiasie dla pól liczbowych jest zbędna
http://nospor.pl/int11-kontra-int.html

[PHP] pobierz, plaintext 
CREATE TABLE `adoptions` (
  `id` mediumint unsigned NOT NULL auto_increment,
  `id_login` int unsigned NOT NULL,
  `pet_type` int unsigned NOT NULL,
  `pet_name` varchar(25) NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin2 AUTO_INCREMENT=2 ;
[PHP] pobierz, plaintext 

Zastosowałem się do wyjaśnień z Twojej strony

A czy w tabeli LOGIN id to jest INT czy może MEDIUMINT ? Bo wcześniej pisałeś mediumint. Zdecyduj sie.

Czy typow zwierzaków będziesz miał naprawdę aż kilka miliardów, że dajesz im TYP INT? Myśle, ze tinyint z powodzeniem wystarczy.

Nie, nie aż tak dużo to nie będzie z 3-4 typy.

[PHP] pobierz, plaintext 
CREATE TABLE `adoptions` (
  `id` mediumint unsigned NOT NULL auto_increment,
  `id_login` mediumint unsigned NOT NULL,
  `pet_type` tinyint unsigned NOT NULL,
  `pet_name` varchar(25) NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin2 AUTO_INCREMENT=2 ;
[PHP] pobierz, plaintext 

No i super.

Jedyne co mnie draźni to nazwa id_login. Toż to powinno być id_user. Masz tabele user a nie tabele login. Login to pole w tabeli user

A, właśnie. Co z domyślnymi poziomami ? Nowo stworzony zwierzaczek powinien mieć jeden. Czyli stworzyć nową kolumnę

level i ustawić domyślnie 1 ?

@up
Poprawione

Tak. Pole level typu tinyint unsigned - zakładam, że więcej niż 255 poziomów mieć nie będziesz. No chyba ,że będziesz, to wówczas daj większe pole

Oj poziomów będzie dużo więcej.

[PHP] pobierz, plaintext 
CREATE TABLE `adoptions` (
  `id` mediumint unsigned NOT NULL auto_increment,
  `id_user` mediumint unsigned NOT NULL,
  `pet_type` tinyint unsigned NOT NULL,
  `pet_name` varchar(25) NOT NULL,
  `level` mediumint unsigned NOT NULL default '1',
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8_general_ci AUTO_INCREMENT=2 ;
[PHP] pobierz, plaintext 

nie: default '1'
a: default 1
Skoro dopiero zaczynasz, to ucz się pisać poprawnie. 1 to liczba a nie tekst.

http://dev.mysql.com/doc/refman/5.0/en/integer-types.html

Kurde, co to za gra, w której ktoś może dojśc do 16mln poziomów? WOW
Wiem, czepiam się, ale nie lubie bezsensu nadmiarowych pól.

Czy max 65 tys nie jest już wystarczające?

[PHP] pobierz, plaintext 
// Sprawdzamy czy nazwa zwierzęcia zawiera dozwolone znaki.
if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $pet_name)) {
     else { 
          exit("<center>Imię zwierzęcia zawiera niedozwolone znaki!</center>"); 
     } 
// Jeśli wszystko jest ok tworzymy rekord.
     else { 
          $insert = "INSERT INTO `adoptions` ('id_user', `pet_name`, `pet_type`) VALUES ('".$id_user."','".$pet_name."', '".$pet_type."')"; 
          $result = mysql_query($insert) or die("<center>Niestety nieudało się zadoptować zwierzątka.</center>"); 
          echo "<center>Gratulacje adopcja powiodła się.</center>"; 
     }
     $result2 = mysql_query("SELECT `users` (`id`)");
     if (!$result2) {
     echo 'Nie można uruchomić zapytania: ' . mysql_error();
     exit;
}
$id_user = mysql_fetch_row($result2);
}
[PHP] pobierz, plaintext 

Parse error: syntax error, unexpected T_ELSE in D:\Program Files\WebServ\httpd\adopt.php on line 37

Wszystko pozamykane, a więc, albo jestem ślepy, albo coś popsułem.

Po pierwszym ifie masz od razu else

No tak, ale co w związku z tym?

No przecież to jest błąd składni:

[PHP] pobierz, plaintext 
// Sprawdzamy czy nazwa zwierzęcia zawiera dozwolone znaki.
if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $pet_name)) {
     else { 
          exit("<center>Imię zwierzęcia zawiera niedozwolone znaki!</center>"); 
     } 
[PHP] pobierz, plaintext 

robisz if() {
// a tu od razu else - tak nie może być

Faktycznie dzięki.

Jest teraz inny problem w kodzie mam

[PHP] pobierz, plaintext 
// Sprawdzamy czy nazwa zwierzęcia zawiera dozwolone znaki.
if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $pet_name)) {
     echo ('<center>Imię zwierzęcia zawiera niedozwolone znaki!</center>');
     } 
[PHP] pobierz, plaintext 

Po wpisaniu w pole pet_name obojętnie jakiego wyrazu - bez polskich znaków oraz znaków specjalnych wywala mi, że imię jest nieprawidłowe pomimo, że wpisuje dobre. W czym może tkwić problem?

Ten if:

[PHP] pobierz, plaintext 
// Sprawdzamy czy nazwa zwierzęcia zawiera dozwolone znaki.
if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $pet_name)) {
     echo ('<center>Imię zwierzęcia zawiera niedozwolone znaki!</center>');
     }
[PHP] pobierz, plaintext 

oznacza mniej więcej tyle:

jeśli $pet_name będzie zawierać przynajmniej jedną literę, cyfrę, bądź znak podkreślenia to wyświetl tekst: "Imię zwierzęcia...."
Także jeśli wpiszesz samo: "_" to też ci wywali że imię jest nieprawidłowe.

Czyli, że musiałbym ten kod przerobić?

Nawet jeśli dam tak:

[PHP] pobierz, plaintext 
if (preg_match("@^(?:http://)?([^/]+)@i" , $pet_name)) {
[PHP] pobierz, plaintext 

To też wywala.

Chcę, aby nie można było wpisywać znaków specjalnych /.,!@#$%^&*();:' lub " , a żeby litery i cyfry działały normalnie.


Hmm

To spróbuj tak:

[PHP] pobierz, plaintext 
// Sprawdzamy czy nazwa zwierzęcia zawiera dozwolone znaki.
if (!preg_match("/^[a-z0-9]+$/i" , $pet_name)) {
     echo ('<center>Imię zwierzęcia zawiera niedozwolone znaki!</center>');
}
[PHP] pobierz, plaintext 

Do pierwszego wyrażenia daj operator negacji !.
if(!preg...
przepuści on w $pet_name wielkie, małe litery, cyfry oraz stópkę_

Działa dzięki piękne.

Poniższy kod ma dodawać rekord z bazy danych ten drugi ma sprawdzać czy ID usera jest takie same jak ID w tabelce users. Coś nie działa :/

[PHP] pobierz, plaintext 
// Jeśli wszystko jest ok tworzymy rekord.
     else { 
          $insert = "INSERT INTO `adoptions` ('id_user', `pet_name`, `pet_type`) VALUES ('".$id_user."','".$pet_name."', '".$pet_type."')"; 
          $result = mysql_query($insert) or die("<center>Niestety nieudało się zadoptować zwierzątka.</center>"); 
          echo "<center>Gratulacje adopcja powiodła się.</center>"; 
     }
     $result2 = mysql_query("SELECT * FROM users WHERE id = '$id_user'");
     if (!$result2) {
     echo 'Nie można uruchomić zapytania: ' . mysql_error();
     exit;
}
$id_user = mysql_fetch_row($result2);
[PHP] pobierz, plaintext 

A co nie działa? co się wyświetla?

Temat: Jak poprawnie zadac pytanie

Notice: Undefined variable: id_user in D:\Program Files\WebServ\httpd\adopt.php on line 3

[PHP] pobierz, plaintext 
// Jeśli wszystko jest ok tworzymy rekord.
     else { 
          $insert = "INSERT INTO `adoptions` ('id_user', `pet_name`, `pet_type`) VALUES ('".$id_user."','".$pet_name."', '".$pet_type."')"; 
          $result = mysql_query($insert) or die("<center>Niestety nieudało się zadoptować zwierzątka.</center>"); 
          echo "<center>Gratulacje adopcja powiodła się.</center>"; 
     }
     $result2 = mysql_query("SELECT * FROM users WHERE id = '$id_user'");
     if (!$result2) {
     echo 'Nie można uruchomić zapytania: ' . mysql_error();
     exit;
}
$id_user = mysql_fetch_row($result2);
[PHP] pobierz, plaintext 

Powyzszy kod ma dodawać rekord z bazy danych ten drugi ma sprawdzać czy ID usera jest takie same jak ID w tabelce users. Coś nie działa :/

Pokaż bazę danych...

Bo skrypt , że tak powiem nie wie co to jest: id_user

[PHP] pobierz, plaintext 
CREATE TABLE `users` (
  `id` mediumint(8) NOT NULL auto_increment,
  `login` varchar(25) NOT NULL,
  `haslo` varchar(32) NOT NULL,
  `email` varchar(255) NOT NULL,
  `rank` tinyint(4) NOT NULL default '0',
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=2 ;
 
CREATE TABLE `adoptions` (
  `id` mediumint unsigned NOT NULL auto_increment,
  `id_user` mediumint unsigned NOT NULL,
  `pet_type` tinyint unsigned NOT NULL,
  `pet_name` varchar(25) NOT NULL,
  `level` int unsigned NOT NULL default 0,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=2 ;
[PHP] pobierz, plaintext 

ID z tabelki users musi być takie same w tabelce adoptions.

Dlatego taki kod jest tego tworzenia adopcji. ^^

[PHP] pobierz, plaintext 
$sql = 'INSERT INTO adoptions SET id_user="'.$id_user.'", pet_name="'.$pet_name.'", pet_type="'.$pet_type.'"';
[PHP] pobierz, plaintext 

spróbuj ten kod bo mi sie tamtego nie che poprawiać...

Dodaje rekord, ale nie pobiera kolumna id_user jest pusta, a powinna zawierać moje id, które wyciągam z tabelki users.

No i można tworzyć konta bez podania danych pomimo, że mam zdefiniowane

[PHP] pobierz, plaintext 
// Sprawdzamy czy pola zostały uzupełnione.
if(empty($pet_name)) {
     echo ("<center>Proszę uzupełnić polę imię zwierzęcia.</center>");
}
[PHP] pobierz, plaintext 

Zastosuj np. exit

[PHP] pobierz, plaintext 
if(empty($pet_name)) {
echo ("<center>Proszę uzupełnić polę imię zwierzęcia.</center>");
exit;
}
[PHP] pobierz, plaintext 

albo przypisuj jakiejś zmiennej falsh:



Albo dodaj metodę błędów...
Ps;
Porąbało mi się z niemieckim ma być
false

Mam takie opcje w wyboru formularzu.

[PHP] pobierz, plaintext 
$pet_type = strip_tags(mysql_escape_string($pet_type));
[PHP] pobierz, plaintext 

Tutaj flirtuje dane.

[PHP] pobierz, plaintext 
$insert = 'INSERT INTO adoptions SET id_user="'.$_SESSION['id'] .'", pet_name="'.$pet_name.'", pet_type="'.$pet_type.'"';
[PHP] pobierz, plaintext 

A, tym zapytaniem dodaje rekord do bazy. No i mam problem, a mianowicie. Wszystko działa sprawnie oprócz pet_type po prostu dodaje mi do kolumny wartość 0, a powinno dodawać Samiec bądź Samica. W czym może tkwić problem?

sprawdz co zawiera $pet_type przed $insert

Nic nie wyświetla, a dodałem. Lecz po wyborze - Samiec / Samica wyświetla mi dopiero po założeniu konta wynik.

A skąd idzie $pet_type ? Nie powinieneś mieć tam $_POST['pet_type'] albo $_GET['pet_type'] ?

[PHP] pobierz, plaintext 
    $pet_type = strip_tags(mysql_escape_string($_POST['pet_type']));
[PHP] pobierz, plaintext 

Niestety to nic nie dało.
Problem tkwił w błędnym umiejscowionym <?php. Już sobie poradziłem i skrypt działa w pełni sprawnie.
Temat do zamknięcia.