Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP] Problem z kupowaniem przedmiotow w sklepie
Forum PHP.pl > Forum > Przedszkole
Rafiks1992
30.06.2017, 10:32:21
Siemka ponownie wink.gif dotarlem do kolejnego etapu, wczoraj dodalem nowa tabele o nazwie shop w ktorej umiescilem 3 kolumny id, cena, nazwa a takze dodalem tam 4 losy do kupienia: mikrolos, minilos, megalos, hiperlos ustawiajac oczywiscie id na A.I a takze ceny tych losow i nazwy. W tabeli users dodalem takze kolumny: mikrolos, minilos, megalos, hiperlos aby miec mozliwosc wyswietlenia ile dany uzytkownik posiada tych losow. Poogarnialem wszystko i zmienilem w mysql losowe dane dla kazdego goscia aby sprawdzic czy dziala i faktycznie wyswietla ilosc losów w Twój profil. Tylko teraz mam problem ze skryptem kupowania kolejnych losów. Stworzylem nowy plik i nazwie shop.php z takim kodem:

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. $title = "Sklep";
  4. include_once 'header.php';
  5.  
  6. if (!$session->is_logged_in()) {
  7. 	$session->message("Musisz sie zalogowac!", "success");
  8. 	redirect_to("login.php");
  9. }
  10. ?>
  11.  
  12. <h1>Sklep</h1>
  13.  
  14. <?php 
  15. echo 'Witamy w sklepie, mozesz tutaj kupic losy.';
  16. ?> <br><br><br>
  17.  
  18. <?php
  19. $userdata = User::getUser($session->user_name);
  20. echo 'Masz aktualnie: ' .$userdata->money. 'zl.';
  21. ?> <br>
  22.  
  23. <?php 
  24. echo 'Kup:';
  25. ?> <br>
  26.  
  27. <?php 
  28.  
  29. if(isset($_POST['kup_los'])) {
  30. 	if(!empty($_POST['mikrolos'])) {
  31. 		$mikrolos = ($_POST['mikrolos']*$cost);
  32. 		if($money >= $mikrolos) {
  33. 			mysql_query("UPDATE users SET mikrolos='".$_POST['mikrolos']."' WHERE id=".$pokaz['id']);
  34. 			mysql_query("UPDATE users SET money=money-".$cost." WHERE id=".$pokaz['id']);
  35. 		} else {
  36. 			echo "Nie stać cie";
  37. 		}
  38. 	}
  39. }
  40.  
  41.  
  42.  
  43. ?>
  44.  
  45. <form action='' method='post' class='form'>
  46. Mikrolos: <input type='text' name='mikrolos' value=''>
  47. <button name='kup_los' type='submit'>Kup</button>
  48. </form> 
  49.  
  50. <?php
  51.  
  52. include_once 'footer.php'; 
  53.  
  54. ?>
[PHP] pobierz, plaintext



Zrobilem narazie tylko ten pierwszy los czyli mikrolos do kupienia, jednak gdy na stronie w formularzu wpisze chocby 1 - a stac mnie na ten los to nic sie nie dzieje, moglby mi ktos wyjasnic dlaczego?
trueblue
30.06.2017, 10:37:49
1. Jaką wartość ma zmienna $cost?
2. Ilości losów nie powinny być przechowywane w kolumnach o tych nazwach. To ma być odrębna tabela z kolumnami: id, los, ilosc.
viking
30.06.2017, 10:38:50
Pewnie połączenia z bazą nawet nie masz. Wczoraj w kodzie User było przekazywane jakieś global $database.
Rafiks1992
30.06.2017, 10:42:31
Hm czyli na samym poczatku musze jeszcze dodac sprawdzenie czy jest polaczenie z baza danych, zaraz sie sprobuje tym zajac.
Zmienna $cost powinna miec wartosc z tablicy shop z kolumny cena i jest taka linijka dodana juz tutaj:
$cost = ($_POST['mikrolos']);
Twojej drugiej uwagi troszke nie rozumiem, dlaczego nie moze byc tak zrobione?


