logowanie PHP

Question

<?php
if (IsSet($_POST['username']) && IsSet($_POST['pass'])) {
unset($_SESSION);
session_start();
 
   $open = fopen("uzytkownicy.txt", "r"); // w pliku uzytkownicy.txt mamy liste userow z haslami w formacie JSON
   $open_txt = fread($open, filesize("uzytkownicy.txt")); // czytamy zawartosc pliku
   $open_txt = json_decode($open_txt, true); // "odkodowujemy"
   fclose($open); // zamykamy zbedny plik
 
   for ($i=1; $i<count($open_txt['user'])+1; $i++) {
        if ($open_txt['user'][$i]['name'] == $_POST['username'] && $open_txt['user'][$i]['pass'] == $_POST['pass']) {
            $_SESSION['user'] = $_POST['username'];
            $_SESSION['pass'] = $_POST['pass'];
            break;
        }
   }
}
?>

Cześć. Znalazłem na jakimś forum pelny kod PHP do zrobienia logowania na mojej stronce. Użytkownik który to wysłał napisał że w pliku uzytkownicy.txt nalezy zapisac dane logowania w formacie json. Chciałby ktoś powiedzieć jak mam to zrobić? Logowanie będzie tylko dla jednej osoby 

 

Comments

Jeśli tak, to odradzam takiego postępowania. Nie trudno o źle wykonany kod.

---

@kvbbs, Z doświadczenia wiadomo że tak więc Jeżeli to jest prawda to Lepiej się doucz PHP a takie rzeczy będziesz umiał robić sam. A to chyba lepsze nie?

---

Potrzebuję logowania tylko dla siebie dlatego nie wiem czy jest sens używać mysql

---

Jest i to duży. Bo to nawet lepsze niż JSON i baza w txt

---

@kvbbs, Trochę podpowiem: https://pl.wikipedia.org/wiki/JSON

Musisz w pliku zapisać dane logowania w formacie JSON ;-)

A struktura danych zaszyta jest w if-e w pętli ;-p

Answer 1

Dawno nie kodowałem w PHP, ale jak dla mnie jak chcesz mieć tylko jednego usera to jak dla mnie dawanie pliku jest bez sensu.
Ja bym poszedł coś w stylu:

<?php
$username = "twojlogin";
$password = "zakodowanehasło"; // Kiedyś się używało md5(), a teraz jest mało bezpieczne i wypadało by poszukać i zastąpić czymś obecnie polecanym
if (IsSet($_POST['username']) && IsSet($_POST['pass'])) {
  unset($_SESSION);
  session_start();
  if($_POST['username']==$username && md5($_POST['pass'])==$password){
    $_SESSION['user'] = $_POST['username'];
    // zapisanie zmiennej hasło do sesji raczej jest średnim pomysłem jak w przykładzie
  }
}
?>

Czy nie zrobiłem jakiegoś byka nie dam głowy, bo nie testwałem i pisałem z palca, ale myślę że sam sposób podejścia jest zrozumiały.