Mam przykładowy kodzik:

[PHP] pobierz, plaintext 
<?php
 
function pisz() {
 
$zmienna = 1;
$zmienna2 = 2;
 
return ($zmienna + $zmienna2);
 
}  // nie chodzi o sam kod, jest on tylko przykładowy 
?>
[PHP] pobierz, plaintext 

I teraz w dokumencie php chcąc tą funckję wywołać to muszę przykładowo pisz();

Pytanie czy da się zrobić tak żeby funkcja była wywołana wtedy gdy jest umieszczona w kodzie bez tego wywoływania (np pisz();)czy za każdym razem trzeba to zrobić?

http://php.net/manual/en/function.function-exists.php

Ale ja nie chcę sprawdzać czy funkcja została wywołana tylko przy jej definiowaniu ją "od razu" samoistnie wywołać

aaa, to było jaśniej

to w pliku ktorym ja definiujesz od razu daj jej wywołanie

Ale tez muszę ją wywołać ręcznie? Nie mogę w funkcji zaznaczyc zeby sie wywołała? Wiem że to trochę dziwne co mówię

Deklaracja funkcji nie uruchamia jej. Zastanawia mnie, czemu nie możesz po prostu użyć po prostu "pisz();"?

możesz pobrać wszystkie funkcje
http://php.net/manual/en/function.get-defined-functions.php
php samo z siebie nie wywołuje wszystkich możliwych funkcji

No mogę mogę ale tak mnie to zastanawiało troszkę patrzylem w kody cms i tak nie zauważyłem jak wywołują ale deklarują musze to dokladniej przeanalizować dzięki

Nie można wywołać funkcji bez jej wywołania. A jeżeli wywołanie funkcji umiejścisz w jej ciele to powstanie nieskończona pęta.
To co piszesz świadczy o błędnym sposobie podejścia do konstruowania skryptów. Jest to niezgodne z zasadami sztuki mówiącymi chociażby o wielokrotnym wykorzystaniu tego samego kodu. Deklaracji funcji i innego kodu nie umieszczaj w miejscu jej wywołania ale w bibiotekach. Inaczej powstaje chaos.



Być może używają wywołań funkcji w sposób nieliteralny nie np. "pisz()" tylko np. $functionPisz() ale na pewno ten cms nie działa to w taki sposób że deklaracja fukcji wywołuje ją w miejscu jej deklaracji.

Czyli najlepiej utworzyc jakis plik np functions.php i tam wszystkie funkcje trzymać?

Funkcję można też wywołać np. tak
http://php.net/manual/en/function.call-user-func.php

Możesz wywołać funkcję anonimową, podobnie jak w JS: (function(){alert(1);})();

[PHP] pobierz, plaintext 
call_user_func(function(){
 echo '1';
});
[PHP] pobierz, plaintext 

Tylko jaki to ma sens...

To jest tzw. wywołanie rekurencyjne i nie zawsze oznacza nieskończoną pętlę



nie, bo po jakimś czasie Ci się plik rozrośnie i nagle zauważysz że ma kilkadziesiąt/set funkcji
pogrupować w pliki "tematycznie" czyli klasy -> doczytać oop

Moglbys coś wiecej o tych klasach? Jakis artykułl o tym czy cos? Nie mysl ze czekam na gotowe ale na razie nie wiem dokladnie co za "czarna magia" to jest "(

Zgasza się.
Wyobraź sobie co musiał byś zrobić kiedy będziesz chciał użyć funkcji "pisz" ponowanie w jakimś innym miejscu, jeżeli zadeklarujesz ją np. w pliku szalobnu. Iportować szablon html żeby wywołać funkcję? Totalna pomyłka. Lepszym rozwiązaniem jest kiedy deklaracje funcji umiesza się w jednym pliku np. functions.php który pełni rolę bibioteki i inludować tylko zawsze ten sam plik zawsze kiedy potrzebujesz użyć zadeklarowanych w nim funkcji. Między innymi dlatego język jest tak skonstruowany bo zakłada takie własnie użycie.

Mariner,



W takim razie jak się do wypowiedzi kolegi ustosunkujesz?

To o co pytasz to programowanie obiektowe. O programowaniu obiektowych w php jest dużo w necie, nie problem znaleźć. Niemniej jest to jedna z najbardziej istotnych kwestii i podstawa pracy programisty. Także obowiązkowo musisz znać i posługiwac się programowaniem obiektowym.



