Zagnieżdżanie jednej funkcji w drugiej Opcje

[ddzkm]

Stworzyłem niedawno skrypt odpowiadający za rejestrowanie zdarzeń:

[PHP] pobierz, plaintext 
<?php
$date = date('d-m-Y');
$time = date('H:i:s');
$log_saver = 'folder/'.$date.'.log';
$ip = $_SERVER['REMOTE_ADDR'];
 
// Jeśli dziennik nie został utworzony stwórz go
 
    if (!file_exists($log_saver))
        {
        $create = fopen($log_saver, 'w');
        fwrite($create, ''.$date.' '.$time.': Utworzenie dziennika logowania zdarzeń na dzień '.$date.';'.'
');
        fclose($create);
        }
 
function log_page_loaded()
    {
    global $ip;
    global $log_saver;
    global $date;
    global $time;
    $host = gethostbyaddr($ip);
    $file = fopen($log_saver, 'a');
    fwrite($file, ''.$date.' '.$time.': Wejście na stronę z adresu IP "'.$ip.'" ('.$host.');'.'
');
    fclose($file);
    }
 
function log_user_login()
    {
    $username = $_POST['username'];
    global $ip;
    global $log_saver;
    global $date;
    global $time;
 
    $file = fopen($log_saver, 'a');
    fwrite($file, ''.$date.' '.$time.': Użytkownik '.$username.' zalogował się pomyślnie z adresu IP "'.$ip.'";'.'
');
    fclose($file);
    }
 
function log_user_logout()
    {
    $username = $_SESSION['login'];
    global $ip;
    global $log_saver;
    global $date;
    global $time;
    $file = fopen($log_saver, 'a');
    fwrite($file, ''.$date.' '.$time.': Użytkownik '.$username.' wylogował się;'.'
');
    fclose($file);
    }
 
function log_user_not_login()
    {
    $username = $_POST['username'];
    global $ip;
    global $log_saver;
    global $date;
    global $time;
    $file = fopen($log_saver, 'a');
    fwrite($file, ''.$date.' '.$time.': Nieudana próba zalogowania się użytkownika '.$username.' z adresu IP "'.$ip.'";'.'
');
    fclose($file);
    }
 
?>
[PHP] pobierz, plaintext 

Powyszy skrypt działa bez problemów, ale postanowiłem go trochę uprościć...

[PHP] pobierz, plaintext 
<?php
$date = date('d-m-Y');
$time = date('H:i:s');
$log_saver = 'logs/'.$date.'.log';
$ip = $_SERVER['REMOTE_ADDR'];
$host = gethostbyaddr($ip);
$username = $_SESSION['login'];
 
if (!file_exists($log_saver))
    {
    $create = fopen($log_saver, 'w');
    fwrite($create, ''.$date.' '.$time.': Utworzenie dziennika logowania zdarzeń na dzień '.$date.';normal;'.'
');
    fclose($create);
    }
 
function add_log()
    {
    global $date;
    global $time;
    global $log_saver;
    global $ip;
    global $host;
    global $username;
    global $log_message;
    global $log_type;
 
    $file = fopen($log_saver, 'a');
    fwrite($file, ''.$date.' '.$time.': '.$log_message.';'.$log_type.''.'
');
    fclose($file);
    }
 
function log_page_loaded()
    {
    $log_type = 'normal';
    $log_message = 'Wejście na stronę z adresu IP "'.$ip.'" ('.$host.')';
    add_log();
    }
 
function log_user_login()
    {
    $log_type = 'action';
    $log_message = 'Użytkownik '.$username.' zalogował się pomyślnie z adresu IP "'.$ip.'"';
    add_log();
    }
 
function log_user_logout()
    {
    $log_type = 'action';
    $log_message = 'Użytkownik '.$username.' wylogował się';
    add_log();
    }
 
function log_user_not_login()
    {
    $log_type = 'error';
    $log_message = 'Nieudana próba zalogowania się użytkownika '.$username.' z adresu IP "'.$ip.'"';
    add_log();
    }
 
?>
[PHP] pobierz, plaintext 

...i przestal działać. Funkcja add_log(); wykonuje się poprawnie, ale nie zapisuje treści komunikatu. Czy da sie coś z tym zrobić, czy trzeba korzystać z I rozwiązania?

Pozdrawiam!

[wookieb]

[PHP] pobierz, plaintext 
function add_log($log_message)
[PHP] pobierz, plaintext 

i wywolujesz poprzez

[PHP] pobierz, plaintext 
add_log($log_message)
[PHP] pobierz, plaintext 

Usuwasz też

[PHP] pobierz, plaintext 
global $log_message;
[PHP] pobierz, plaintext 

Ten post edytował wookieb 8.08.2009, 09:12:18

[pyro]

a najprościej rzecz biorąc byłoby to zrobić obiektowo (klasę)

[ddzkm]

Dopiero się uczę PHP, na razie wymyśliłem to co widać powyżej