Forum PHP.pl > [SQL][MySQL][PHP][MYSQL & PHP] Zmiana avatara pomocy!

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [SQL][MySQL][PHP][MYSQL & PHP] Zmiana avatara pomocy!

MefjuU

25.04.2018, 18:09:01

Witam otóż mam problem z bazą danych, stosowałem poniższy kod aby zmienić avatara poprzez przyciski INPUT RADIO ale gdy daję zmień to nic się nie dzieje. Błąd jest w kodzie który zmienia wartość w bazie (UPDATE).

[SQL] pobierz, plaintext 
<?php
 
require './site/header.php'; // Dołącz początkowy kod HTML
require './config/config.php'; // Dołącz plik konfiguracyjny i połączenie z bazą
require_once './config/user.class.php';
 
 
IF (!user::isLogged()) {
    echo '<p class="error">Przykro nam, ale ta strona jest dostępna tylko dla zalogowanych użytkowników.</p>';
}
 
else {
	@$id = $_GET['id'];
	$profile = user::getDataById ($id);
	/**
	 * Sprawdź czy formularz został wysłany
	 */
	IF (@$_POST['send'] == 1) {
		// Zabezpiecz dane z formularza przed kodem HTML i ewentualnymi atakami SQL Injection
		@$avatar = mysql_real_escape_string(htmlspecialchars($_POST['avatar']));
 
		$errors = ''; // Zmienna przechowująca listę błędów które wystąpiły
 
 
		// Sprawdź, czy nie wystąpiły błędy
		IF (!$avatar ) $errors .= '- Musisz wybrać avatara aby go zmienić.<br />';
 
		/**
		 * Jeśli wystąpiły jakieś błędy, to je pokaż
		 */
		IF ($errors != '') {
			echo '<p class="error">Zmiana avatara nie powiodła się, popraw następujące błędy:<br />'.$errors.'</p><br/>';
		}
 
		/**
		 * Jeśli nie ma żadnych błędów - kontynuuj rejestrację
		 */
		else {
 
			// Szukanie czy użytkownik istnieje
			$userExist = mysql_fetch_array(mysql_query("SELECT COUNT(*) FROM users WHERE id = '$id'"));
			// Zapisz dane do bazy
			mysql_query("UPDATE users SET avatar='".$avatar."' WHERE id = '$profile'") OR die ('<p class="error">Wystąpił błąd podczas zmiany avatara.</p>'); 
// TUTAJ WŁAŚNIE NIE WIEM JAK TO ZROBIĆ ^
 
 
			echo '<p class="success">Avatar został zmieniont.</p><br/>';
		}
	}
}
?>
[SQL] pobierz, plaintext

a tak wygląda do tego formularz HTML (zdefiniowałem tu własny styl taki jak <p4> itp..):

[HTML] pobierz, plaintext 
<p4>Zmiana avatara</p4>
	<div class="hr"></div>
	<p>Wybierz jeden z dostępnych avatarów.</p><br/>
	<form method="post" action="">
	<table style="width: 100%; text-align: center; margin-top: 10px; margin-bottom: 20px;">
		<tr style="height: 64px;">
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar" value="default-avatar.png"><br/><img src="./avatars/default-avatar.png"></img></input></td>
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar" value="avatar1.png"><br/><img src="./avatars/avatar1.png"></img></input></td>
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar"><br/><img src="./avatars/default-avatar.png"></img></input></td>
		</tr>
		<tr style="height: 64px;">
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar"><br/><img src="./avatars/default-avatar.png"></img></input></td>
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar"><br/><img src="./avatars/default-avatar.png"></img></input></td>
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar"><br/><img src="./avatars/default-avatar.png"></img></input></td>
		</tr>
		<tr style="height: 64px;">
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar"><br/><img src="./avatars/default-avatar.png"></img></input></td>
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar"><br/><img src="./avatars/default-avatar.png"></img></input></td>
			<td><input style="box-shadow: inset 0 0px 0px 0 transparent;" type="radio" name="avatar"><br/><img src="./avatars/default-avatar.png"></img></input></td>
		</tr>
	</table>
	<input type="hidden" name="send" value="1" />
	<input type="submit" class="sub" style="margin-left: 38.5%; text-align: center; text-shadow: 0 -1px 1px rgba(0,0,0,0.4);" value="Wybierz"></input>
	</form>
[HTML] pobierz, plaintext

wcześniej zamiast:

[SQL] pobierz, plaintext 
// Zapisz dane do bazy
			mysql_query("UPDATE users SET avatar='".$avatar."' WHERE id = '$profile'") OR die ('<p class="error">Wystąpił błąd podczas zmiany avatara.</p>');
[SQL] pobierz, plaintext

stosowałem poniższy kod ale on zamiast zmieniać danemu użytkownikowi avatar to tworzył tak jakby nowego (oznaczyłem to na screenie bazy danych na czerwono) który miał już dobrze wypełnione pole avatar. Ale nie o to mi chodzi. To ma podmieniać a nie tworzyć

[SQL] pobierz, plaintext 
// Zapisz dane do bazy
			mysql_query("INSERT INTO users (avatar) VALUES('$avatar');") OR die ('<p class="error">Wystąpił błąd podczas zmiany avatara.</p>');
[SQL] pobierz, plaintext

Dodam jeszcze screen bazy danych:
http://ifotos.pl/z/qnanrep << LINK 1
https://ibb.co/gs04Rx << LINK 2

(JAKBY LINK DO OBRAZU NIE DZIAŁAŁ TO TUTAJ MOŻESZ GO POBRAĆ)
http://hostuje.net/file.php?id=b5090eab152...ec86d290abf3166

