Edycja profilu na plikach Opcje

[danielwwa91]

Mam następny problem, otóż zrobiłem sobie panel użytkownika, w ktorym chciałbym, aby użytkownicy mogli edytować swoje dane. Niestety nie mam pojęcia jak do tego się zabrać.. siedzę już nad tym 2 dzień i nadal nic konkretnego. Proszę Was o pomoc.

plik rejestracja.php, w którym się tworzy profil usera:

[PHP] pobierz, plaintext 
<div class="post"><Br>
<div id="newstitle">Rejestracja w serwisie xxx - Krok 2</div>
<div id="newsbg">
 
<?php
 
session_start();
 
if (($nick=='') && ($pass2=='') && ($pass=='')){
 
 echo ('Pola oznaczone gwiazdką należy wypełnić!'); die;}
 
if (strpos($nick,' ')) {echo('Nick nie może zawierać spacji!'); die;}
if (strlen($pass)<3){ echo('Podane hasło jest za krótkie!'); die;}
if (strlen($pass)>8){ echo('Podane hasło jest za długie!'); die;}
 
if (strlen($nick)<3){ echo('Podany nick jest za krótki!'); die;}
if (strlen($nick)>15){ echo('Podany nick jest za długi! (max 15 znaków)'); die;}
 
if ($nick==''){ echo ('Pola oznaczone gwiazdką należy wypełnić!'); die;}
 
