[PHP] Nie chce mi wyswietlic danych z bazy Opcje

[Rafiks1992]

Witam! To moj pierwszy post, zaczynam uczyc sie php ogladalem troche filmikow i zaczalem tworzyc strone, wlasciwie to moze byc nawet cos podobnego do gry via www w celach nauki lecz mam pewien problem dlatego chcialem prosic tutaj o pomoc

Dokladniej chodzi mi o to ze nie chce wyswietlic mi danych z bazy, dodalem do tabeli users kolumne money i ustawilem wartosc domyslna na 3.
Teraz w pliku ktory wyswietla profil uzytkownika chce dodac opcje ktora bedzie wyswietlala aktualny stan konta uzytkownika czyli 3.

Tak wiec w pliku youraccount.php dodaje taki kod:

[PHP] pobierz, plaintext 
<?php
 
$money = 'SELECT money FROM users';
echo 'Twoja ilosc pieniedzy to:'.$money;
 
?>
[PHP] pobierz, plaintext 

Jednak gdy zaktualizuje plik i odswieze strone to pokazuje mi poprostu moje zapytanie czyli:

Twoja ilosc pieniedzy to: SELECT money FROM users

Moglby ktos mnie nakierowac jak to poprawic? Szukalem w internecie kilka sposobow i probowalem tez w innych formatach to zrobic ale nic mi nie pomoglo. Mam nadzieje ze nie zostane na dzien dobry zlinczowany za to ale kazdy kiedys zaczynal (IMG:style_emoticons/default/wink.gif) !

Ten post edytował Kshyhoo 28.06.2017, 08:59:16

Powód edycji: [Kshyhoo]: Używaj bb-code do kodu...

[viking]

Wyświetlasz teraz string. Musisz jeszcze to zapytanie jakoś wykonać i przetworzyć. Najlepiej za pomocą PDO.

[Pyton_000]

Oglądałeś filmiki... Albo nie doszedłeś jeszcze do etapu pracy z BD albo pominąłeś kilka odcinków...

[Rafiks1992]

Wporzadku bede uzywal bbcode jesli bedzie jakis kod odemnie (IMG:style_emoticons/default/biggrin.gif) ! Co do mojego pytania w jaki sposob moge przetworzyc to zapytanie do sql? Bo wyswietlenie chyba nie jest zle poprzez echo? Co do kolegi wyzej owszem wiele rzeczy pominalem ale tak to jest jak chce sie szybko szybko cos zrobic ^^ bez podstaw i ksiazki jakiejs chyba nie ogarne tego ale chcialem odrazu poprzez praktyke sie uczyc (IMG:style_emoticons/default/smile.gif)

[viking]

Podałem ci link gdzie masz wszystko opisane. Szybki przykład

[Rafiks1992]

Wybacz kolego Viking nie zauwazylem tego PDO myslalem ze to tylko podkreslenie jakies a tu link fajny (IMG:style_emoticons/default/biggrin.gif)

No wiec tak przerobilem to wszystko i wyszlo mi mniej wiecej cos takiego:

[PHP] pobierz, plaintext 
<?php 
 
$sql = 'SELECT money FROM users WHERE money = ?';
 
$dbh = new PDO('mysql:host=nazwa hosta;dbname=nazwa bazy danych', 'nazwa uzytkownika', 'haslo');
array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES 'utf8'");
 
$sth = $dbh->prepare($sql);
 
$sth->bindParam(1, $_POST['money']);
 
$sth->execute();
 
$result = $sth->fetchAll();
 
foreach ($result as $row) {
	echo 'Twoja kasa to:' .$row['money'];
 
}
?>
[PHP] pobierz, plaintext 

Oczywiscie pod nazwy podlozylem polaczenie z baza uzytkownik i tak dalej zmienilem takze zmienne na money wkleilem to zaktualizowalem i w sumie na stronie jak odswiezylem to nie wyswietla sie absolutnie nic zadnego bledu nie ma, ale nie ma tez wyswietlonego pustka na stronie nawet nie wyswietla sie tekst Twoja kasa to:.. Dodam ze dodaje to pod danymi uzytkownika prawie na samym koncu kodu.

[Pyton_000]

Bo do zapytania odbierasz zmienną z `$_POST['money']`;

[Rafiks1992]

A pownienem uzyc $_GET? Bo nie rozumiem teraz jak wyjsc na prostą i wyeliminowac ten problem (IMG:style_emoticons/default/tongue.gif)

[nospor]

Miedzy uszami masz taki malutki guzik. Wlacz go, on odpowiada za myslenie (IMG:style_emoticons/default/tongue.gif)

