Jak zablokować wyswietlanie sie wiadomosci przy uruchomieniu skryptu Opcje

[dzesi]

Witam pierwszy raz mam taki problem ze niemoge sie z nim sam uporać
A więc mam dany skrypt np rejestracji i w nim mam komunikaty dla poszczególnych funkcji, błędów itd
ale jak wywołam go poprzez przeglądarkę pokazują mi sie komunikaty, ale ja bym chciał żeby one pokazywał sie tylko wtedy kiedy jest dany błąd .Jak ożywam echo pokazują sie wszystkie komunikaty ale jak daje return to zaden sie nie pokazuje, może to błąd logiczny z mojej winy co o tym sądzicie ?
np dajmy taki fragment kodu

[PHP] pobierz, plaintext 
<?php
public function spradzanie_hasla_i_spradzanie_loginu()  { // nazwa funkcji wymyślone pisana na szybko dla przykładu
 
   if ($this->haslo!=$this->haslodwa){
  return "Podajesz złe hasło !!!";
  }
  elseif ($this->login==$ob->login)
  {
	  return 'Taki user istnieje';
  }
	}
?>
[PHP] pobierz, plaintext 

Jeśli ktoś wpisze dwa inne hasła pokazuje sie błąd "Podajesz złe hasło !"; itd
Może troszkę głupie pytanie ale jakie są wasze rady ?

[deirathe]

4 linkijka zamiast "=" "==";
return nie wyswietla ko unikatow tylko zwraca wartosc co jest wielka roznica, jezeli chcesz zeby wyswietlalo komunikat to robisz taK:

[PHP] pobierz, plaintext 
<?php
echo sprawdzanie_hasla_i_loginu();
?>
[PHP] pobierz, plaintext 

jak nie podziala to wyslij caly kod moze masz wiecej bugow niz sie spodzeiwasz
pozdrawiam

[dzesi]

Witam po raz kolejny po męczarniach z których tak nic nie wynikło jestem zmuszony napisać do was o poradę
o to kawałek klasy w której występuje problem

[PHP] pobierz, plaintext 
<?php
// konstruktor 
 
if (!empty($_POST['login'])&& !empty( $_POST['haslo'])&& !empty($_POST['haslotwo']))
	{
				  $this->login=$_POST['login'];
				  $this->haslo=$_POST['haslo'];
				  $this->haslotwo = $_POST['haslotwo'] ;				
	 }
	 else {
	  echo ("Nie wprowadziłeś Wszystkich  danych do formularza "); // pojawia sie nawet jak odpalę goły skrypt 
	  }
		}
/////////////////////////////////////////////////// sprawdzanie czy hasła i login został  wpisany taki sam jak w formularzu 
 $ob=mysql_fetch_object($sprawdzlog);			 
	   }
	public function spradzanie_hasla()  {
 
   if ($this->haslo!==$this->haslotwo);
 
	}
	public function sprawdzanie_loginu()
	{
	if($this->login==$ob->login);
 
	}
/////////////////////////////////////////////////////////////////
$start=new Rejestracja($_POST[login],$_POST[haslo],$_POST[haslotwo]);
if(!$start->spradzanie_hasla())
{
	echo  'Podajeś złe hasło !!!';
}
if(!$start->sprawdzanie_loginu())
{
	echo 'Taki user istnieje';
}
$start->polaczenie();
if ($start->spraw_user())
{
 echo "Dodano Nowego usera pomyślnie ";  
}
 
if (!$start->rej_user())
{
	   echo "Nie udało sie dodać";
}
//echo  "Dziękujemy za rejestracje w naszym serwisie";
?>
[PHP] pobierz, plaintext 

Proszę mi powiedzieć jakie błędy popełniam bo to ważne zęby więcej nie popełniać błędów podobnych

[phpion]

1. Nie używaj $_POST w klasie (o ile to klasa bo średnio dostrzegam class...). Do konstruktora przekazuj dane w ten sposób:

[PHP] pobierz, plaintext 
<?php
$o = new Obiekt($_POST['zmienna');
?>
[PHP] pobierz, plaintext 

a sam konstruktor definiuj w ten sposób:

[PHP] pobierz, plaintext 
<?php
class Obiekt {
public function __construct($zmienna) {
// rob cos ze $zmienna
}
}
?>
[PHP] pobierz, plaintext 

2. Masz dziwną manię pisania metod, które nic nie robią:

[PHP] pobierz, plaintext 
<?php
public function spradzanie_hasla()  {
if ($this->haslo!==$this->haslotwo);
}
?>
[PHP] pobierz, plaintext 

Przydałby się jakiś return czy coś.

Ogólnie kod to totalny bałagan...

Ten post edytował phpion.com 6.10.2007, 14:49:27