[PHP] Unset() nie działa poprawnie Opcje

[zabic]

Witam
Na poczatek kod: http://www.phpriot.com/2835

Tablica tmp_msg zawiera wszystkie informacje o napotkanych błędach. Później wyświetlam jej zawartość a następnie ją usuwam i co dziwne, te komunikaty dalej są obecne w tej tablicy. Dostaję je nawet po przeładowaniu strony. Co robię źle ?

pozdrawiam
zabic

[Pr0100]

[PHP] pobierz, plaintext 
$this->tmp_msg = array();
[PHP] pobierz, plaintext 

[zabic]

Nie pomogło. Dalej dostaje komunikaty

Temat dalej jest aktualny (IMG:style_emoticons/default/biggrin.gif)

[Zyx]

A nie zauważyłeś, że dwie linijki wyżej wywołujesz metody, które coś do tej tablicy wpisują?

[zabic]

Owszem, zauwazyłem. Ów metody, w razie potrzeby powinny umieszczać ów komunikaty w tej tablicy, później następuje wypisanie jej zawartości a na końcu chcialbym ją wyczyścić tylko nie wiem czemu, ale mi się to nie udaje ;p Jak usunąć jej zawartość ? ;p

Zresztą skopiuj sobie ten kod, zakomentuj konstruktor i zobacz jak działa;p

Ten post edytował zabic 4.05.2010, 12:36:01

[pyro]

Jak dla mnie to to wcale nie będzie działać jak powinno, gdyż:

[PHP] pobierz, plaintext 
$name=$this->validName(isset($data['name']));
        $surname=$this->validSurname(isset($data['surname']));
 
[PHP] pobierz, plaintext 

I dalej te funkcje:

[PHP] pobierz, plaintext 
public function validName($data) {
        preg_match_all('#^[a-zA-Z]*$#',$data, $data);
        return (empty($data[0][0])) ? $this->tmp_msg[]=$this->msg[1] : $data;    
    }
 
    public function validSurname($data) {
        preg_match_all('#^[a-zA-Z]*$#',$data, $data);
        return (empty($data[0][0])) ? $this->tmp_msg[]=$this->msg[2] : $data;    
    }
 
[PHP] pobierz, plaintext 

Jako argument do tych funkcji dajesz isset($data['surname']) lub name isset($data['name']), czyli wartość true/false, czyli te wyrażenia PCRE na nic się nie zdadzą przeciw temu typowi danych (IMG:style_emoticons/default/smile.gif)

[zabic]

Usunąlem isset() i dalej mam pelną tablice ;p

Ten post edytował zabic 4.05.2010, 13:44:39

[pyro]

Muszę przyznać, że trochę denerwująca jest Twoja ignorancja. Nie muszę nawet tego sprawdzać, bo widzę dokładnie co się dzieje w skrypcie. Napisałem teraz na szybko coś, a żebyś TY sobie sprawdził i się zdziwił, bo jestem pewien, że mam rację. Nie sprawdzałem, ale syntax errorów raczej nie ma:

[PHP] pobierz, plaintext 
<?php
 
class user {
 
 
public function validName($data) {
	var_dump($data); echo '<br />';
        preg_match_all('#^[a-zA-Z]*$#',$data, $data);
        return (empty($data[0][0])) ? $this->tmp_msg[]=$this->msg[1] : $data;    
    }
 
    public function validSurname($data) {
    	var_dump($data);
        preg_match_all('#^[a-zA-Z]*$#',$data, $data);
        return (empty($data[0][0])) ? $this->tmp_msg[]=$this->msg[2] : $data;    
    }
 
 
 
}
 
$data = array('name'=>'Jan','surname'=>'Kowalski');
$instance = new user();
$name=$instance->validName(isset($data['name']));
$surname=$instance->validSurname(isset($data['surname']));
 
?>
[PHP] pobierz, plaintext 

[zabic]

MIałeś racje dlatego szybko poprawiłem swojego wczesniejszego posta;p hmm... co jest z ta tablicą? Czemu ciągle ona coś zawiera mimo tego, że ją unsetuję ?

[pyro]

A jak wywołujesz dane funkcje z tej klasy?

[mortus]

Jak zawiera, skoro nie zawiera. Sprawdź:

[PHP] pobierz, plaintext 
<?php
class user {
    private $msg = array(1=>'Niepoprawne imię', 2=>'Niepoprawne nazwisko');
    public $tmp_msg = array();
 
    public function __construct() {
        //require_once ('class.database.php');
        //$DB = new database();
    }
 
    public function register($data) {
        $name = $this->validName($data['name']);
        $surname = $this->validSurname($data['surname']);
 
        if (! empty($this->tmp_msg)) {
            echo '<pre>';
            print_r($this->tmp_msg);
            echo '</pre>';
 
            unset($this->tmp_msg);
 
			echo '<pre>';
            print_r($this->tmp_msg);
            echo '</pre>';
 
        }
    }
 
    public function validName($data) {
        preg_match_all('#^[a-zA-Z]*$#', $data, $data);
        return ( empty($data[0][0])) ? $this->tmp_msg[] = $this->msg[1] : $data;
    }
 
    public function validSurname($data) {
        preg_match_all('#^[a-zA-Z]*$#', $data, $data);
        return ( empty($data[0][0])) ? $this->tmp_msg[] = $this->msg[2] : $data;
    }
}
 
$u = new user();
$data = array('name'=>'name', 'surname'=>'\surname');
$u->register($data);
print_r($u->tmp_msg);
$u->register($data);
?>
[PHP] pobierz, plaintext 

Jeśli ponownie wykonasz funkcję register dla obiektu $u, to pole tmp_msg klasy user znowu będzie zawierać przez krótką chwilę komunikaty.

Ten post edytował mortus 4.05.2010, 14:04:20

[zabic]

heh... wyczyściłem cache przeglądarki i już jest ok. Jeszcze jedno pytanko. W którym miejscu dodać funckję isset() by nie dostawać Notice: Undefined index: status in C:\www\register.php on line 10?

[PHP] pobierz, plaintext 
if($_POST['status']==1) {
			require_once('class.user.php'); //10 linia
			$USER=new user();
			$USER->register($_POST);
		}
[PHP] pobierz, plaintext 

[mortus]

Dziesiątą linią jest linia wyżej niż ta z komentarzem. W tym if-ie daj isset

[PHP] pobierz, plaintext 
if(isset($_POST['status']) && $_POST['status'] == 1) {
[PHP] pobierz, plaintext 

Poza tym isset() nie jest funkcją.

[zabic]

heh... działa...

Jak to isset nie jest funckją skoro pobiera argumenty i zwraca wartośc ? (IMG:style_emoticons/default/snitch.gif)

[mortus]

Jest specjalną konstrukcją języka PHP. Działa jak funkcja, ale funkcją nie jest. Poczytaj w Manual-u w ramce Notes.

[zabic]

Bardzo dziękuję za pomoc