Szukasz teraz usera na podstawie pieniedzy.... Nie sadzisz ze powinienes go szukac na podstawie ID tudziez LOGIN? I to ID czy LOGIN masz pobrac z sesji, get czy skadkolwiek gdzie to trzymasz

[viking]

Wszystko zależy skąd pochodzi ta zmienna. form method=post będzie w $_POST, z pasku adresu w $_GET.

[Rafiks1992]

No tak jedna dobra wiadomosc ktora dobrze nakieruje i juz ma to jakis sens (IMG:style_emoticons/default/biggrin.gif) w pliku Twoj profil mam taki kod:

[PHP] pobierz, plaintext 
<?php 
 
$title = "Twoje konto";
include_once 'header.php'; 
 
if (!$session->is_logged_in()) {
	$session->message("Musisz sie zalogowac!", "success");
	redirect_to("login.php");
}
 
 
?>
 
<h1>Twoje konto</h1>
 
<?php
if (isset($_POST['submit'])) {
	$newpassword = sanitize($_POST['newpassword']);
	$newemail = sanitize($_POST['newemail']);
	$oldpassword = sanitize($_POST['oldpassword']);
 
	$userdata = User::getUser($session->user_name);
 
	$validate = new Validation();
	$validate->addSource($_POST);
	$validate->addRule('newpassword', 'string', false, '4', '20', true);
	$validate->addRule('newemail', 'email', false, '3', '30', true);
	$validate->addRule('oldpassword', 'string', true, '4', '20', true);
	$validate->run();
 
	if (!empty($newemail)) {
		$email_result = User::exists("email",$newemail);
 
		if($email_result) {
			$session->message("Inny uzytkownik zarejestrowany jest juz na tego e-maila!", "warning");
			redirect_to("account.php");
		}
	}
 
	if (!empty($oldpassword)) {
		if (secure_string($oldpassword) !== $userdata->password) {
			$session->message("Niepoprawne stare haslo!", "warning");
			redirect_to("account.php");
		}
	}
 
 
	if (sizeof($validate->errors) > 0) {
		echo("<div id='normal'><div class='warning'>");
		foreach ($validate->errors as $error) { echo "- ".$error."<br/>"; }
		echo("</div></div><br />");
 
	} else {
 
		$user = new User();
		$user->id = $userdata->id;
		$user->username = $userdata->username;
		if (!empty($newpassword)) {
		$user->password = secure_string($newpassword);	
		} else {
		$user->password = $userdata->password;	
		}
		if (!empty($newemail)) {
		$user->email = $newemail;	
		} else {
		$user->email = $userdata->email;	
		}
		$user->status = $userdata->status;
		$user->actcode = $userdata->actcode;
		$user->created_at = $userdata->created_at;
		$result = $user->update();
 
		if ($result) {
			$session->message("Zmiany zapisane!", "success");
			redirect_to("account.php");
		} else {
			$session->message("Blad, nie zapisano zmian!", "warning");
			redirect_to("account.php");
		}
 
	}
}
 
if (isset($_POST['delete'])) {
 
	$userdata = User::getUser($session->user_name);
	$oldpassword = sanitize($_POST['oldpassword']);
 
	$validate = new Validation();
	$validate->addSource($_POST);
	$validate->addRule('oldpassword', 'string', true, '4', '20', true);
	$validate->run();
 
	if (!empty($oldpassword)) {
		if (secure_string($oldpassword) !== $userdata->password) {
			$session->message("Niepoprawne stare haslo!", "warning");
			redirect_to("account.php");
		}
	}
 
	if (sizeof($validate->errors) > 0) {
		echo("<div id='normal'><div class='warning'>");
		foreach ($validate->errors as $error) { echo "- ".$error."<br/>"; }
		echo("</div></div><br />");
 
	} else {
 
		$user = new User();
		$user->id = $userdata->id;
		$result = $user->delete();
 
		if($result) {
			$session->logout();
			$session->message("Twoje konto zostalo usuniete!!", "success");
			redirect_to("index.php");
		}
	}	
}
 
?>
 
<form method="post" name="submit" id="submit" action="account.php">
<table>
    <tr>
      <td>Nick: </td>
      <td><input name="username" id="username" readonly="readonly" type="text" maxlength="15" size="15" value="<?php echo $session->user_name; ?>"/></td>
    </tr>
	<tr>
      <td>Nowe hasło: </td>
      <td><input name="newpassword" id="newpassword" type="password" maxlength="30" size="15" value="<?php
 
if(isset($_POST['newpassword'])) {
  echo $_POST['newpassword'];
}
 
?>"/></td>
    </tr>
 
	<tr>
      <td>Nowy E-mail : </td>
      <td><input name="newemail" id="newemail" type="text" maxlength="30" size="15" value="<?php
 
if(isset($_POST['newemail'])) {
  echo $_POST['newemail'];
}
 