EDIT:
dodalem na poczatku kodu cos takiego:
require_once 'includes/config.php'; - czyli wymaga polaczenia z baza danych
trueblue
30.06.2017, 10:47:47
Na tym polega relacyjność baz danych.
Wyobraź sobie, że zmieniasz nazwę losu lub dodajesz kolejny.
Rafiks1992
30.06.2017, 10:50:17
Czyli jak rozumiem musze wywalic z users te kolumny z losami ktore utworzylem tzw. ekwipunek zrobic nowa tabele o nazwie ekwipunek i tam dac kolumny id nazwa_losu oraz ilosc tak?
viking
30.06.2017, 10:51:10
Ja bym ci polecał na początek zrobić sobie zupełnie niezależnie prosty formularz, poczytać o bazach danych i za pomocą pdo spróbować dodać / wybierać rekordy. Zapożyczyłeś jakiś obcy kod którego jeszcze nie rozumiesz i starasz się na nim pracować. Wczoraj nawet myliłeś kolumny z tabelami.
Rafiks1992
30.06.2017, 22:04:04
Probuje uczyc sie piszac i praktykujac moze stad tyle prostych bledow tongue.gif tak czy inaczej dobre naprowadzenie ze zrobieniem nowej tabeli oczywiscie stworzylem nowa tabele o nazwie eq i dalem w niej kolumny id, uid, name, count - powinno byc lepiej tylko teraz to polaczyc

New:


Wzialem sobie troszke do serca Twoja porade ale walcze takze z moja praktyka bo lubie uczyc sie piszac a nie siedziec i czytac ksiazki dlatego znalazlem na youtube filmik jak gosc pisze gre via www i tworzy tam sklep opisuje krok po kroku jak to robi i w sumie wszystko wyglada o wiele przejrzysciej i lepiej jak obejrzalem i porobilem podobnie do niego ale jednak nie dziala mi kilka rzeczy i moze tutaj ktos by cos dopatrzyl?

kod aktualnie wyglada tak: shop.php

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. require_once 'includes/config.php';
  4.  
  5. $title = "Sklep";
  6. include_once 'header.php';
  7.  
  8. if (!$session->is_logged_in()) {
  9. 	$session->message("Musisz sie zalogowac!", "success");
  10. 	redirect_to("login.php");
  11. }
  12.  
  13. ?>
  14.  
  15. <h1>Sklep</h1>
  16.  
  17. <?php 
  18. echo 'Witamy w sklepie, mozesz tutaj kupic losy do naszych gier.';
  19. ?> <br><br><br>
  20.  
  21. <?php
  22. $userdata = User::getUser($session->user_name);
  23. echo 'Masz aktualnie: ' .$userdata->money. 'zl.';
  24. ?> <br>
  25.  
  26. <?php 
  27. echo 'Kup:';
  28. ?> <br>
  29.  
  30. <?php 
  31.  
  32. function kup_los($id){
  33. 	$userdata = User::getUser($session->user_name);
  34. 	$tekst='Niewykonana funkcja';
  35. 	$los=mysql_query("SELECT * FROM shop WHERE id=$id LIMIT 1");
  36. 	if (mysql_num_rows($los)===1){
  37. 		$los=mysql_fetch_assoc($los);
  38. 		$user=mysql_query("SELECT * FROM users, eq WHERE users.id=eq.uid LIMIT 1");
  39. 		$user=mysql_fetch_assoc($user);
  40. 		var_dump($los,$user);
  41. 		if ($user['money']>=$los['cost']){
  42. 			$tekst='Przedmiot kupiony!';
  43. 			mysql_query("INSERT INTO eq (id,uid,name,cost,count) VALUES ('$user','$los[id]','$los[name]','$los[cost]')");
  44. 		}
  45. 	}
  46. 	return $tekst;
  47. }
  48.  
  49. echo kup_los(1);
  50.  
  51. ?>
  52.  
  53.  
  54.  
  55. <?php
  56.  
  57. error_reporting(E_ALL);
  58. ini_set('display_errors', 1);
  59. ?>
  60.  
  61. <?php
  62.  
  63. include_once 'footer.php'; 
  64.  
  65. ?>
