Forum PHP.pl > Klasa wiadomości - sprawdzanie czy naciśnięto przycisk etc.

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: Klasa wiadomości - sprawdzanie czy naciśnięto przycisk etc.

Forum PHP.pl > Forum > PHP > Object-oriented programming

Riot

25.11.2014, 17:26:30

Cześć, po sporej dawce wiedzy OOP (przynajmniej wg mnie) postanowiłem napisać klase która się mi wkońcu do czegoś przyda, mianowicie system wiadomości. Rozkminiłem sobie to tak że będą odzielne klasy - Nowa_Wiadomosc oraz Skrzynka_Odbiorcza. Pierwsze pytanie - dobrze myśle?

Napisałem sobie na razie taki szybki kod, nawet nie przypominający OOP:

[HTML] pobierz, plaintext 
<html>
<body>
	<form action="" method="POST">
	<input type="text" name="wiadomosc">
	<input type="submit" name="wyslij">
	</form>
</body>
</html>
[HTML] pobierz, plaintext

[PHP] pobierz, plaintext 
<?php
 
class Nowa_Wiadomosc {
 
	function napisz_wiadomosc (){
		if (filtruj_pole($_POST["wyslij"])) {
			if (filtruj_pole($_POST["wiadomosc"], 2)) {
			//wysyla wiadomosc...
			}
		}
	}
 
	function filtruj_pole($pole, $min_znakow = 0) { //sprawdza czy pole jest prawidlowo wypelnione
		if (isset($pole)) {
			if (strlen($pole) >= $min_znakow) {
			return strlen($pole);
			}
		}
	}
}
 
$wiad = new Nowa_Wiadomosc();
echo $wiad->filtruj_pole($_POST["wiadomosc"]);
?>
[PHP] pobierz, plaintext

Tera tak - jak sprawdzić z gracją czy wciśnieto przycisk "wiadomosc"? Mogę zrobić if (isset($_POST["wiadomosc"])) {...} ale interesuje mnie jak to zrobić w OOP. Gdybyście podrzucili jakiś kod, byłoby znakomicie. Dzięki

Turson

25.11.2014, 18:34:23

1. Klasy i metode pisze się w stylu camelCase.
2. Przyjęło się, że używa się angielskich nazw przy nazewnictwie klas i metod.
3.

Cytat

Rozkminiłem sobie to tak że będą odzielne klasy - Nowa_Wiadomosc oraz Skrzynka_Odbiorcza. Pierwsze pytanie - dobrze myśle?

W teorii tak.
4.

Cytat

Tera tak - jak sprawdzić z gracją czy wciśnieto przycisk "wiadomosc"? Mogę zrobić if (isset($_POST["wiadomosc"])) {...} ale interesuje mnie jak to zrobić w OOP

a if(isset(.. nie jest z gracją? Innego sposobu nie ma. Może być if(!empty ale czy to dodaje gracji? Póki nie masz własnego mechanizmu obsługi żądań to nic innego nie wymyślisz.
5.

[PHP] pobierz, plaintext 
if (filtruj_pole($_POST["wyslij"])) {
[PHP] pobierz, plaintext

Zapominasz o słowie kluczowym $this
6.

[PHP] pobierz, plaintext 
	function filtruj_pole($pole, $min_znakow = 0) { //sprawdza czy pole jest prawidlowo wypelnione
if (isset($pole)) {
if (strlen($pole) >= $min_znakow) {
return strlen($pole);
}
}
}
[PHP] pobierz, plaintext

if(isset($pole - bez sensu. Pole musi istnieć skoro przekazujesz je jako argument. Jak nie przekażesz to PHP wywali bład,

Riot

25.11.2014, 19:28:08

Cytat(Turson @ 25.11.2014, 18:34:23 )

1. Klasy i metode pisze się w stylu camelCase.
2. Przyjęło się, że używa się angielskich nazw przy nazewnictwie klas i metod.

Zdaje sobie z tego sprawę

Cytat(Turson @ 25.11.2014, 18:34:23 )

Zapominasz o słowie kluczowym $this

Too wtedy jeden obiekt = pole. Tak?

Cytat(Turson @ 25.11.2014, 18:34:23 )

if(isset($pole - bez sensu. Pole musi istnieć skoro przekazujesz je jako argument. Jak nie przekażesz to PHP wywali bład,

Podejrzewałem że działa to w sposób -> przesyłam sobie to pole, sprawdza czy istnieje. Bo w źródle można zmienić nazwę pola

Widocznie sie myliłem.

Turson

25.11.2014, 20:06:06

Cytat

Too wtedy jeden obiekt = pole. Tak? wink.gif

Przeczytaj kurs OOP jeszcze raz

Cytat

Podejrzewałem że działa to w sposób -> przesyłam sobie to pole, sprawdza czy istnieje. Bo w źródle można zmienić nazwę pola biggrin.gif Widocznie sie myliłem.

Ale nie sprawdzasz czy ktoś nie zmienił nazwy a tylko czy zmienna pochodząca z wymaganego argumentu istnieje.

Riot

25.11.2014, 21:55:12

Czytałem pincet razy. Kilka lat pisania w języku w którym dopiero wprowadzają 'pseudo' OOP robi swoje. Mogłeś napisać po prostu gdzie popełniam błąd

Turson

25.11.2014, 21:57:45

Sprawdź $this i self

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.