Kshyhoo

25.04.2018, 18:17:32

A action gdzieś idzie?

[PHP] pobierz, plaintext 
<form method="post" action="">
[PHP] pobierz, plaintext

MefjuU

25.04.2018, 19:11:57

Nie action nigdzie nie idzie bo ten skrypt jest w tym samym pliku. Tak samo mam zrobioną rejestrację i logowanie i tam wszystko działa dobrze. tutaj też action działa bo wcześniej dodawało mi nowego usera z wartością avatar tyko więc to chyba dowód na to że działać działa ten action.

leonpro778

25.04.2018, 20:23:46

Czemu w linijce:

[PHP] pobierz, plaintext 
$userExist = mysql_fetch_array(mysql_query("SELECT COUNT(*) FROM users WHERE id = '$id'"));
[PHP] pobierz, plaintext

Szukasz po ID a w linijce:

[PHP] pobierz, plaintext 
mysql_query("UPDATE users SET avatar='".$avatar."' WHERE id = '$profile'")
[PHP] pobierz, plaintext

Dajesz profile. Co się kryje pod $profile ?

MefjuU

26.04.2018, 00:34:17

Ta 1 linijka co jest po ID to sam dodałem żeby poeksperymentować bo chciałem wyszukać użytkownika (tego co zmienia avatara) po ID a pod profile kryje się chyba użytkownik jak się nie mylę obczaj ten kod pobrałem z niego wartości do innego stworzonego pliku profile.php a to ten standardowy:

[SQL] pobierz, plaintext 
<?php
session_start();
 
require 'header.php'; // Dołącz początkowy kod HTML
require 'config.php'; // Dołącz plik konfiguracyjny i połączenie z bazą
require_once 'user.class.php';
 
/**
 * Tylko dla zalogowanych użytkowników
 */
IF (!user::isLogged()) {
    echo '<p class="error">Przykro nam, ale ta strona jest dostępna tylko dla zalogowanych użytkowników.</p>';
}
 
else {
    $id = $_GET['id'];
 
    /**
     * Sprawdź czy użytkownik o podanym ID istnieje
     */
    $userExist = mysql_fetch_array(mysql_query("SELECT COUNT(*) FROM users WHERE id = '$id'"));
 
    // Użytkownik nie istnieje
    IF ($userExist[0] == 0) {
        die ('<p>Przykro nam, ale użytkownik o podanym identyfikatorze nie istnieje.</p>');
    }
 
    /**
     * Użytkownik istnieje, tak więc pokaż jego profil
     */
 
    // Zapisz dane użytkownika o podanym ID, do zmiennej $profile
    $profile = user::getDataById ($id);
 
    echo '<h1>Profil użytkownika '.$profile['login'].'</h1>';
 
    echo '<b>Nick:</b> '.$profile['login'].'<br />';
    echo '<b>Email:</b> '.$profile['email'].'<br />';
 
}
 
require 'footer.php'; // Dołącz końcowy kod HTML
 
?>
[SQL] pobierz, plaintext

tutaj jest jeszcze 1 ważny plik może go rozszyfrujesz i dowiesz się jak tu tego użytkownika znaleść bo ja z dobre 3 godziny się tak bawiłem i zrobiłem tylko taki postęp że niby działa ale nic nie zmienia avatara w bazie danych:

[SQL] pobierz, plaintext 
<?php
/**
 * Klasa wykonująca wszystkie podstawowe operacje
 */
 
class user {
 
    public static $user = array();
 
    /**
     * Zwraca tablicę ze wszystkimi danymi o użytkowniku.
     * Indeksy tablicy odpowiadają nazwom pól w bazie danych (login, pass etc...)
     * @param string $login
     * @param string $pass
     * @return array
     */
    public FUNCTION getData ($login, $pass) {
        IF ($login == '') $login = $_SESSION['login'];
        IF ($pass == '') $pass = $_SESSION['pass'];
 
        self::$user = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE login='$login' AND pass='$pass' LIMIT 1;"));
        RETURN self::$user;
    }
 
 
    /**
     * Zwraca tablicę ze wszystkimi danymi o użytkowniku, tak jak powyższa metoda klasy,
     * ale rozpoznaje użytkownika nie po podaniu loginu i hasła tylko po podaniu ID.
     * Używana np. do wyświetlania strony profilu.
     * @param int $id
     * @return array
     */
    public FUNCTION getDataById ($id) {
        $user = mysql_fetch_array(mysql_query("SELECT * FROM users WHERE id='$id' LIMIT 1;"));
        RETURN $user;
    }
 
    /**
     * Jeśli użytkownik jest zalogowany - zwraca true, w przeciwnym wypadku - false
     * @return bool
     */
    public FUNCTION isLogged () {
     IF (empty($_SESSION['login']) || empty($_SESSION['pass'])) {
      RETURN false;
     }
 
     else {
      RETURN true;
     }
    }
 
    /**
     * "Soli" hasło przed jego zahashowaniem funkcją md5()
     * @param string $pass
     * @return string
     */
    public FUNCTION passSalter ($pass) {
        $pass = '$@@#$#@$'.$pass.'q2#$3$%##@';
        RETURN md5($pass);
    }
 
}
[SQL] pobierz, plaintext

UDAŁO MI SIĘ Problem polegał w tej linijce powinna ona wyglądać tak o:

[SQL] pobierz, plaintext 
// Zapisz dane do bazy
			mysql_query("UPDATE users SET avatar='$avatar' WHERE id='$user[id]';") OR die ('<p class="error">Wystąpił błąd podczas zmiany avatara.</p>');
[SQL] pobierz, plaintext

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.