[PHP] Pierwsza Klasa Opcje

[Grafnastyk]

Witam, czy mógłby mi ktoś wyjaśnić dlaczego moja funkcja use_monster zwraca tylko pierwszą zmienną? Domyślam się, że błąd jest banalny, ale nigdzie nie mogę znaleźć rozwiązania a błędu nie wyrzuca mi żadnego.

[PHP] pobierz, plaintext 
<?php
 
    class Monster
    {
        public
        $name,
        $hp,
        $def,
        $atk;
 
     public function Create_monster($m_name, $m_hp, $m_def, $m_atk)
     {
         $this->name = $m_name;
         $this->hp = $m_hp;
         $this->def = $m_def;
         $this->atk = $m_atk;
 
     }
 
        public function use_monster()
        {
            return $this->name;
            return $this->hp;
            return $this->def;
            return $this->atk;
 
 
        }
 
    }
 
    $monster1 = new Monster;
 
    $monster1->Create_monster('Troll',100,100,100);
 
    echo $monster1->use_monster();
[PHP] pobierz, plaintext 

[markonix]

A jak sobie wyobrażasz, żeby zwróciło Ci jednocześnie kilka wartości? Po przecinku?

[viking]

return wychodzi natychmiast z metody. Najlepiej zwróć tablicę.
Natomiast poczytaj trochę o PSR i zdecyduj się na jedno nazewnictwo, najlepiej camelCase.

[leonpro778]

Nazewnictwo to aż nie taki wielki problem (chociaż też są przyjęte pewne standardy). Zacznij lepiej od zapamiętywania pewnych przyjętych "nawyków" przy programowaniu obiektowym.

1. Do każdego pola powinna być przypisana metoda GET i SET.
2. Pola metody powinny być prywatne a dostęp do nich (poprzez metody GET i SET) niech będzie publiczne.
3.Metodę Create_monster wrzuć w konstruktor.
4. Odnośnie nazewnictwa:
Nazwy pól, czyli hp, att nazywamy z _ na początku. Czyli $this->_mp
Nazwy metod, czyli przykładowo jakis GET. Pierwsza litera jest mała, czyli getHp()
Nazwa obiektu z dużej litery, czyli tak jak masz u siebie class Monster

I tyle jak na początek :-)

Odnośnie problemu - dostęp do tych danych, pobieraj wtedy kiedy ich potrzebujesz. I niestety nie możesz zrobić tego cztery razy pisząc return :-)

[Pyton_000]

Torchę popłynąłeś...

1. Gettery i Settery (bo tak to się nawzyaw) nie są wymagane, a czasami wręcz nie potrzebne.
2. _ się nie używa do określania widoczności properties.

Odnoścnie "nazewnictwa kodu" oraz tego co i jak polecam zapoznać się z
http://www.php-fig.org/psr/psr-1/
http://www.php-fig.org/psr/psr-2/

[Boshi]

"Troche.." on zatonął.

prefixów w postaci kresek się nie używa, akcesory nie są potrzebne zawsze, przykład Value obiect..

[leonpro778]

Tak trochę to nie ja "popłynąłem" co WY mnie nie zrozumieliście.

Po pierwsze:
Nic nie pisałem o GETTERACH i SETTERACH. To, że użyłem nazwy GET i SET wynika tylko właśnie z tego, że w jakiś sposób trzeba zmienną w pewien sposób "zmodyfikować" i do tego służy metoda, którą nazwałem sobie SET (popatrzcie na przykład jaki podałem w nazewnictwie). Mój błąd, że nie napisałem poprawnie.

Po drugie:
Prefiks w postaci _ przed zmienną ma określać WIDOCZNOŚĆ Gdzie ja tak napisałem? Napisałem tylko, że pola powinny być prywatne a metody publiczne. Oczywiście taki znak przed zmienną pomaga w późniejszej identyfikacji zmiennej ale nigdzie nie napisałem, że _ zmienia widoczność.

Ten post edytował leonpro778 10.12.2017, 09:18:37

[viking]

Czyli właśnie setter.
_ już od wielu lat się nie używa. I dobrze pyton napisał choć może skrótowo.

[markuz]

[PHP] pobierz, plaintext 
<?php
 
namespace Game;
 
class Monster
{
    private $name;
    private $attack;
    private $defense;
    private $life;
 
    public function __construct(string $name, int $attack, int $defense, int $life)
    {
        $this->name    = $name;
        $this->attack  = $attack;
        $this->defense = $defense;
        $this->life    = $life;
    }
 
    public function getName(): string
    {
        return $this->name;
    }
 
    public function getAttack(): int
    {
        return $this->attack;
    }
 
    public function getDefense(): int
    {
        return $this->defense;
    }
 
    public function getLife(): int
    {
        return $this->life;
    }
}
[PHP] pobierz, plaintext 