Żeby organizować funckje w klasy wpierw trzeba znać przeznaczenie tych funkcji. Wobec zadanego pytania, czy deklaracje umieszczać w miejscu wywołania, wystarczającą informacją jest to że należy trzymać kod w plikach bioblitecznych a czy ma to być tylko jeden plik oraz czy ma zwierać klasy zamiast funkcji, to kwestia względna i w tym pytaniu zdaje się nie istotna.



Naprawdę? A gdzie jest napisane że "zawsze oznacza"?

O programowaniu obiektowym słyszałem i też go troszkę już próbuję Ale jak to się ma do tego pliku functions.php z wszystkimi funkcjami? Kolega wyżej napisał że to zły pomysł bo się plik rozrośnie do wielu linijek kodu...

Duże partie kodu bibiotecznego umieszcza się w róznych plikach ze względów wydajnościowych, żeby nie ładować funkcji których nie potrzebujemy. Odpowiedź kolegi, o wielu plikach bibiotecznych opiera się na złozeniu że będziesz miał dużo funkcji. Ja tego nie wiem ile masz funkcji. Starałem się przedstawić Tobie jedynie tylko koncepcję mówiącą że deklarację umiejszczamy w bibiotekach a czy będzie to jeden plik czy wiele kwestia względna zależy od Ciebie.

W dużej ilość funcji faktycznie dobrym pomysłem jest ich podzielenie na plikich wedłuch spełnianych przez nie zadań. Np. wszystkie funkcje odpowiedzialne za komunikację z bazą umieszczamy w pliku functions_db.php, funkcje odpowiedzialne za przetwarzenie html functions_helper.php i tak dalej. Wtedy mamy łatwe rozeznanie który plik w danym momencie mamy zaincludować. I tak, jak wykonujemy update bazy danych to includujemy plik z funkcjami odpowiedzialnymi za komunikację z bazą danych, jak obrabiamy gdzieś html to potrzebuemy plik tylko z funkcjami odpowiedzialnymi za html. Zauważ że dzięki temu nie ładujemy wszystkich fukcji, osczędzając dzięki temu pamięć i nie obciążając serwer.
Z drugiej zaś strony jezeli masz np. tylko dwie funkcje to jaki jest sens umieszczania ich w osobnych plikach? Chyba żaden. Lepiej umieścić je w jednym pliku.
Tak to generalnie w skrócie wygląda.

Napisałeś to tak jak by zawsze wywoływało nieskończoną pętlę... precyzuj albo nie pisz głupot.


@autor, możesz pogrupować sobie w foldery odpowiednie biblioteki funkcyjne, wtedy łatwiej się odnaleźć we wszystkim.

Za mojej wypowiedzi nic nie wynika że zawsze czy nie zawsze bo to nie istotne a "tak jak by" to chyba Ty i kolega wyciagacie wnioski zbyt pochopnie.

Pytalem czysto teoretycznie milo mi ze taka dyskusja sie rozwinela

Fred, a tak w ogóle to co, dopiero zaczynasz? Długo już się bawisz skryptami?

Nie zaczynając nowego tematu

@Mariner zaczynam dopiero przyznam że mnie to interesuje i jak mam jakąś sprawę niejasną to się zwracam do userów na forum bo często takich rzeczy na google nie ma

Ale do rzeczy, chodzi mi o funkcję zwracającą wartość, wszystko jasne mam taki kod:

[PHP] pobierz, plaintext 
<?php
 
function connect_database() {
 
//dane do bazy ukryte
 
    $database = @new mysqli(HOST, USER, PASSWORD, DATABASE);
 
        if($database->connect_errno) {
 
            $output = array('type' => 'error', 'text' => 'Błąd połączenia z bazą danych! Sprawdź konfigurację połączeniową!');
            return $output;
        }
        if(!$database->set_charset("utf8")) {
 
            $output = array('type' => 'error', 'text' => 'Błąd wyboru kodowania!');
            return $output;
        }
 
    $output = array('type' => 'success', 'text' => 'OK');
    return $output;
 
 
}
 
$database = connect_database();
echo $database['type']. '  '. $database['text'];
 
 
?>
[PHP] pobierz, plaintext 

