Laravel und eine While-Schleife

Ich bin neu in Laravel und im moment habe ich ein Stück code in eine Steuerung, die ohne die while Schleife funktioniert es, ruft es meine Abfrage aus der Datenbank.

public function dash($id, Request $request) {
        $user = JWTAuth::parseToken()->authenticate();
        $postdata = $request->except('token');

        $q = DB::select('SELECT * FROM maps WHERE user_id = :id', ['id' => $id]);

        if($q->num_rows > 0){
            $check = true;

            $maps = array();
            while($row = mysqli_fetch_array($q)) {
                $product = array(
                    'auth' => 1,
                    'id' => $row['id'],
                    'url' => $row['url'],
                    'locationData' => json_decode($row['locationData']),
                    'userData' => json_decode($row['userData']),
                    'visible' => $row['visible'],
                    'thedate' => $row['thedate']
                );
                array_push($maps, $product);
            }
        } else {
            $check = false;
        }

        return response()->json($maps);
    }

Ich versuche, eine Schleife durch die zurückgegebenen Daten aus $q und verwenden json_decode auf 2 key/val-Paare, aber ich kann nicht einmal das richtige tut.

InformationsquelleAutor ngplayground | 2015-10-29
  1. 2

    Nicht verwenden mysqli zum Durchlaufen der Ergebnisse (Laravel nicht verwenden mysqli). Ergebnisse kommen zurück von Laravel ist Abfrage-generator sind Überfahrbar, so können Sie einfach mit einer foreach-Schleife:

    $q = DB::select('...');
foreach($q as $row) {
    //...
}

    Jeder $row ist ein Objekt und kein array:

    $product = array(
    'auth' => 1,
    'id' => $row->id,
    'url' => $row->url,
    'locationData' => json_decode($row->locationData),
    'userData' => json_decode($row->userData),
    'visible' => $row->visible,
    'thedate' => $row->thedate
);
    InformationsquelleAutor Jeff Lambert
  2. 2
    1. Sie sind nicht mit $postdata in dieser Funktion, so entfernen Sie es.
    2. Nicht verwenden, mysqli in Laravel. Verwenden Modelle und/oder die DB-Abfrage-Funktionalität gebaut in.
    3. Du bist die übergabe der falschen Sache zu mysqli_fetch_array. Es ist immer die Rückgabe einer nicht-falschen Wert und das ist der Grund, warum die Schleife nie enden.
    4. Warum sind Sie Schleifen über der Zeile Daten? Einfach nur wieder die Abfrage-Ergebnisse-Sie sind bereits in einem array. Wenn Sie Dinge wie "locationData' und 'userData' dekodiert werden JSON-dann verwenden Sie ein Modell mit Methoden, dies zu tun Sachen für Sie. Denken Sie daran, mit MVC sollte man immer etwas-bezogenen Daten in Modelle.

    Damit ein besserer Weg, dies zu tun ist mit Laravel Modelle und Beziehungen:

    //put this with the rest of your models
//User.php
class User extends Model
{
    function maps ()
    {
        return $this->hasMany ('App\Map');
    }
}

//Maps.php
class Map extends Model
{   
    //you're not using this right now, but in case your view needs to get
    //this stuff you can use these functions
    function getLocationData ()
    {
        return json_decode ($this->locationData);
    }

    function getUserData ()
    {
        return json_decode ($this->userData);
    }
}

//now in your controller:
public function dash ($id, Request $request) {
    //$user should now be an instance of the User model
    $user = JWTAuth::parseToken()->authenticate();

    //don't use raw SQL if at all possible
    //$q = DB::select('SELECT * FROM maps WHERE user_id = :id', ['id' => $id]);
    //notice that User has a relationship to Maps defined!
    //and it's a has-many relationship so maps() returns an array
    //of Map models
    $maps = $user->maps ();

    return response()->json($maps);
}
    • +1 hoffentlich nicht nit Kommissionierung hier, aber ich nennen würde das Modell Map als jede Instanz stellt nur 1 Zeile.
    • Ich verwendet, um zu tun, aber dann habe ich festgestellt, dass ich Häufig Dinge zu tun, wie Maps::where('foo', 'bar')->get(); und so für den code, den ich geschrieben hatte, machte es Sinn, es zu behalten im plural da, dass Modell ist verantwortlich für ALLE Karten, das heißt, die Tabelle der Karten. Aber Sie haben definitiv einen Punkt. Ich denke, es hängt nur davon ab, den Stil der typischen Verwendung.
    InformationsquelleAutor erapert
  3. 0

    Können Sie mit einer Schleife über $q mit einem foreach:

    foreach ($q as $row) {
  //Do work here
}

    Sehen die Laravel Dokumentation für weitere Informationen.

    InformationsquelleAutor Ivan
Schreibe einen Kommentar