Forum PHP.pl > [PHP]Pytanie o funkcję

Pomoc - Szukaj - Użytkownicy - Kalendarz

jaca121212

14.05.2019, 14:37:48

Witam, może dość głupie pytanie ale "KTO PYTA NIE BŁĄDZI" a więc do rzeczy:
Pomyślałem czy można zrobić funkcję w PHP która by odpowiadała za sprawdzanie czy użytkownik jest zalogowany i ma dostęp do danej treści na stronie.
Jeżeli tak, czy dałoby się to zrobić w funkcji. Gdzie zamiast pisania na każdej stronie, podstronie dość długiego kodu, wystarczyłoby wywołać funkcję.
Przykład:
Na każdej stronie , podstronie ma zapisany taki kod

[PHP] pobierz, plaintext 
<?
 session_start();
if(isset($_SESSION['succes-user'])){
 
               unset ($_SESSION['succes-user']);
              }
if(isset($_SESSION['zalogowany']) && isset($_SESSION['zalogowany']) == true)
{
 
header('Location: <a href="http://localhost/nowywyglad/news/app-layout/main.php&#39%3b%29;" target="_blank">http://localhost/nowywyglad/news/app-layout/main.php');</a>
}
?>
[PHP] pobierz, plaintext

gdzie zamiast takiego wyżej kodu można by było zapisać w ten sposób

[PHP] pobierz, plaintext 
function name()
{
kod  sprawdzający usera
}
 
if(name()==true)
{
Treść dozwolona tylko dla zalogowanych
}
[PHP] pobierz, plaintext

Tomplus

14.05.2019, 15:52:38

Na każdej stronie powinnien być kod

[PHP] pobierz, plaintext 
require "plik_sprawdzajacy_czy_zalogowany.php";
[PHP] pobierz, plaintext

a funkcja name() powinna się nazywać inaczej:

isLogged() i zwracać wartość true:false
getName() -> dopiero powinna zwracać nazwę użytkownika

wtedy if(isLogged()) { content }

jaca121212

14.05.2019, 17:35:16

Chciałem to zrobić w ten sposób ale nie mogę tego ugryźć.

Kod

<?php
require('../true.php');
?>
<!Doctype>
<HTML lang="pl">
<head>
    <meta charset="UTF-8">
    <link rel="stylesheet" type="text/css" href="../app-css/main.css">
</head>
<body>
<hrader>
    <nav>
    <ul>

         <li><a href="#">Profil <?php
        if(isLogin())
         {
             echo $_SESSION['user'];
         }
          else
           {
               echo "Gość";
           }
          ?></a></li>
        <li><a href="#">Ogłoszenia</a></li>
        <li><a href="#">Szukaj</a></li>
        <li><a href="http://localhost/nowywyglad/news/app-request/logout.php">Wyloguj się</a></li>
    </ul>
    </nav>
</hrader>

oraz osobny plik true.php

[PHP] pobierz, plaintext 
<?php 
 session_start();
function isLogin()
{
if(isset($_SESSION['succes-user']))
 {          
 unset ($_SESSION['succes-user']);
 }
if(isset($_SESSION['zalogowany']) && isset($_SESSION['zalogowany']) == true)
	{
			echo"zalogowany";
	}
 
}
 
 
?>
[PHP] pobierz, plaintext

Błędów nie ma żadnych wartość po zalogowaniu zwraca Zalogowany Gość po wylogowaniu zwraca samo Gość

Tomplus

14.05.2019, 17:43:01

tak jak napisałem funckja isLogin() powinna zwracać albo true albo false, a nie `echo`

[PHP] pobierz, plaintext 
function isLogin()
{
	isset($_SESSION['succes-user']) ? unset ($_SESSION['succes-user']):null;
	if(isset($_SESSION['zalogowany']) && isset($_SESSION['zalogowany']) == true)
	{
		return true;
	}
	return false;
}
[PHP] pobierz, plaintext

Tzn. echo też może jak potrzebujesz prześledzić działanie funkcji.

W ogóle w takich miejscach krócej jest użyć coś takiego, bez żadnych ifów, klamerek

[PHP] pobierz, plaintext 
<li><a href="#">Profil <?php  echo isLogin() ? $_SESSION['user'] : "Gość"; ?></a></li> 
[PHP] pobierz, plaintext

jaca121212

14.05.2019, 17:47:03

W tym zapytaniu mają być dwa pytajniki bo krzyczy mi błędem

[PHP] pobierz, plaintext 
isset($_SESSION['succes-user'])  unset ($_SESSION['succes-user']):null;
[PHP] pobierz, plaintext

Cytat

Parse error: syntax error, unexpected 'unset' (T_UNSET) in C:\xampp\htdocs\nowywyglad\news\true.php on line 5

Naprawiłem ten błąd dodając

Kod

session_unset

ale gdy zaloguje się to pojawia się kolejny błąd.

Cytat

Profil Warning : session_unset() expects exactly 0 parameters, 1 given in C:\xampp\htdocs\nowywyglad\news\true.php on line 5 aza121212

Tomplus

14.05.2019, 17:57:27

Gdzie ja użyłem dwa pytajniki?
Jest tylko jeden znak zapytania.

widzisz session_unset() to też rozwiązanie - manuela zawsze pomoże.

Jak masz wersję PHP7 to możesz użyć podwójny znak zapytania:

[PHP] pobierz, plaintext 
$var = 10;
echo $abc ?? $var ?? 20;
[PHP] pobierz, plaintext

W powyższym przykładzie wynikiem będzie oczywiście 10.
Mimo zę nie ma zadeklarowanej zmiennej $abc, a jakby nie było $var to wynik byłby 20.

jaca121212

14.05.2019, 18:06:17

A ten błąd co oznacza ?

Cytat

Profil Warning : session_unset() expects exactly 0 parameters, 1 given in C:\xampp\htdocs\nowywyglad\news\true.php on line 5 aza121212

I jak go naprawić ?

viking

14.05.2019, 18:11:30

Przecież masz dokładnie napisane. Funkcja spodziewa się 0 parametrów a podajesz jej 1.

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.