Co do działania tego kodu nie mam zastrzeżeń, ale skoro ja sam w funkcji definiuję co ona ma zwrócić to w przypadku wystąpienia błędu zwraca tablicę i wyświetla "Błąd połączenia z bazą danych! Sprawdź konfigurację połączeniową!" ale do tego wywala też błąd że nie można się połączyć z mysqli (wiadomo 2 linijki błędu) ale takie pytanie jako początkujący mam: wiem że dam sobie @ przed zmienną i się ten błąd już nie wyświetlia ale dlaczego tak jest? Przecież ja zwracam tylko tablicę output a ten błąd nie powinien się zwrócić... (POWINIEN? NO TAK ALE CZEMU? )

Bo php to jezyk interpretowany zreszta w normalnym jezyku masz tak samo. On wykonuje wszystko po kolei i jesli napotka blad to ten blad sie pokaze bo na nim interpreter sie zatrzyma. Dodanie @ to nic innego jak zaciemnienie czyli blad sie nie pokaze ale itak wystapi. Dlatego @ jest zla i sie jej nie uzywa. A wyjatki przechwytuje sie w blokach try catch

No ale w tym przypadku uzasadnione jest dodanie @ bo ja i tak wiem czy jest błąd czy go nie ma

Owszem sprawdzasz czy wystapil ale to nie sprawi ze jak on wystapi to ze skrypt sie nie zatrzyma @ oszukujesz poprostu skrypt i on idzie dalej

Czyli co proponujesz zmienić w tej funkcji?

@----EDIT----

[PHP] pobierz, plaintext 
<?php
 
function connect_database() {
 
    define('HOST', 'mysqdl1.ugu.pl');
    define('DATABASE', 'db675386');
    define('USER', 'db675386');
    define('PASSWORD', 'ZAQ!2wsx');
 
    $database = new mysqli(HOST, USER, PASSWORD, DATABASE);
 
        if($database->connect_errno) {
 
            $output = array('type' => 'error', 'text' => 'Błąd połączenia z bazą danych! Sprawdź konfigurację połączeniową!');
            die;
        }
        if(!$database->set_charset("utf8")) {
 
            $output = array('type' => 'error', 'text' => 'Błąd wyboru kodowania!');
            die;
        }
 
    $output = array('type' => 'success', 'text' => 'OK');
    return $output;
 
 
}
 
?>
[PHP] pobierz, plaintext 

Dobrą rzeczą byłoby zatrzymanie działania całego kodu jeśli błąd np?

Uzyj poprostu http://php.net/manual/en/class.mysqli-sql-exception.php

mógłbyś troszkę jaśniej do czego to służy bo jedno zdanie niewiele mi mówi

No wylapuje Ci wyjatki ktore wystapia podczas dzialania skrypu i zwraca odpowiedni komunikat. Mozna nie zwracac a tylko przechwycic i przekazac dalej poczytaj o wyjatkach w php a zrozumiesz bo nie ma sensu zebym przepisywal tobie tu definocje

Poczytam z chęcią

Czyli dzięki tym wyjątkom mogę tą moją przykładową funkcję jakoś lepiej zbudować?

Tak bo zlapiesz wszystkie bledy i przy okazji masz nad nimi kontrole

[PHP] pobierz, plaintext 
<?php
 
function database() {
 
    define('HOST', '...');
    define('USER', '...');
    define('PASSWORD', '...');   
    define('DATABASE', '...');
 
    try {
 
        $database = new mysqli(HOST, USER, PASSWORD, DATABASE);
 
            if($database->connect_errno) {
 
                throw new Exception('Błąd połączenia z bazą danych!');
            }
            if($database->set_charset("utf-8")) {
 
                throw new Exception('Błąd wyboru kodowania!');
            }
 
    }
    catch(Exception $error) {
 
        echo $error->getMessage();
    }
 
}
 
database();
?>
[PHP] pobierz, plaintext 

Teraz "ulepszyłem" (albo i nie ) skrpycik. Jeżeli można to prosiłbym o rady i opinie czy zrobiłem to dobrze, lub czy można to jeszcze poprawić.

Niestety, chociaż wszystko działa jak najbardziej poprawnie to dalej przy błędnych danych do bazy wyświetli się mój komunikat ale też php doda swoje 2 linijki że połączenia nawiązać nie można

@up
A to wszystko dlatego, że koledzy wyżej zapomnieli, że notice/warning to nie exception.

Ale jeżeli przed tym dorzucisz (np. na samym początku kodu)

[PHP] pobierz, plaintext 
mysqli_report(MYSQLI_REPORT_STRICT);
[PHP] pobierz, plaintext 