[PHP] pobierz, plaintext


A wiec do rzeczy, kolega ktory polecil mi tutaj w poscie dodanie osobnej tabeli z kolumnami id cena ilosc mial bardzo trafiony pomysl, zrobilem tak a takze dodalem jeszcze jedna nowa rzecz (ktora mi wlasnie nie dziala biggrin.gif) uid - user id. Musi to byc poniewaz w tej tabeli musze wiedziec ktory uzytkownik ile ma jakich losów, logiczne oczywiscie. Dodatkowo jest wykonanie funkcji kup_los z id 1 - w tabeli shop pod id1 jest mikrolos, dalem tak specjalnie dla sprawdzenia czy po kliknieciu sklep i odswiezeniu dodaje rekord i kupuje ten wlasnie los i tak jest, tylko teraz jedna rzecz mnie martwi - w mysql sprawdzam dodany rekord i wszystko ladnie pieknie dziala poza wlasnie dodaniem prawidlowego UID - dodaje jako 0 tylko dlaczego? edit: zapomnialem jeszcze jedno, dodalem var_dumpa aby sprawdzic co wyswietla mi on pod $los a takze pod $user i $los wyswietla wszystkie dane prawidlowo z tablicy ale $user wyswietla mi dane uzytkownika o id1 a jak zaloguje sie na inne konto ktore sobie sam oczywiscie utworzylem ma id 2 ale wciaz wyswietla mi dane z tego pierwszego konta
trueblue
30.06.2017, 22:07:45
Przejrzyj dokładnie zapytanie za pomocą, którego dodajesz zakupiony los.
Przy okazji. Nazwa losu nie powinna być przechowywana w tej tabeli.
Rafiks1992
30.06.2017, 22:31:01
kurcze no przejrzalem ten filmik jeszcze raz spogladlem na kod i jest bardzo podobnie jak u niego, lecz u niego dziala a u mnie nie ^^
viking
1.07.2017, 05:51:18
Tak jak ci mówiłem, zacznij od przyjrzenia się co robi klasa database - w metodzie User::getUser zmienna $database. Bo na razie wrzucasz zupełnnie oddzielnie funkcje mysql_ których już nie ma nawet w PHP.
Rafiks1992
1.07.2017, 09:13:12
Jak to nie ma w php tongue.gif? masz na mysli ze wrzucam komendy np z php 4.5 czy jakiegos tam a teraz trzeba 7.1 biggrin.gif? Cholercia to bardzo niedobrze robie jesli faktycznie tak jest, co do tego user $database wrzucam plik userclass.php aby to przyblizyc poniewaz szczerze mowiac jest tutaj nawalone tyle kodu ze nie ogarnalem polowy

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. class User {
  4.  
  5.     protected static $table_name = "users";
  6.     protected static $db_fields = array('id', 'username', 'password', 'email', 'status', 'actcode' ,'created_at' ,'money');
  7.  
  8.     public $id;
  9.     public $username;
  10.     public $password;
  11.     public $email;
  12.     public $status = "pending";
  13.     public $actcode;
  14.     public $created_at;
  15.     public $money;
  16.  
  17.     public static function authenticate($username = "", $password = "") {
  18.         global $database;
  19.  
  20.         $username = $database->escape_value($username);
  21.         $password = $database->escape_value($password);
  22.         $password = secure_string($password);
  23.  
  24.         $sql = "SELECT * FROM " . self::$table_name . " users WHERE username = '{$username}' AND password = '{$password}' LIMIT 1";
  25.         $result_array = self::find_by_sql($sql);
  26.         return !empty($result_array) ? array_shift($result_array) : false;
  27.     }
  28.  
  29. 	public static function activate($username, $actcode) {
  30. 		global $database;
  31. 		global $session;
  32.  
  33. 		$username = $database->escape_value($username);
  34.         $actcode = $database->escape_value($actcode);
  35.  
  36. 		$sql = "SELECT status FROM users WHERE username = '{$username}' AND actcode = '{$actcode}' AND status = 'pending' LIMIT 1";
  37. 		$database->query($sql);
  38. 		if($database->affected_rows() == 1) {
  39. 			$sql = "UPDATE users SET status = 'confirmed' WHERE username = '{$username}' AND actcode = '{$actcode}'";
  40. 			$result = $database->query($sql);
  41.  
  42. 			if($result) {
  43. 				return true;
  44. 			} else {
  45. 				return false;
  46. 			}} else {
  47. 			return false;
  48. 		}
  49. 	}
  50.  
  51. 	public static function getUser($username) {
  52. 		global $database;
  53.  
  54. 		$username = $database->escape_value($username);
  55.  
  56. 		$sql = "SELECT * FROM users WHERE username = '{$username}' LIMIT 1";
  57. 		$result_array = self::find_by_sql($sql);
  58. 	//	var_dump($result_array);
  59.         return !empty($result_array) ? array_shift($result_array) : false;
  60.  
  61. 	}
  62.  
  63. 	public static function user_email($username, $email) {
  64. 		global $database;
  65.  
  66. 		$username = $database->escape_value($username);
  67. 		$email = $database->escape_value($email);
  68.  
  69. 		$sql = "SELECT username FROM users WHERE username = '{$username}' AND email = '{$email}' LIMIT 1";
  70. 		$database->query($sql);
  71. 		if($database->affected_rows() == 1) {
  72. 			return true;
  73. 		} else 
  74. 			return false;
  75. 	}
  76.  
  77. 	public static function exists($where,$username) {
  78. 		global $database;
  79.  
  80. 		$username = $database->escape_value($username);
  81. 		$where = $database->escape_value($where);
  82.  
  83. 		$sql = "SELECT '{$where}' FROM users WHERE {$where} = '{$username}' LIMIT 1";
  84. 		$database->query($sql);
  85. 		if($database->affected_rows() == 1) {
  86. 			return true;
  87. 		} else 
  88. 			return false;
  89. 	}
  90.  
  91.  
  92.     public static function find_all() {
  93.         return self::find_by_sql("SELECT * FROM " . self::$table_name);
  94.     }
  95.  
  96.     public static function find_by_id($id = 0) {
  97.         $result_array = self::find_by_sql("SELECT * FROM " . self::$table_name . " WHERE id={$id} LIMIT 1");
  98.         return !empty($result_array) ? array_shift($result_array) : false;
  99.     }
  100.  
  101.     public static function find_by_sql($sql = "") {
  102.         global $database;
  103.         $result_set = $database->query($sql);
  104.         $object_array = array();
  105.         while($row = $database->fetch_array($result_set)) {
  106.             $object_array[] = self::instantiate($row);
  107.         }
  108.         return $object_array;
  109.     }
  110.  
  111.     public static function count_all() {
  112.         global $database;
  113.         $sql = "SELECT COUNT(*) FROM " . self::$table_name;
  114.         $result_set = $database->query($sql);
  115.         $row = $database->fetch_array($result_set);
  116.         return array_shift($row);
  117.     }
  118.  
  119.     private static function instantiate($record) {
  120.         $object = new self;
  121.         foreach($record as $attribute => $value) {
  122.             if($object->has_attribute($attribute)) {
  123.                 $object->$attribute = $value;
  124.             }
  125.         }
  126.         return $object;
  127.     }
  128.  
  129.     private function has_attribute($attribute) {
  130.         return array_key_exists($attribute, $this->attributes());
  131.     }
  132.  
  133.     protected function attributes() {
  134.         $attributes = array();
  135.         foreach(self::$db_fields as $field) {
  136.             if(property_exists($this, $field)) {
  137.                 $attributes[$field] = $this->$field;
  138.             }
  139.         }
  140.         return $attributes;
  141.     }
  142.  
  143.     protected function sanitized_attributes() {
  144.         global $database;
  145.         $clean_attributes = array();
  146.         foreach($this->attributes() as $key => $value) {
  147.             $clean_attributes[$key] = $database->escape_value($value);
  148.         }
  149.         return $clean_attributes;
  150.     }
  151.  
  152.     public function create() {
  153.         global $database;
  154.         $attributes = $this->sanitized_attributes();
  155.         $sql = "INSERT INTO " . self::$table_name . " (";
  156.         $sql .= join(", ", array_keys($attributes));
  157.         $sql .= ") VALUES ('";
  158.         $sql .= join("', '", array_values($attributes));
  159.         $sql .= "')";
  160.         if($database->query($sql)) {
  161.             $this->id = $database->insert_id();
  162.             return true;
  163.         } else {
  164.             return false;
  165.         }
  166.     }
  167.  
  168.     public function update() {
  169.         global $database;
  170.         $attributes = $this->sanitized_attributes();
  171.         $attribute_pairs = array();
  172.         foreach($attributes as $key => $value) {
  173.             $attribute_pairs[] = "{$key}='{$value}'";
  174.         }
  175.         $sql = "UPDATE " . self::$table_name . " SET ";
  176.         $sql .= join(", ", $attribute_pairs);
  177.         $sql .= " WHERE id=" . $database->escape_value($this->id);
  178.         $database->query($sql);
  179.         return ($database->affected_rows() == 1) ? true : false;
  180.     }
  181.  
  182.     public function delete() {
  183.         global $database;
  184.         $sql = "DELETE FROM " . self::$table_name;
  185.         $sql .= " WHERE id=" . $database->escape_value($this->id);
  186.         $sql .= " LIMIT 1";
  187.         $database->query($sql);
  188.         return ($database->affected_rows() == 1) ? true : false;
  189.     }
  190.  
  191. }
  192.  
  193. ?>