1. Obiekt tworzymy przez konstruktor a nie dodatkową metode.
2. Używamy normalnych nazw, bez żadnych skrótów które mogą wprowadzić w błąd.
3. Domyślnie każda właściwość powinna być prywatna

Ten post edytował markuz 10.12.2017, 13:05:08

[SmokAnalog]

Tak trochę to nie ja "popłynąłem" co WY mnie nie zrozumieliście.

Poczytałeś o dobrych nawykach, ale nie doczytałeś, że trzeba je stosować z głową. Getterów i setterów się używa tylko tam, gdzie są potrzebne. A z tym dolnym podkreśleniem to niektórzy mają taką zasadę, ale moim zdaniem jest idiotyczna. A już mówić o niej jak o jakiejś prawdzie objawionej to jakiś kosmos.

Ten post edytował SmokAnalog 10.12.2017, 13:15:13

[leonpro778]

Poczytałeś o dobrych nawykach, ale nie doczytałeś, że trzeba je stosować z głową.

Co jest błędnego w moim rozumowaniu?

Getterów i setterów się używa tylko tam, gdzie są potrzebne.

Wyjaśnijmy jedno, czym dla Ciebie jest:
getAttack() {
return $this->_att;
}

A z tym dolnym podkreśleniem to niektórzy mają taką zasadę, ale moim zdaniem jest idiotyczna.

Framework Zend - tam długo stosowali taką zasadę. Czy ja wiem czy jest to taka idiotyczna zasada?

A już mówić o niej jak o jakiejś prawdzie objawionej to jakiś kosmos.

Heh, czy ja pisałem o jakiejś prawdzie objawionej?

[nospor]

Framework Zend - tam długo stosowali taką zasadę. Czy ja wiem czy jest to taka idiotyczna zasada?

Skolo klucz z twojej wypowiedzi: STOSOWALI - czas przeszly
Kiedys ludzie przez wieki zyli w jaskiniach....

Teraz mamy PSR ktore standaryzuje pewne rzeczy, ale zawsze znajdzie sie taki ktos jak ty, kto nie ma internetu a jak juz sie dorwie do modemu u kogos to szerzy herezje sprzed stu lat na lewo i prawo

[viking]

Wpisz sobie

[PHP] pobierz, plaintext 
<?php
class Test {
    private $att;
    private $def;
 
}
[PHP] pobierz, plaintext 

W Netbeans wciśnij alt+insert - getter & setter. Masz wygenerowane wszystkie metody tylko że nazewnictwo leży. Podkreślenie owszem, stosowało się mocno w ZF1, tyle że gdybyś tak teraz oznaczał wszystko w ten sposób miałbyś potężne problemy np z hydratorem (który swoją drogą stosowałem praktycznie od jego wymyślenia również w projektach ZF1 i wymagało przemyślenia od nowa zwłaszcza nazw encji - modeli).

Ten post edytował viking 10.12.2017, 14:38:47

[leonpro778]

Skolo klucz z twojej wypowiedzi: STOSOWALI - czas przeszly
Kiedys ludzie przez wieki zyli w jaskiniach....

Dobra, zadałem sobie trudu aby odnaleźć ich bieżącą dokumentację:

For variables that are declared with private or protected visibility, the first character of the variable name MAY be a single underscore. This is the only acceptable application of an underscore in a variable name, and is discouraged (as it makes refactoring to public visibility more difficult). Member variables declared with public visibility SHOULD NOT start with an underscore.

Oooo, i w końcu dobry argument:

Wpisz sobie

[PHP] pobierz, plaintext 
<?php
class Test {
    private $att;
    private $def;
 
}
[PHP] pobierz, plaintext 

W Netbeans wciśnij alt+insert - getter & setter. Masz wygenerowane wszystkie metody tylko że nazewnictwo leży. Podkreślenie owszem, stosowało się mocno w ZF1, tyle że gdybyś tak teraz oznaczał wszystko w ten sposób miałbyś potężne problemy np z hydratorem (który swoją drogą stosowałem praktycznie od jego wymyślenia również w projektach ZF1 i wymagało przemyślenia od nowa zwłaszcza nazw encji - modeli).

Masz rację z generowaniem nazw. Ja jednak używam PHPStorm Efekt podobny dlatego i tak nazwy sobie piszę praktycznie od podstaw. Odpiszę dokładnie wieczorem bo z t telefonu piszę i ciężko jak cholera.

[nospor]

Nie bardzo rozumiem, o czym ma swiadczyc ten wycinek z dokumetnacji. Przeciez wyraznie napisali ze sie z tego wycofuja. Trzymaja tylko jako relikt przeszlosci.

Dostales linki do PSR - z nimi sie zapoznaj bo to jest to co teraz obowiazuje