Porównanie wartości bazy danych z funkcją Opcje

[marcin690]

Hello! Jestem początkującym programistą, tworzę drobną aplikację na wzór CRM. Chciałbym w panelu wyświetlać rolę użytkownika, która zapisana jest w Mysql jako liczba
W bazie danych jest to rola 1 lub 2 - zatrudniony oraz administrator, zatem stworzyłem coś takiego:

[PHP] pobierz, plaintext 
$userrole = $row["userrole"]; 
 
 deklaracja funkcji
{
   if ($userrole === "1") {
   	echo "Zatrudniony";
   } else {
   	echo "Administrator";
   }
}
 
[PHP] pobierz, plaintext 

Mysql

Ale dostaję komunikat Undefined variable: userrole in
i każdy użytkownik jest administratorem, mimo że powinien być zatrudnionym

[viking]

Nigdy nie rób uprawnień admina na else. Pewnie nie przekazujesz zmiennej do funkcji. Poczytaj o zasięgu zmiennych.

[dvdx95]

a jak wyglada twoje zapytanie sql?

[marcin690]

a jak wyglada twoje zapytanie sql?

Tak wygląda:

[PHP] pobierz, plaintext 
 
 
$mysqli = new mysqli(DB_SERVER, DB_USERNAME, DB_PASSWORD, DB_NAME);
$mysqli -> query("SET NAMES 'utf8' COLLATE 'utf8_polish_ci'");
 
$sql = "SELECT * FROM user";
$result = $mysqli->query($sql);
 
if ($result->num_rows > 0) {
 
   $row = $result->fetch_assoc();
} else {
    echo "0 results";
};
 
$identyfikator = $row["id"];
$username = $row["username"];
$name= $row["name"];
$email = $row["email"];
$phone = $row["phone"];
$lastlogin = $row["lastlogin"];
$address = $row["address"];
$userrole = $row["userrole"];
 
 
function userrole_display() // deklaracja funkcji
{
   if ($userrole === "1") {
   	echo "Zatrudniony";
   } else {
   	echo "Administrator";
   }
}
 
[PHP] pobierz, plaintext 

[nospor]

Jak juz viking napisal, twoj problem to:

Pewnie nie przekazujesz zmiennej do funkcji. Poczytaj o zasięgu zmiennych.

ps:
Ale zdajesz sobie sprawe ze to
$sql = "SELECT * FROM user";
zwraca ci wszystkich userow jakich masz w tabeli?

A zas to
$row = $result->fetch_assoc();
pobierze ci pierwszy zwrocony wiersz? A przez to jak kiedys baza zmieni kolejnosc zwracania rekordow to sie nizle zdziwisz

[dvdx95]

[PHP] pobierz, plaintext 
if ($result->num_rows > 0) {
 
   while ($row = $result->fetch_array()) {
     $identyfikator = $row["id"];
     $username = $row["username"];
     $name= $row["name"];
     $email = $row["email"];
     $phone = $row["phone"];
     $lastlogin = $row["lastlogin"];
     $address = $row["address"];
     $userrole = $row["userrole"];
     userrole_display($userrole);
   }
}
 else {
    echo "0 results";
};
 
function userrole_display($userrole) // deklaracja funkcji
{
   if ($userrole === "1") {
   	echo "Zatrudniony";
   } else {
   	echo "Administrator";
   }
}
[PHP] pobierz, plaintext 

Ten post edytował dvdx95 5.07.2019, 17:13:27

[Tomplus]

Viking ma rację. Nie daje się uprawnień administracyjnych w taki sposób, nawet funkcja jest dziwna.

userole_display() 1 - jestem zatrudniony, 0,2,3,n - jestem adminem
A gdzie status? nie jestem zatrudniony?