[PHP] Problem z kupowaniem przedmiotow w sklepie Opcje

[Rafiks1992]

Siemka ponownie (IMG:style_emoticons/default/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 
<?php
 
$title = "Sklep";
include_once 'header.php';
 
if (!$session->is_logged_in()) {
	$session->message("Musisz sie zalogowac!", "success");
	redirect_to("login.php");
}
?>
 
<h1>Sklep</h1>
 
<?php 
echo 'Witamy w sklepie, mozesz tutaj kupic losy.';
?> <br><br><br>
 
<?php
$userdata = User::getUser($session->user_name);
echo 'Masz aktualnie: ' .$userdata->money. 'zl.';
?> <br>
 
<?php 
echo 'Kup:';
?> <br>
 
<?php 
 
if(isset($_POST['kup_los'])) {
	if(!empty($_POST['mikrolos'])) {
		$mikrolos = ($_POST['mikrolos']*$cost);
		if($money >= $mikrolos) {
			mysql_query("UPDATE users SET mikrolos='".$_POST['mikrolos']."' WHERE id=".$pokaz['id']);
			mysql_query("UPDATE users SET money=money-".$cost." WHERE id=".$pokaz['id']);
		} else {
			echo "Nie stać cie";
		}
	}
}
 
 
 
?>
 
<form action='' method='post' class='form'>
Mikrolos: <input type='text' name='mikrolos' value=''>
<button name='kup_los' type='submit'>Kup</button>
</form> 
 
<?php
 
include_once 'footer.php'; 
 
?>
[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]

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]

Pewnie połączenia z bazą nawet nie masz. Wczoraj w kodzie User było przekazywane jakieś global $database.

[Rafiks1992]

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

Ten post edytował Rafiks1992 30.06.2017, 10:46:17

[trueblue]

Na tym polega relacyjność baz danych.
Wyobraź sobie, że zmieniasz nazwę losu lub dodajesz kolejny.

[Rafiks1992]

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]

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]