$usernew = "users/".$nick.".php";
if (file_exists($usernew)) {
print ("Login <b>$nick</b> jest już zarejestrowany w naszej bazie. Prosimy wybrać inny."); } else {
 
if ($pass == $pass2){
 
$dataut=date('d.m.Y, H:i:s');
$adduser = fopen ("users/".$nick.".php", "w");
$zawuser = "$grupa|$nick|$pass|$email|$imie|$dataut";
 
 
fputs ($adduser, $zawuser);
fclose ($adduser);
 
$adduserpa = "users/profile/".$nick.".php";
 
$_SESSION['usernick']=$nick;
$_SESSION['username']=$imie;
$_SESSION['usermail']=$email;
 
if (file_exists($adduserpa)) {
print ("Login <b>$nick</b> jest już zarejestrowany w naszej bazie. Prosimy wybrać inny."); } else {
 
$adduserp = fopen ("users/profile/".$nick.".php", "w");
$zawuserp = "
 
<div class=post><br>
 
<div id=newstitle>Przeglądasz profil użytkownika: $nick</div>
 
<div id=newsbg> 
 
<div id=userprofile>Informacje podstawowe:</div>
<div id=userprofilebg>
 
Nick użytkownika: <b>$_SESSION[usernick]</b><br>
Data utworzenia: <b>$dataut</b><br>
Poziom: <b>$grupa/4</b><br><br>
 
 
</div>
 
<div id=userprofile>O użytkowniku:</div>
<div id=userprofilebg>
 
Imię: <b>$_SESSION[username]</b><br>
Data urodzenia: <b></b><br>
Skąd: <b></b><br>
Email: <b>$_SESSION[usermail]</b><br>
 
</div>
 
</div>
 
";
 
fputs ($adduserp, $zawuserp);
fclose ($adduserp); }
 
echo ('<font color=green><b>Witaj '.$nick.'! Dziękujemy za rejestrację w serwisie fwsite.pl!</b></font><br><br>
 
Twój nick: <b>'.$nick.'</b> | Hasło: <b>'.$pass.'</b> - prosimy zapisać te dane.<br>
 
Należysz do grupy użytkowników!
 
<br><br>
 
Możesz się teraz <a href=?site=logowanie>zalogować</a> na swoje konto!');
 
}
 
else {
	echo ('Podane hasła są różne!'); }
}
 
?>
 
</div>
[PHP] pobierz, plaintext 

plik użytkownika zaraz po rejestracji (nie mylić z profilem):

[PHP] pobierz, plaintext 
1|danielwwa|abc123|danielwwa@o2.pl|Daniel
[PHP] pobierz, plaintext 

profil użytkownika po rejestracji:

[PHP] pobierz, plaintext 
 
<div class=post><br>
 
<div id=newstitle>Przeglądasz profil użytkownika: danielwwa</div>
 
<div id=newsbg> 
 
<div id=userprofile>Informacje podstawowe:</div>
<div id=userprofilebg>
 
Nick użytkownika: <b>danielwwa</b><br>
Poziom: <b>1/4</b><br><br>
 
</div>
 
<div id=userprofile>O użytkowniku:</div>
<div id=userprofilebg>
 
Imię: <b>Daniel</b><br>
 
</div>
 
</div>
 
[PHP] pobierz, plaintext 

I to chyba będzie na tyle. Wiem, że trzeba będzie wczytać to inputa np. imię i zrobić <form> wtedy zadziała, ale nie wiem jak zacząć.

Z góry dziękuję za pomoc.

Ten post edytował danielwwa91 12.01.2010, 02:26:35

[Zyx]

Takiego prostego pliku profilu nie potrafisz wczytać? Jest funkcja explode(). Polecałbym jednak wybrać co innego, nawet serializację tablic PHP, bo teraz już Ci mogę powiedzieć, że nie przewidziałeś sytuacji, co będzie jak ktoś sobie wpisze znak "|" w formularzu. Sprawa jest prosta: masz wiele braków w podstawach programowania i dlatego nie potrafisz się z tym uporać. Podziel sobie kod na jakieś moduły, wydziel wspólne części oraz te, które się zmieniają. Masz trzy podstawowe rzeczy do zaprogramowania:

1. Dodawanie - wczytuje formularz, zapisuje do pliku.
2. Edycja - wczytuje plik, wstawia do formularza, odbiera formularz, zmienia możliwe do zmiany dane, zapisuje do pliku.
3. Usuwanie - kasuje plik.

W ogóle powinieneś to na bazach danych robić. Profile + pliki to lekko beznadziejny pomysł.

Kolejny mankament: co ma wspólnego rejestracja z sesją? I po co przechowujesz w niej tyle danych, skoro wystarczy Ci sam identyfikator i odczytanie całej reszty z pliku profilu?

Błędy typowo programistyczne:

1. if (($nick=='') - tak się nie pisze. Jakie $nick? Z jakiego to antycznego źródła się uczysz? Idź je wyrzuć, bo naucza takiego bałaganu i takich muzealnych rzeczy, że wstyd. $_POST, $_GET należy używać.

2. Dlaczego profil przechowujesz jako HTML? Co stoi na przeszkodzie, by go zapisać w takiej samej postaci, jak podstawowe dane? W ten sposób sam sobie zamknąłeś drogę do jego prostej edycji.

3. Po co tyle bezsensownych zmiennych tymczasowych?

[PHP] pobierz, plaintext 
$adduserpa = "users/profile/".$nick.".php";
[PHP] pobierz, plaintext 

Wykorzystujesz ją tylko w jednym miejscu, wywal ją, albo bądź konsekwentny.

[Kshyhoo]

1. Sprawdzasz nadane prawa,
2. Ładujesz formularz (wyciągasz z bazy dane z rejestracji),
3. Dalej odbywa się tak, jak zwykle...

EDIT: Zyx jak zwykle szybszy ;p

Ten post edytował Kshyhoo 12.01.2010, 08:51:53

[danielwwa91]

Zyx, wielkie dzięki, bardzo mi pomogłeś. Na bazach nie robie, bo nie potrafię, najpierw wole nauczyć się na plikach, a później stopniowo zaczynać z bazą danych (IMG:style_emoticons/default/smile.gif) Przepisze to jeszcze raz wg. Twoich rad i zobaczymy co wyjdzie.

[Kshyhoo]

A w czym problem. Jest sporo kursów, gdzie można łatwo i bezboleśnie przyswoić sobie np. mysql'a

[danielwwa91]

Kshyhoo, dzięki ale nie (IMG:style_emoticons/default/smile.gif)
Troszkę zmieniłem system działania, teraz inaczej zapisują się zmienne do pliku usera.
Mam kolejny problem, bo jak chcę się zalogować i wpisuję poprawny login, a hasło błędne, to mnie i tak poprawnie loguje.. jak wpiszę poprawny login i poprawne hasło, to także mnie loguje.
Coś tu musi być nie tak, tylko co?

logowanie2.php:

[PHP] pobierz, plaintext 
<div class="post"><br>
<div id="newstitle">Logowanie do panelu użytkownika</div>
<div id="newsbg">
 
<?php 
session_start(); 
 
if ($pass=='') { echo ('Proszę wypełnić wszystkie pola!'); die;} 
 
$userfile = "users/".$nick.".php"; 
 
if (file_exists($userfile)){  
 
$plik = fopen ($userfile, "w"); 
$zawartosc_pliku = fgets ($plik, 2048); 
fclose ($plik); 
 
$nickp = $_POST['nick']; 
$passp = $_POST['pass']; 
 
if(isset($nick) && isset($pass)) { 
if ($nick==$nickp && $pass==$passp) { 
$_SESSION['zalogowany']=true; 
 
session_write_close(); 
 
echo ("<font color=green><b>Witaj $nick! Zostałeś poprawnie zalogowany do 
 
serwisu fwsite.pl!</b></font>!<br>Masz teraz całkowity dostęp do serwisu!
 
<br><br>? <a href=index.php>przejdź na stronę główną</a>"); 
} 
else { 
echo ('<font color=red>Wprowadzone hasło jest błędne!</font><br><br><a 
 
href=java script:history.back();>? wróć i popraw</a>'); 
die; 
} 
} 
} 
else { 
echo "<font color=red>Użytkownik <b>$nick</b> nie istnieje!</font>"; 
}; 
?>
 
</div>
[PHP] pobierz, plaintext 

rejestracja2.php:

[PHP] pobierz, plaintext 
<div class="post"><br>
<div id="newstitle">Rejestracja w serwisie xxx - Krok 2</div>
<div id="newsbg">
 
<?php
 
session_start();
 
if (($nick=='') && ($pass2=='') && ($pass=='')){
 
 echo ('Pola oznaczone gwiazdką należy wypełnić!'); die;}
 
$nick = preg_replace("#\|#si", "", $nick);
$nick = preg_replace("#\<#si", "", $nick);
$nick = preg_replace("#\>#si", "", $nick);
 
$pass = preg_replace("#\|#si", "", $pass);
$pass = preg_replace("#\<#si", "", $pass);
$pass = preg_replace("#\>#si", "", $pass);
 
$pass2 = preg_replace("#\|#si", "", $pass2);
$pass2 = preg_replace("#\<#si", "", $pass2);
$pass2 = preg_replace("#\>#si", "", $pass2);
 
$email = preg_replace("#\|#si", "", $email);
$email = preg_replace("#\<#si", "", $email);
$email = preg_replace("#\>#si", "", $email);
 
$imie = preg_replace("#\|#si", "", $imie);
$imie = preg_replace("#\<#si", "", $imie);
$imie = preg_replace("#\>#si", "", $imie);
 
if (strpos($nick,' ')) {echo('Nick nie może zawierać spacji!'); die;}
if (strlen($pass)<3){ echo('Podane hasło jest za krótkie!'); die;}
if (strlen($pass)>8){ echo('Podane hasło jest za długie!'); die;}
 
if (strlen($nick)<3){ echo('Podany nick jest za krótki!'); die;}
if (strlen($nick)>15){ echo('Podany nick jest za długi! (max 15 znaków)'); die;}
 
if ($_GET['nick']==''){ echo ('Pola oznaczone gwiazdką należy wypełnić!'); die;}
 
$usernew = "users/".$nick.".php"; 
if (file_exists($usernew)) { print ("Login <b>$nick</b> jest już zarejestrowany w naszej bazie. Prosimy 
wybrać inny."); } else {
 
if ($pass == $pass2){
 
$dataut=date('d.m.Y, H:i:s');
$adduser = fopen ("users/".$nick.".php", "w");
$phpend = "?>";
$zawuser = "<?php \$nick=\"$nick\"; \$pass=\"$pass\"; \$email=\"$email\"; \$imie=\"$imie\"; \$poziom=\"$poziom\"; $phpend";
 
fputs ($adduser, $zawuser);
fclose ($adduser);
 
$adduserpa = "users/profile/".$nick.".php";
 
$_SESSION['usernick']=$nick;
$_SESSION['username']=$imie;
$_SESSION['usermail']=$email;
$_SESSION['userlvl']=$poziom;
 
if (file_exists($adduserpa)) {
print ("Login <b>$nick</b> jest już zarejestrowany w naszej bazie. Prosimy 
 
wybrać inny."); } else {
 
$adduserp = fopen ("users/profile/".$nick.".php", "w");
$zawuserp = "brak";
 
fputs ($adduserp, $zawuserp);
fclose ($adduserp); }
 
echo ("");
 
echo ('<font color=green><b>Witaj '.$nick.'! Dziękujemy za rejestrację w 
 
serwisie fwsite.pl!</b></font><br><br>
 
Twój nick: <b>'.$nick.'</b> | Hasło: <b>'.$pass.'</b> - prosimy zapisać te 
 
dane.<br>
 
Należysz do grupy użytkowników!
 
<br><br>
 
Możesz się teraz <a href=?site=logowanie>zalogować</a> na swoje konto!');
 
}
 
else {
	echo ('Podane hasła są różne!'); }
}
 
 
?>
 
</div>
[PHP] pobierz, plaintext 

i przykładowy plik usera:

[PHP] pobierz, plaintext 
<?php $nick="daniel"; $pass="haslo"; $email="danielwwa@xx.pl"; $imie="Daniel"; $poziom="1"; ?>
[PHP] pobierz, plaintext 

kurcze już naprawdę nie wiem co może być nie tak.. proszę Was o pomoc.

Ten post edytował danielwwa91 13.01.2010, 00:30:20

[Kshyhoo]

Zobacz, jak się logujesz:

[PHP] pobierz, plaintext 
$nickp = $_POST['nick'];
$passp = $_POST['pass'];
 
if(isset($nick) && isset($pass)) { 
[PHP] pobierz, plaintext 

Jeżeli login istnieje i hasło istnieje. To można wpisać cokolwiek... Prześledź ten kod:

[PHP] pobierz, plaintext 
function autoryzacja($login, $haslo) {
  $plik = file("hasla.txt");
  foreach($plik as $linijka) {
    $dane = explode("|", $linijka);
    if (trim($login)==trim($dane[0]) and trim($haslo)==trim($dane[1]))
    return true;
  }
  return false;
}
 
$login = "..."; //pobrane z formularza na stronie
$haslo = "..."; //pobrane z formularza na stronie
 
if (autoryzacja($login, $haslo)) {
 
  echo "autoryzacja poprawna...";
  // tutaj zawartość strony po autoryzacji
 
} else {
 
  echo "login lub hasło podane błędnie...";
  // tutaj strona, z prośba o ponowne zalogowanie
 
}
[PHP] pobierz, plaintext 

[danielwwa91]

super, o to chodziło! po prostu zapomniałem o funkcjach (IMG:style_emoticons/default/blinksmiley.gif)
jestem Ci bardzo wdzięczny, oczywiście + dla Ciebie (IMG:style_emoticons/default/smile.gif)