Aktionen, Funktionen und Bedingungen

Bei der Entwicklung von Themen und Plugins ist es manchmal erforderlich, einem Hook mithilfe von bedingten Anweisungen einige Funktionen hinzuzufügen.

Beispiel:

function my_custom_function() {
    if( is_home()) {
       <---what should the function do--->
    }
}

 add_action( 'some_hook', 'my_custom_function' );

Nach meinem Verständnis wird is_homeder Inhalt der Funktion nicht ausgeführt , wenn eine andere Bedingung vorliegt ( return false), die Funktion wird jedoch ausgeführt, obwohl sie "leer" ist. Dies bedeutet, dass eine leere Funktion an den Hook übergeben wird. Auf diese Weise werden alle Beispiele im Codex mit bedingten Tags angezeigt.

Ich verstehe, dass dies sicher ist und keine signifikanten Auswirkungen auf die Ladezeiten haben sollte (wenn es überhaupt Auswirkungen auf die Ladezeiten gibt).

Ich habe gedacht, dass derselbe Code wie im Beispiel wie folgt geschrieben werden kann

if( is_home()) {
   function my_custom_function() {
     <---what the function should do--->
   }

  add_action( 'some_hook', 'my_custom_funtion' );

}

Dies überspringt alles vollständig, wenn is_homefalse zurückgegeben wird.

Es macht mir nichts aus, eine dieser beiden Methoden anzuwenden. Aber was ich wissen möchte, weil das erste Beispiel ausgiebig verwendet wurde, gibt es einen Codierungsstandard, der besagt, dass dies die richtige Methode ist, oder ist dies nach Ansicht von WordPress-Entwicklern eher die bevorzugte Methode oder nur eine persönliche Präferenz.

Die WordPress-Codierungsstandards für PHP geben nichts darüber an, und es gibt keinen anderen Standard. Daher liegt es an den Entwicklern, die eine oder andere Art zu wählen.

Ich muss sagen, dass die beiden Methoden unterschiedliche Ansätze haben; Während die erste bedingte Logik enthält, ist die zweite eine bedingte Funktionsdeklaration. Wenn Sie also versuchen, die Funktion aufzurufen, erhalten Sie einen schwerwiegenden Fehler.

Auch wenn die erste Methode verwendet wird , in dem die Funktion läuft (mit einem sehr geringen Einfluss auf die Leistung , die nicht relevant und ist verloren in der Anwendungslast), ist es ein besserer Ansatz ist es, es zu benutzen, weil , wenn die zweite Methode verwenden, die Geschäftslogik Ihre Anwendung wird von den Funktionen in die Dateianalyse verschoben.

Darüber hinaus sollten Sie berücksichtigen, dass es einen dritten Weg gibt, den Sie nicht erwähnt haben:

function my_custom_function() {
    // what the function should do
}

if ( is_home() ) {
    add_action( 'some_hook', 'my_custom_function' );
}

Der Vorteil dieses Ansatzes ist bei Verwendung der OOP-Programmierung deutlicher erkennbar: In diesem Fall ist eine klassenbedingte Deklaration nicht sinnvoll (und eine methodenbedingte Deklaration ist überhaupt nicht möglich), aber es ist sehr sinnvoll, Aufgaben nur unter bestimmten Bedingungen auszuführen ( Haken feuern).

Erstellen Sie keine Funktionen im laufenden Betrieb. Dies ist schwer zu lesen und zu debuggen. Implementieren Sie stattdessen die Trennung von Bedenken und trennen Sie die Registrierung von Rückrufen von ihrer Ausführung (Geschäftslogik). Das logische Festlegen der bedingten Prüfungen vor der Rückrufregistrierung ist jetzt ganz einfach.
Warten Sie, bis die Aktion template_redirectdiesen Registrierungshandler instanziiert, da Sie dann wissen, ob is_home()sie überprüft werden kann.

Beispiel

class Theme_Hooks
{ 
    public function setup()
    {
        if ( ! is_404() ) {
            add_action(
                get_stylesheet() . '_breadcrumb',
                [ new Breadcrumb, 'render' ]
            );
        }

        if ( is_home() ) {
            add_action(
                get_stylesheet() . '_home_widget',
                [ new Home_Widget, 'render' ]
            );
        }
    }
}

add_action( 'template_redirect', [ new Theme_Hooks, 'setup' ] );

Ich möchte nur hinzufügen, dass man im Allgemeinen vorsichtig sein sollte, wenn man bedingte Tags verwendet wie:

if( is_*() )
{
    // stuff
}

im globalen Bereich von functions.php, da es ausgeführt wird, bevor ein Filter oder eine Aktion mit do_action()oder ausgelöst wird apply_filters().

Ich würde persönliche Präferenz sagen.

Außerdem ist es sauberer, den Funktionsaufruf außerhalb der if-Anweisung und die darin enthaltene Aktion auszuführen. Auf diese Weise wird die Funktion von der Aktion immer noch nur aufgerufen, wenn die if-Anweisung true zurückgibt.

Wenn Sie Funktionsdefinitionen immer außerhalb der if-Anweisung halten, wird sichergestellt, dass Sie niemals eine Funktion aufrufen, die zufällig abgefangen und nicht abrufbar ist.

Ich möchte nur eine weitere Option hinzufügen. Das Hybrid Core Framework hat eine Methode namens "Smarter Hooks" implementiert, die den Kontext erkennt.

Die vollständige Quelle finden Sie hier:

https://github.com/justintadlock/hybrid-core/blob/master/functions/core.php

Idealerweise kann es folgendermaßen verwendet werden:

Ein Beispiel für einen einfachen Hook wäre 'hybrid_header'. Die Funktion do_atomic () erweitert diese um zusätzliche Hooks wie 'hybrid_singular_header', 'hybrid_singular-post_header' und 'hybrid_singular-post-ID_header'.