[PHP]Problem z funkcją:/ Opcje

[robos85]

mam taką funkcję:

[PHP] pobierz, plaintext 
<?php
function check_login_input(){
  $fields=array('login','pass');
  $error=array();
  if(!isset($_POST['log_in'])){
      return "";
  }
  else{
      foreach($fields as $value){
          switch($value)
          {
              case 'login'<span style="color: #007700":
                  if(!(preg_match('/^[a-zA-Z0-9ąĄćĆęĘłŁńŃóÓśŚżŻźŹ_]{3,32}$/', sqlesc($_POST[$value])))){
                      $error[]='musi zawierać od 3 do 32 znaków,<br />dozwolone znaki to litery, cyfry oraz znak podkreślenie(_)';
                  }
                  break;
              case 'pass'<span style="color: #007700":
                  if(strlen($_POST[$value])<6 || strlen($_POST[$value])>40){
                      $error[]='musi zawierać od 6 do 40 znaków';    
                  }
                  break;
          }
      }
          if(isset($error)){
              return $error;
          }
          else{
              return '';
          }
  }
}
?>
[PHP] pobierz, plaintext 

lecz nie działa poprawnie, bo nie zwraca prawidłowego error.
Co tu skopałem

[Shili]

Po case 'cośtam' umieszcza się :

Nie zwraca prawidłowego, to znaczy jaki zwraca i dla jakich danych?
Weź pod uwagę, że skrypt masz niepełny, nie można go uruchomić na lokalu nie przerabiając i dorabiając, więc precyzuj najbardziej jak to tylko możliwe.

[robos85]

: mam, coś ten bbcode mi ucina. Podaję kod na phpfi: http://phpfi.com/351671
tak mam funkcję napisaną

[Shili]

Nadal nie wiem dokładnie jaki jest problem.

Swoją drogą za każdym razem, nawet dla dobrych danych powinno Ci zwrócić błąd (o to chodzi?). Bo zmienną error masz ustawioną wcześniej, czyli isset zwróci zawsze true. Nie wiem czy nie lepiej zamiast

[PHP] pobierz, plaintext 
<?php
if(isset($error)){
              return $error;
          }
          else{
              return '';
          }
?>
[PHP] pobierz, plaintext 

Byłoby napisać

[PHP] pobierz, plaintext 
<?php
if(count($error)){
              return $error;
          }
          else{
              return '';
          }
?>
[PHP] pobierz, plaintext 

Jeśli nie o to chodziło z nieprawidłowym error, to dużo bardziej dokładnie opisz z czym jest problem

Ten post edytował Shili 9.09.2008, 09:30:06

[robos85]

łączę to e smarty do wywalania błędów poprawności forma. Podaję cały kod jaki używam:
plik php który przetwarza a później do smarty daje dane

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['log_in'])){
check_login_input();
}
?>
[PHP] pobierz, plaintext 

i przypisanie do smarty

[PHP] pobierz, plaintext 
<?php
$smarty->assign('error',$error);
?>
[PHP] pobierz, plaintext 

a tu linia w smarty:

[HTML] pobierz, plaintext 
{$error.login}
[HTML] pobierz, plaintext 

tak dt się przedstawia i nie działa.
Jak na sztywno (nie orzez funkcję) zadeklaruję array to działa, więc stąd wniosek, ze ta funkcja trafiona.

[Shili]

{$error.login}

Oczywiście, że nic Ci nie zwróci

Przypatrz się, jak deklarujesz elementy tablicy - nie nadajesz im kluczy o nazwie login i haslo, nadajesz im kolejne liczby. Wyświetl sobie pod koniec funkcji print_r($error) i sprawdź sobie swoje indeksy

Chcesz się tak odwoływać? To nie korzystaj ze składni $error[] = ... a $error['klucz'] = ...
Gdzie klucz, to oczywiście dla podanego przykładu login.

Ten post edytował Shili 9.09.2008, 09:37:16

[robos85]

zmiana na

[HTML] pobierz, plaintext 
{$error[0]}
[HTML] pobierz, plaintext 

nie pomaga
zmiana error w funkcji na:

[PHP] pobierz, plaintext 
<?php
$error[$value]="...";
?>
[PHP] pobierz, plaintext 

też nie pomaga

jakiś pomysł?

Ten post edytował robos85 9.09.2008, 09:45:58

[Shili]

Zrób tak:

[PHP] pobierz, plaintext 
<?php
print_r(check_login_input())
?>
[PHP] pobierz, plaintext 

tam gdzie wywołujesz funkcję. Żeby nie zwracać błędów smarty możesz tymczasowo dać potem die();

Coś się wyświetla?
Jakie dane wpisujesz w formularz?

Zmieniłeś

[PHP] pobierz, plaintext 
<?php
if(isset($error)){
?>
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
<?php
if(count($error)) {
?>
[PHP] pobierz, plaintext 

?

Podaj jeszcze sposób wywoływania tejże funkcji.

Jeśli to ten kod:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['log_in'])){
check_login_input();
}
?>
[PHP] pobierz, plaintext 

to chyba zapomniałeś wyniku funkcji przypisać do jakiejś zmiennej:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['log_in'])){
$error = check_login_input();
}
?>
[PHP] pobierz, plaintext 

[robos85]

Przez nieuwagę nie przypisałem wyniku do zmiennej
Już działa:)