Probuje uczyc sie piszac i praktykujac moze stad tyle prostych bledow (IMG:style_emoticons/default/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 
<?php
 
require_once 'includes/config.php';
 
$title = "Sklep";
include_once 'header.php';
 
if (!$session->is_logged_in()) {
	$session->message("Musisz sie zalogowac!", "success");
	redirect_to("login.php");
}
 
?>
 
<h1>Sklep</h1>
 
<?php 
echo 'Witamy w sklepie, mozesz tutaj kupic losy do naszych gier.';
?> <br><br><br>
 
<?php
$userdata = User::getUser($session->user_name);
echo 'Masz aktualnie: ' .$userdata->money. 'zl.';
?> <br>
 
<?php 
echo 'Kup:';
?> <br>
 
<?php 
 
function kup_los($id){
	$userdata = User::getUser($session->user_name);
	$tekst='Niewykonana funkcja';
	$los=mysql_query("SELECT * FROM shop WHERE id=$id LIMIT 1");
	if (mysql_num_rows($los)===1){
		$los=mysql_fetch_assoc($los);
		$user=mysql_query("SELECT * FROM users, eq WHERE users.id=eq.uid LIMIT 1");
		$user=mysql_fetch_assoc($user);
		var_dump($los,$user);
		if ($user['money']>=$los['cost']){
			$tekst='Przedmiot kupiony!';
			mysql_query("INSERT INTO eq (id,uid,name,cost,count) VALUES ('$user','$los[id]','$los[name]','$los[cost]')");
		}
	}
	return $tekst;
}
 
echo kup_los(1);
 
?>
 
 
 
<?php
 
error_reporting(E_ALL);
ini_set('display_errors', 1);
?>
 
<?php
 
include_once 'footer.php'; 
 
?>
[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 (IMG:style_emoticons/default/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

Ten post edytował Rafiks1992 30.06.2017, 22:07:04

[trueblue]

Przejrzyj dokładnie zapytanie za pomocą, którego dodajesz zakupiony los.
Przy okazji. Nazwa losu nie powinna być przechowywana w tej tabeli.

[Rafiks1992]

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]

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]

Jak to nie ma w php (IMG:style_emoticons/default/tongue.gif) ? masz na mysli ze wrzucam komendy np z php 4.5 czy jakiegos tam a teraz trzeba 7.1 (IMG:style_emoticons/default/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 
<?php
 
class User {
 
    protected static $table_name = "users";
    protected static $db_fields = array('id', 'username', 'password', 'email', 'status', 'actcode' ,'created_at' ,'money');
 
    public $id;
    public $username;
    public $password;
    public $email;
    public $status = "pending";
    public $actcode;
    public $created_at;
    public $money;
 
    public static function authenticate($username = "", $password = "") {
        global $database;
 
        $username = $database->escape_value($username);
        $password = $database->escape_value($password);
        $password = secure_string($password);
 
        $sql = "SELECT * FROM " . self::$table_name . " users WHERE username = '{$username}' AND password = '{$password}' LIMIT 1";
        $result_array = self::find_by_sql($sql);
        return !empty($result_array) ? array_shift($result_array) : false;
    }
 
	public static function activate($username, $actcode) {
		global $database;
		global $session;
 
		$username = $database->escape_value($username);
        $actcode = $database->escape_value($actcode);
 
		$sql = "SELECT status FROM users WHERE username = '{$username}' AND actcode = '{$actcode}' AND status = 'pending' LIMIT 1";
		$database->query($sql);
		if($database->affected_rows() == 1) {
			$sql = "UPDATE users SET status = 'confirmed' WHERE username = '{$username}' AND actcode = '{$actcode}'";
			$result = $database->query($sql);
 
			if($result) {
				return true;
			} else {
				return false;
			}} else {
			return false;
		}
	}
 
	public static function getUser($username) {
		global $database;
 
		$username = $database->escape_value($username);
 
		$sql = "SELECT * FROM users WHERE username = '{$username}' LIMIT 1";
		$result_array = self::find_by_sql($sql);
	//	var_dump($result_array);
        return !empty($result_array) ? array_shift($result_array) : false;
 
	}
 
	public static function user_email($username, $email) {
		global $database;
 
		$username = $database->escape_value($username);
		$email = $database->escape_value($email);
 
		$sql = "SELECT username FROM users WHERE username = '{$username}' AND email = '{$email}' LIMIT 1";
		$database->query($sql);
		if($database->affected_rows() == 1) {
			return true;
		} else 
			return false;
	}
 
	public static function exists($where,$username) {
		global $database;
 
		$username = $database->escape_value($username);
		$where = $database->escape_value($where);
 
		$sql = "SELECT '{$where}' FROM users WHERE {$where} = '{$username}' LIMIT 1";
		$database->query($sql);
		if($database->affected_rows() == 1) {
			return true;
		} else 
			return false;
	}
 
 
    public static function find_all() {
        return self::find_by_sql("SELECT * FROM " . self::$table_name);
    }
 
    public static function find_by_id($id = 0) {
        $result_array = self::find_by_sql("SELECT * FROM " . self::$table_name . " WHERE id={$id} LIMIT 1");
        return !empty($result_array) ? array_shift($result_array) : false;
    }
 
    public static function find_by_sql($sql = "") {
        global $database;
        $result_set = $database->query($sql);
        $object_array = array();
        while($row = $database->fetch_array($result_set)) {
            $object_array[] = self::instantiate($row);
        }
        return $object_array;
    }
 
    public static function count_all() {
        global $database;
        $sql = "SELECT COUNT(*) FROM " . self::$table_name;
        $result_set = $database->query($sql);
        $row = $database->fetch_array($result_set);
        return array_shift($row);
    }
 
    private static function instantiate($record) {
        $object = new self;
        foreach($record as $attribute => $value) {
            if($object->has_attribute($attribute)) {
                $object->$attribute = $value;
            }
        }
        return $object;
    }
 
    private function has_attribute($attribute) {
        return array_key_exists($attribute, $this->attributes());
    }
 
    protected function attributes() {
        $attributes = array();
        foreach(self::$db_fields as $field) {
            if(property_exists($this, $field)) {
                $attributes[$field] = $this->$field;
            }
        }
        return $attributes;
    }
 
    protected function sanitized_attributes() {
        global $database;
        $clean_attributes = array();
        foreach($this->attributes() as $key => $value) {
            $clean_attributes[$key] = $database->escape_value($value);
        }
        return $clean_attributes;
    }
 
    public function create() {
        global $database;
        $attributes = $this->sanitized_attributes();
        $sql = "INSERT INTO " . self::$table_name . " (";
        $sql .= join(", ", array_keys($attributes));
        $sql .= ") VALUES ('";
        $sql .= join("', '", array_values($attributes));
        $sql .= "')";
        if($database->query($sql)) {
            $this->id = $database->insert_id();
            return true;
        } else {
            return false;
        }
    }
 
    public function update() {
        global $database;
        $attributes = $this->sanitized_attributes();
        $attribute_pairs = array();
        foreach($attributes as $key => $value) {
            $attribute_pairs[] = "{$key}='{$value}'";
        }
        $sql = "UPDATE " . self::$table_name . " SET ";
        $sql .= join(", ", $attribute_pairs);
        $sql .= " WHERE id=" . $database->escape_value($this->id);
        $database->query($sql);
        return ($database->affected_rows() == 1) ? true : false;
    }
 
    public function delete() {
        global $database;
        $sql = "DELETE FROM " . self::$table_name;
        $sql .= " WHERE id=" . $database->escape_value($this->id);
        $sql .= " LIMIT 1";
        $database->query($sql);
        return ($database->affected_rows() == 1) ? true : false;
    }
 
}
 
?>
[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]

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]

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 (IMG:style_emoticons/default/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 (IMG:style_emoticons/default/smile.gif)

Ten post edytował Rafiks1992 1.07.2017, 22:41:00