Warum kann ich nicht $ app in aufgerufene Funktion für Slim verwenden?

Question

Ich versuche, eine wiederverwendbare Funktion zu machen, aber jedes Mal, wenn ich versuche, $ app in der Remote-Funktion zu verwenden, bekomme ich einen leeren Bildschirm. Hier ist, was funktioniert:

$app = new \Slim\Slim(); 

//GET CHAPTERS 
$app->get(
    '/chapters', 
    function() use ($app) { 
     $app->contentType('application/json'); 
     executeSql('SELECT * FROM chapters ORDER BY id'); 
    } 
); 

//GENERIC SQL EXECUTE 
function executeSql($sql) { 
    try { 
     $db = getConnection(); 
     $stmt = $db->query($sql); 
     $results = $stmt->fetchAll(PDO::FETCH_OBJ); 
     $db = null; 
     echo json_encode($results); 
    } catch(PDOException $e) { 
     echo '{"error":{"text":'. $e->getMessage() .'}}'; 
    } 
} 

Aber ich versuche, den json-Header in der Funktion zu bewegen und kann das nicht an der Arbeit (bekommt den leeren weißen Bildschirm):

$app = new \Slim\Slim(); 

//GET CHAPTERS 
$app->get(
    '/chapters', 
    function() { 
     executeSql('SELECT * FROM chapters ORDER BY id'); 
    } 
); 

//GENERIC SQL EXECUTE 
function executeSql($sql) use ($app) { 
     $app->contentType('application/json'); 
    try { 
     $db = getConnection(); 
     $stmt = $db->query($sql); 
     $results = $stmt->fetchAll(PDO::FETCH_OBJ); 
     $db = null; 
     echo json_encode($results); 
    } catch(PDOException $e) { 
     echo '{"error":{"text":'. $e->getMessage() .'}}'; 
    } 
} 

Ist etwas falsch mit meiner Syntax oder wie verwende ich PHP? Ich habe auch ohne die "Use ($ app)" überhaupt versucht aber immer noch das gleiche Problem.

Answer 1

Einfache Möglichkeit, dies zu beheben, ist die Verwendung von getInstance();

Answer 2

Das Schlüsselwort use gilt nur für Abschlusskonstrukte, abgesehen von der Verwendung mit Namespaces. Was Sie mit einer normalen Funktion machen, wird nicht unterstützt. Sie müssen global $app verwenden, was normalerweise als schlechte Praxis angesehen wird.

Answer 3

Sie könnten das Objekt $ app immer als Argument an Ihre Funktion übergeben.

$app = new \Slim\Slim(); 

//GET CHAPTERS 
$app->get(
    '/chapters', 
    function() use ($app) { 
     executeSql($app, 'SELECT * FROM chapters ORDER BY id'); 
    } 
); 

//GENERIC SQL EXECUTE 
function executeSql(\Slim\Slim $app, $sql) { 
     $app->contentType('application/json'); 
    try { 
     $db = getConnection(); 
     $stmt = $db->query($sql); 
     $results = $stmt->fetchAll(PDO::FETCH_OBJ); 
     $db = null; 
     echo json_encode($results); 
    } catch(PDOException $e) { 
     echo '{"error":{"text":'. $e->getMessage() .'}}'; 
    } 
} 

Answer 4

Es ist sauberer, Datenbankaufrufe und App-Logik/Präsentation getrennt zu halten. Ich würde Ihr "Kapitel" -Objekt sofort zurückgeben und alles andere außerhalb der executeSql-Funktion behandeln.

$app = new \Slim\Slim(); 

// GET CHAPTERS 
$app->get('/chapters', function() use ($app) { 

    $app->contentType('application/json'); 

    $chapters = executeSql('SELECT * FROM chapters ORDER BY id'); 

    if ($chapters) { 
    $app->response->setStatus(200); 
    echo json_encode($results); 
    } else { 
    $app->response->setStatus(400); 
    echo '{"error":{"text":"error getting chapters"}}'; 
    } 

}); 

// GENERIC SQL EXECUTE 
function executeSql($sql) { 
    try { 
    $db = getConnection(); 
    $stmt = $db->query($sql); 
    $results = $stmt->fetchAll(PDO::FETCH_OBJ); 
    $db = null; 
    return $results; 
    } catch(PDOException $e) { 
    echo '{"error":{"text":'. $e->getMessage() .'}}'; 
    } 
}