[PHP] pobierz, plaintext



Z tego co tutaj czytam zmienna $database daje nam chyba mozliwosc laczenia z baza lub sprawdzenia uzytkownika bo jest wszedzie username, useremail = database
viking
1.07.2017, 13:24:13
Wrzucasz komendy które wyleciały z PHP7. Wydaje mi się też że nie wyświetlasz błędów w swoim kodzie. Ustaw display_errors bezpośrednio w php.ini. Podajesz tutaj tylko jedną klasę User która używa $database - przez słowo global, nie powinno się tak robić - dependency injection jak już, została ona gdzieś wcześniej zainicjowana. Jak dasz w swojej metodzie global $database; to będziesz mógł z tej klasy też korzystać. Bardzo dobrze że chcesz się uczyć z kodu ale ten akurat nie jest jakoś szczególnie wart uwagi. Powinieneś zacząć od nauki klas jako takich, żebyś wiedział jak działają. Osobiście widzę potencjał w tobie ale powinieneś zacząć od prostych rzeczy na podstawie oficjalnej dokumentacji.
Rafiks1992
1.07.2017, 21:50:25
Dzieki za slowa otuchy, w kazdym razie jak dobrze rozumiem powinienem porzucic ten projekt i zaczac od zera pisac kazda linijke samemu, zero kopiowania i przerabiania prostych napisanych juz silnikow? Oczywiscie w nowym php7 bo jak popatrzalem na te PDO i mysqli to w sumie moze i wiele sie to nie rozni, ale faktycznie troche linijek juz nie jest uzywanych aczkolwiek uwazam ze skoro dziala polaczenie z baza danych to dlaczego by z tego nie korzystac? Nie jest to latwe, masz jakis aktualny kurs do php7? Moze moglbys udostepnic abym mogl faktycznie zaczac rozumiec o co chodzi i co z czym sie je..


EDIT, znalazlem jakies filmiki instruktazowe od goscia ktory chyba poprostu pokazuje i uczy o PDO moglbys zobaczyc czy to w miare aktualne jest biggrin.gif?

https://www.youtube.com/watch?v=tEkrox5gutM...0zw&index=7

Dodawanie rekordow wyglada podobnie, ale jednak dziwny jest ten dwukropek zamiast $ w kodzie smile.gif
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.