[PHP]OOP odwołanie do zmiennych innych metod Opcje

[gcdreak]

Witam! Mam następujący kod:

[PHP] pobierz, plaintext 
<?php
class ftp_serw{
                private $komp = "ftp.sewr.pl";
                private $user = "anomous";
                private $remote_file = "/sciezka/plik.zip";
                
                function __construct($komp){
                    $conn_id = ftp_connect($komp) or die ("Nie można połączyć się z serwerem $komp.<br />");
                        echo "Połączono z serwerem $komp.<br />";
                    $login = ftp_login($conn_id) or die ("Nie można się zalogować jako $user.<br />");
                        echo "Zalogowano jako $user";
                }
                function login($conn_id, $user = FALSE, $passw = FALSE){
                    $login = ftp_login($conn_id) or die ("Nie można się zalogować jako $user.<br />");
                         echo "Zalogowano jako $user";
                }
 
            }
?>
[PHP] pobierz, plaintext 

W konstruktorze mam utworzoną właściwości $conn_id. Potrzebuje jej w metodzie login. Przedstawiony przezemnie kod jest niepoprawny.
Licze na Waszą pomoc.

Powód edycji: Poprawiłem bbCode (ociu)

[PawelC]

Przecież konstruktor jest automatycznie ładowany!

I tą klase co podałeś używasz tak, np:

[PHP] pobierz, plaintext 
<?php
$ftp=new ftp_serw();
 
// dalszy kod odpowiedzialny za wykonanie czegoś
?>
[PHP] pobierz, plaintext 

Jaki błąd ci wywala?

Sprawdziłem u siebie i skrypt działa cały czas, ale nic nie wyświetla.
Skąd masz tą klasę?

Tutaj jest cała masa o ftp http://pl.php.net/manual/pl/ref.ftp.php

Tutaj jest przykład połączenia i zalogowanie na ftp:

[PHP] pobierz, plaintext 
<?php
 
$ftp_serwer            = "ftp.example.com";
$ftp_nazwa_uzytkownika = "foo";
$ftp_haslo             = "bar";
 
// nawiązanie połączenia lub zakończenie działania skryptu
$conn_id = ftp_connect($ftp_serwer) or die("Nie można połączyć się z $ftp_serwer");
 
// próba logowania
if (@ftp_login($conn_id, $ftp_nazwa_uzytkownika, $ftp_haslo)) {
    echo "Połączony jako $ftp_nazwa_uzytkownika@$ftp_serwer\n";
} else {
    echo "Nie można zalogować się jako $ftp_nazwa_uzytkownika\n";
}
 
// zamknięcie połączenia
ftp_close($conn_id);
 
?>
[PHP] pobierz, plaintext 

Prosto z manuala, co najważniejsze działa bez problemu bo sprawdzałem!

Ten post edytował ExPlOiT 25.04.2009, 23:10:56

[gcdreak]

@ExPlOiT
nie zrozumiałeś mnie. Strukturalnie wiem, że jest prosto, utworzyć egzemplaż obiektu też umię.
Jeśli w jednej motodzie mam utworzoną zmienną i potrzebuję ją użyć w drugiej to jak mam to zrobić? Rozumiecie o co mi chodzi?
>> w funkcji ftp_login() muszę podać "conn_id" które zostało utworzone w osobnej metodzie(konstruktorze).

Ten przykład: http://pl.php.net/manual/pl/ref.ftp.php#44110 chyba już wszystko wyjaśnia.
Rano sprawdzę i napiszę jeśli będę miał jakiś problem.

[PawelC]

Zobacz jak to zrobiłem w swojej klasie:

[PHP] pobierz, plaintext 
<?php
public function __construct($host,$login,$haslo,$baza)
            {
                $this->mysqli=new mysqli($host,$login,$haslo,$baza);
            }
            
            
            
            public function dane($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                while($data=$result->fetch_object()){
                
                    echo $data->kategoria;
                }
?>
[PHP] pobierz, plaintext 

Ten kod obrazuje jak to zrobić, w konstruktorze nawiązałem połączenie, które używam w metodzie dane($query) jak sam widzisz, czyli $this->mysqli, a w Twoim wypadku jest to $conn_id. W Twoim wypadku w funkcji ftp_login(), wystarczy użyć $conn_id, np

[PHP] pobierz, plaintext 
<?php
public function ftp_login()
{
$plik="jaki.php";
if($conn_id->ftp_delete($plik)) {
 echo "Plik $plik został usunięty\n";
} else {
 echo "Nie powiodła się próba usunięcia pliku $plik\n";
}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował ExPlOiT 26.04.2009, 00:23:10

[Zyx]

ExPlOiT, dalej piszesz głupoty . Przecież jak niby ma użyć zmiennej $conn_id jak ona nie będzie w tej funkcji istnieć?

gcdreak -> trochę bezsensownie korzystasz z możliwości, jakie daje Ci programowanie obiektowe. Do wewnętrznych zmiennych metody nigdy nie odwołasz się z zewnątrz, gdyż, jak wskazuje nazwa, są one wewnętrzne. Od tego, co musi być współdzielone między metodami, masz pola klasy i tam powinieneś przechowywać takie rzeczy. Co więcej, dotychczasowe pola, jakie utworzyłeś, także nie bardzo mają zastosowanie. Wpisałeś tam dane na sztywno, a uczyniłeś je prywatnymi, czyli de facto nie ma żadnej szansy, aby użytkownik klasy mógł tam wpisać własne ustawienia. Inna sprawa, że w ogóle później z tych pól nie korzystasz...

Powinieneś zrobić tak:

1. Pola $komp, $user, $remoteFile zostawiasz, ale na wstępie dajesz im puste wartości (przy okazji mógłbyś wszystkiemu nadać jakieś jednolite nazewnictwo, tj. albo nazywasz wszystko po angielsku, albo wszystko po polsku).
2. Konstruktor pobiera parametry połączenia jako swoje argumenty i wpisuje je do tych pól.
3. Reszta klasy korzysta już wyłącznie z tych pól.
4. Ponieważ pola są prywatne, a czasem jednak przydaje się możliwość odczytania ich wartości, możesz dodać metody w stylu getServer() albo getUser() zwracające ich wartości. W ten sposób będzie je można odczytać, ale już nie będzie można ich zmodyfikować po nawiązaniu połączenia.
5. Tak samo z tym Twoim $conn_id. Do zwykłej zmiennej nie masz co tego zapisywać. Musisz utworzyć prywatne pole, wstawić wartość właśnie tam i później z niego korzystać. Tutaj już nie robisz metody do zwracania identyfikatora połączenia, gdyż jest to wewnętrzna sprawa Twojej klasy i reszta skryptu nie musi mieć do niego dostępu.
6. Ponadto nie rób obsługi wyświetlania błędów bezpośrednio w klasie, a tym bardziej nie rób tego przez die(). Jedyne, co to daje, to ogranicza poważnie zastosowanie klasy. Zamiast tego wykorzystaj wyjątek, a wtedy programista będzie mógł decydować, co zrobić w przypadku wystąpienia błędu, bo to właśnie on jest w stanie najlepiej ocenić, czy błąd taki jest dla niego na tyle krytyczny, że trzeba zatrzymać skrypt, czy nie, a jeśli tak, to miło byłoby, aby mógł on sobie sam to jakoś ładnie wyświetlić.

PS. ORTOGRAFIA!!! Primo: egzemplarz, secundo: umiem.

[gcdreak]

@Zyx
Dzięki za przydatne rady.