?>"/></td>
    </tr>
 
	 <tr>
      <td>Stare haslo: </td>
      <td><input name="oldpassword" id="oldpassword" type="password" maxlength="30" size="15" value="<?php
 
if(isset($_POST['oldpassword'])) {
  echo $_POST['oldpassword'];
}
 
?>"/></td>
    </tr>
 
  </table>
<input class="button" type="submit" name="delete" id="delete" value="Usun konto!"/>
<input class="button" type="submit" name="submit" id="submit" value="Zapisz!"/>
</form>
 
 
<?php
 
include_once 'footer.php'; 
 
?>
[PHP] pobierz, plaintext 

Pobralem jakis silnik strony z chomikuj sam poczatek bez stylow niczego tylko rejestracja logowanie i wyswietlanie profilu i z tego co tutaj wychodzi to logowanie opiera sie na sesji skoro jest $session dobrze mysle (IMG:style_emoticons/default/wink.gif) ? I teraz zeby zrobic to na podstawie loginu czy id tak jak napisal kolega moderator musialbym to napisac tak:?
SELECT id FROM users WHERE money?(IMG:style_emoticons/default/questionmark.gif) Probowalem ale nic nie pomoglo (IMG:style_emoticons/default/biggrin.gif)

[nospor]

Wcisnij ten guzik do konca.... (IMG:style_emoticons/default/tongue.gif)


Chcesz pobrac MONEY dla user o danym ID. Szukac masz wiec po ID a nie po MONEY....

NIe
SELECT id FROM users WHERE money?

a:
SELECT money FROM users WHERE id=tu_szukane_id

ps: inna kwestia to to, ze juz te dane i tak pewnie masz, o tu:

$userdata = User::getUser($session->user_name);

pobierasz cale info o userze wiec i pewnie MONEY tam masz.

[Rafiks1992]

Zanim guzik do konca wcisniety bedzie to troszke minie (IMG:style_emoticons/default/biggrin.gif)

W kazdym razie to prawda mam w jednej tabeli zrobione id login haslo mail i tak dalej i na koncu dodalem money, tylko jak to teraz wyswietlic..

$userdata = User::getUser($session->user_name);

Jesli zmienilbym tutaj w nawiasie username na money a pozniej dal echo .$userdata to powinno zadzialac dobry mam tok rozumowania czy mam sie wrocic do żłobka (IMG:style_emoticons/default/biggrin.gif) ?

Albo teraz wpadlem na to zeby wykorzystac faktycznie to zapytanie sql tak jak napisales

$money = 'SELECT money FROM users WHERE id=tu_szukane_id - tylko teraz tak w tym id= mam dac nazwe kolumny z tabeli o nazwie id? Bo probowalem dac takie cos:

$money = 'SELECT money FROM users WHERE id='$id';
echo $money; i tez dupa zbita (IMG:style_emoticons/default/tongue.gif)

Ten post edytował Rafiks1992 29.06.2017, 11:04:17

[nospor]

$userdata = User::getUser($session->user_name);
echo $userdata['money'];
?

Jak nie zadziala to pokaz kod funkcji
User::getUser

[Rafiks1992]

Sprobowalem tak jak podales wyskoczyl mi blad:
Fatal error: Cannot use object of type User as array in /home/.../account.php on line 172

Linia 172:
echo $userdata['money'];


A w pliku o ktorym mowiles gdzie podaje sie klase uzytkownika znalazlem dane ktore wyswietlane sa w tabeli dokladniej takie cos:

plik userclass.php

[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);
        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 

Dodalem tam na koncu money oraz public $money ale i tak nic nie pomoglo gdzies jeszcze tego mi brakuje (IMG:style_emoticons/default/tongue.gif)

[nospor]

Nie:
echo $userdata['money'];

a:
echo $userdata->money;

[Rafiks1992]

Oczywiscie zmienilem to i ogolnie wyglada to tak:

$userdata = User::getUser($session->user_name);
echo 'Twoja kasa to:' .$userdata->money;


Podmienilem plik na serwerze i teraz wyswietla sie Twoja kasa to: i pustka nic nie wyswietla znow, albo jest to jakis prosciutki malutki blad albo cos tu jest grubo nie halo (IMG:style_emoticons/default/biggrin.gif)

[nospor]

Powinno dzialac teraz. Jestes pewien ze ten user ma jakakolwiek kase w bazie wpisana?

[Rafiks1992]

Jak najbadziej ustawilem w phpmyadmin w money zeby poczatkowa wartosc wynosila 3 i tak rzeczywiscie jest. Moze w zlym miejscu wklepuje ten kod? na samym koncu praktycznie dodaje bo chce zeby byl pod spodem

[viking]

var_dump($userdata);