to php rzuci wyjątkiem w przypadku nieprawidłowego połączenia się z bazą danych (miej na uwadze, że linijki 13-22 z Twojego kodu nie zostaną wykonane, a co za tym idzie dostaniesz błąd wygenerowany przez php, a nie przez Ciebie).

No ok widzę jak to działa, ale chcąc np uzyskać efekt jaki ja chcę na pewno jakoś się da bo np w Extreme fusion, php fusion czy choćby w php by przemo jest to ładnie rozwiązanie, tzn wyświetla się "ładny" błąd

Czyli bazując na wyjątkach dalej tego efektu nie osiągnę?...
P.S Szukam w plikach od tych cms jak to jest zrobione ale na razie nie mogę się dokopać głębiej...

Nikt nie każe Ci przecież robić

[PHP] pobierz, plaintext 
echo $error->getMessage();
[PHP] pobierz, plaintext 

Zawsze możesz przecież zrobić

[PHP] pobierz, plaintext 
echo $szablonLadnejStrony;
[PHP] pobierz, plaintext 

Czyli co chcesz przez to powiedziec? Jakos ten błąd "upiększyć" ? Nadal nie ma tego efektu jak bym chcial miec muszę się dogrzebac do tych plikow od cms

Możesz zarzucić jakimś screenem, który zaprezentuje jaki efekt chcesz osiągnąć?

Tak za jakies pół godzinki screen będzie jak do domu wrócę

Proszę, tak prezentuje się u mnie:



A tak lepiej na php by przemo:



Jednak tutaj jakoś ładniej użytkownika informuje skrypt

@----EDIT----

Przepatrzyłem te cms'y i ku mojemu zdziwnieniu we wszystkich jest podstawowa obsługa błędów, ale jednak połączenie jest "zabezpieczone" za pomocą @ żeby jednak tych błędów nie wywaliło Jednak trzeba będzie tej małpki użyć

Z tą "@" jest taki problem, że na serwerze produkcyjnym i tak wyłącza się wyświetlanie jakichkolwiek błędów (w tym notice i warningów). Podczas rozwijania kodu zaś, informacja o tego typu błędach jest pożądana. Dlaczego więc stosować "@"?

Po co np. masz jakiś moduł do aplikacji, który dołączasz za pomocą include. Aplikacja jest sprzedawana bez modułów, które można sobie dokupić.
Ale notka w kodzie o nim musi być, bo chyba nie kazesz klientowi samemu edytowac skryptu.
Gdy include nie wykryje pliku to sypnie błedem, ale to, że go nie ma to nie odrazy błąd, tylko to, że ktoś go sobie nie kupił.
"@" świetnie się tutaj nadaje, by to ukryć.

Ale wróćmy do tematu jak uważacie czy tej @ mozna użyć czy raczej to zla praktyka?

Oczywiście, że zła praktyka. Błędy należy wyeliminować/obsługiwać, a nie je tłumić. Chyba, że masz wybitnie uzasadniony przypadek.

Ale przeciez próbuje ten blad wyeliminować tylko za cholerę sie tego nie da zrobić

Upewnij się że prawidłowe są dane za pomocą których następuje próba autoryzacji w serwerze bazy danych. Błąd mówi coś o nieznanym adresie host.
Błąd zrzucany jest dlatego że skrypt nie może połączyć się do bazy bo albo dane są nieprawidłowe lub adresat nie istnieje.

Dane do bazy specjalnie podaję złe żeby poradzić sobie z tym bledem i go ładniej obsluzyc...

Więc zaraz po próbie nawiązania połączenia, powinieneś

1. sprawdzić czy połączenie zostało nawiązane
2. jeżeli połączenia nie nawiązano
2.1 zrzucić wyjątek.

Tylko że taki scenariusz ma sens tylko wtedy kiedy masz w swoim systemie kod który takie wyjątki obsługuje, potrafi je przechwycić z zamienić na odpowiedź.

Jeżeli nie masz, to od punktu 2 możesz wykonać przekierowanie na np. statyczną stronę z błędem.

Owszem wyjątek pięknie zostanie obsłużony i wyświetli rządany komunikat tylko że warning dalej będzie

Kloc Co za bzdury, jak już budujesz aplikacje modułowe to nie dajesz wywołania do modułu który nie jest wbudowany a nie wyciszasz coś
Bo źle łapiesz te wyjątki, błędem rzuca Ci ten if if($database->connect_errno)