Instrukcja if i dziwny problem Opcje

[PawelC]

Witam,
Zastanawia mnie jedna rzecz, mam taki oto kod:

[PHP] pobierz, plaintext 
#Update URL
	public function update($url, $id)
	{
		$result=$this->db->prepare("UPDATE url SET adres=:url where id=:id limit 1");
          $result->bindParam(':url',$url, PDO::PARAM_STR);
          $result->bindParam(':id',$id,PDO::PARAM_INT);
            if(!$result->execute()){
                throw new Exception('Adres nie został zaktualizowany!');
            }
	}
[PHP] pobierz, plaintext 

Idąc na logikę jeżeli wrzucę to w instrukcję if czyli

[PHP] pobierz, plaintext 
if($http->update($url, $id))
  {
      echo "Ok";
  }else{
    echo "Error";
  }
[PHP] pobierz, plaintext 

Po wykonaniu i poprawnej aktualizacji powinno mi pokazać OK, ale mimo to pokazuje Error i oczywiście aktualizuje dany rekord. Nie wiem czy ja coś pomieszałem, czy czegoś tu nie rozumiem.

Ten post edytował ExPlOiT 20.08.2017, 11:34:59

[viking]

Na logikę to metoda powinna coś zwrócić i ewentualnie rzucić wyjątkiem. Update nic nie zwraca.

[PawelC]

Boże faktycznie (IMG:style_emoticons/default/biggrin.gif) po poprawnym powinna zwrócić np return true, lub wyjątek w razie błędu...

[nospor]

Jesli faktycznie nadal chcesz rzucac w funkcji tym wyjatkiem, to caly twoj blok IF ELSE nie ma najmniejszego sensu. Rownie dobrze mozesz zrobic tak

[PHP] pobierz, plaintext 
$http->update($url, $id);
          echo "Ok";
[PHP] pobierz, plaintext 

Wynik bedzie dokladnie taki sam

[PawelC]

@nospor do takiego samego wniosku doszedłem, ale to, że nic nie zwracałem rozwiązało moje problemy. Nie wiedziałem co jest nie tak, ale nie wziąłem pod uwagę, że nawet w podstawach OOP mówią, że metoda powinna zwracać jakąś wartość.

[nospor]

Jesli twoja metoda, rzuca wyjatkiem, to zwracanie true tutaj tez nie ma wiekszego sensu. Dlatego caly czas ci mowie, ze twoj IF ELSE jest bez sensu w tym przypadku.
ALbo wywal wyjatek i zwraca true albo false i wowczas IF ELSE ma racje bytu, albo rzucaj tym wyjatkiem i lap wyjatki TRY CATCH. Ale nie mieszaj tych dwoch metod.

[PawelC]

Wywaliłem wyjątek i zwracam true, dalej to już instrukcja if sprawdza co zostało zwrócone (IMG:style_emoticons/default/smile.gif) Dziękuję bardzo za wskazanie błędów, teraz przynajmniej wiem, o co chodzi.

[nospor]

Wywaliłem wyjątek i zwracam true,

Mamy tylko nadzieje, ze w miejscu wywalonego wyjatku dales return false ?

[PawelC]

Oczywiście (IMG:style_emoticons/default/smile.gif)