Laravel - интеграция большого необработанного запроса в загрузку отношений - PullRequest
0 голосов
/ 23 апреля 2019

Я использую Laravel уже около 4 лет, но сейчас я сталкиваюсь со своей самой большой проблемой.Я занимаюсь рефакторингом и расширяю старое приложение PHP, используемое для сопоставления офисов, где я работаю.Это огромный запрос SQL, который мне нужно как-то интегрировать в QueryBuilder от Laravel, чтобы передать загруженные отношения.

Поток выглядит примерно так

Building => hasMany: Floor => hasMany: Seat => hasMany: BookedSeat => belongsTo: User

, где Building, Floor, Seat и BookedSeat являются Eloquent моделями.

Мой огромный запрос выбирает из BookedSeat Seat резервирования для текущей даты на основе многих других условий, например, если человек, которыйзабронировано место находится в домашнем офисе, в отпуске и т. д. (они хранятся в некоторых других таблицах) и задает свойство для экземпляра BookedSeat с именем Status, чтобы узнать, принято ли значение Seat на текущий день или нет.

Теперь я пытаюсь интегрировать этот необработанный запрос в построение иерархии JSON, которую я позже отправлю приложению Vue.js, работающему во внешнем интерфейсе.

Иерархия - это нечтонапример:

 {
   "buildings": [
     {
        // properties
        "floors" : [
           {
              //properties
              "seats": [
                  {
                     //properties
                     "booked": [
                        {
                           "user": "some user model",
                           "Status": "some booked status"
                        }
                      ]
                  },
                  // other seats
              ] 
           }, 
           // other floors
        ]
     },
     //other buildings
   ]
}

Огромный запрос возвращает массив объектов, которые я затем могу использовать для гидратации коллекции BookedSeat, но я не представляю, как я могу затем использовать эту коллекцию или использовать огромную очередьпопробуйте напрямую, чтобы загрузить BookedSeat для каждого Seat для каждого Floor для каждого Building и позволить каркасу сделать тяжелую работу за меня.

То, что я пробовал, - это создание методанапример:

public static function bookedSeatsForFloor(Relation $seatQuery, Relation $bookedQuery, Carbon $forDate)
    {
        $format = $forDate->format('Y-m-d H:m:i');
        $bindings = $seatQuery->getBindings();

        /** @var AvailableSeatsQuerySimple $availableSeats */
        $availableSeats = new AvailableSeatsQuerySimple($bindings, $format); // bindings are my floor id's and I'm feeding them to my big query in order to have control over which floors to load depending on the user's rights 

        return DB::raw($availableSeats->getRawQuery());
    }

и вызвать его так:

Floor::where('id', $someId)->with(['seats' => static function ($seatQuery) use ($_that) {

    /**
     * Add to each seat the manager and the active booking
     */
    $seatQuery->with(['booked' => static function ($bookedQuery) use ($seatQuery, $_that) {
        return self::bookedSeatsForFloor($seatQuery, $bookedQuery, $_that->forDate);
    }, 'manager'])->orderBy('seat_cir');
}]);

Но мне нужно как-то изменить $bookedQuery в методе bookedSeatsForFloor с помощью $bookedQuery->select('something') или $bookedQuery->setQuery('some query builder instance') но я не знаю, как преобразовать огромный запрос в экземпляр Builder.

Спасибо!

PS: Я бы предпочел пропустить переписывание огромного запроса в красноречивый синтаксисиз-за сложности

ДОБАВЛЕННЫЕ ДЕТАЛИ:

Итак, по запросу, это мой необработанный запрос, в котором я изменил некоторые имена баз данных / таблиц в соответствии с политикой компании

    SET NOCOUNT ON;
DECLARE
    @the_date DATETIME;
SET @the_date = (SELECT CONVERT(DATETIME, ?, 120));

SELECT seat_identifier,
       user_id,
       FromDate,
       ToDate,
       Status
FROM (
         SELECT d.seat_identifier,
                d.user_id,
                d.FromDate,
                d.ToDate,
                CASE
                    WHEN d.Status IS NULL
                        THEN 0
                    WHEN d.Status = 2
                        THEN 2
                    WHEN d.Status != 0
                        THEN CASE
                                 WHEN -- New, OnGoing Request in Main_DB_Name
                                             ho.status = 1 -- New StatusType in Main_DB_Name
                                         OR
                                             ho.status = 4 -- Pending StatusType in Main_DB_Name
                                         OR
                                             twl.status = 1 -- New StatusType in Main_DB_Name
                                         OR
                                             twl.status = 4 -- Pending StatusType in Main_DB_Name
                                         OR
                                             li.status = 1 -- New StatusType in Main_DB_Name
                                         OR
                                             li.status = 4 -- Pending StatusType in Main_DB_Name
                                         OR
                                             ctaf.status = 1 -- New StatusType2 in Main_DB_Name
                                         OR
                                             ctaf.status = 2 -- Ongoing StatusType2 in Main_DB_Name
                                     THEN
                                     2 --> Pending seat in MyApplication


                                 WHEN -- Approved Request in Main_DB_Name
                                             ho.status = 2
                                         OR
                                             twl.status = 2
                                         OR
                                             li.status = 1
                                         OR
                                             li.status = 2
                                         OR
                                             ctaf.status = 1
                                         OR
                                             ctaf.status = 2
                                     THEN 0 -- Free Seat MyApplication

                                 ELSE 1 -- Taken  Seat MyApplication
                        END
                    END as 'Status'
         FROM (
                  SELECT seats.seat_identifier as seat_identifier,
                         c.user_id,
                         c.FromDate,
                         c.ToDate,
                         c.Status
                  FROM (
                           SELECT fo_bs.seat_identifier,
                                  fo_bs.user_id,
                                  fo_bs.FromDate,
                                  fo_bs.ToDate,
                                  fo_bs.Status
                           FROM MyApplication.another_schema.BookedSeats fo_bs
                                    INNER JOIN MyApplication.another_schema.seats AS seats ON fo_bs.seat_identifier = seats.seat_identifier
                               WHERE fo_bs.FromDate <= @the_date
                                    AND fo_bs.ToDate >= @the_date
                                    AND fo_bs.Status IN (1, 2)
                                    AND seats.floor_id IN (###FLOOR_IDS###) -- will replace this from php with a list of "?,?,?" depending on how many floor_ids are in the query bindings
                       ) c
                           INNER JOIN MyApplication.another_schema.seats AS seats ON c.seat_identifier = seats.seat_identifier) d
                  LEFT JOIN (SELECT requester, status
                             from Main_DB_Name.schema.HOME_OFFICE
                                 WHERE Main_DB_Name.schema.HOME_OFFICE.from_date <= @the_date
                                      and Main_DB_Name.schema.HOME_OFFICE.to_date >= @the_date) ho ON d.user_id = ho.requester
                  LEFT JOIN (SELECT requester, status
                             from Main_DB_Name.schema.TEMPORARY_WORK_LOCATION
                                 WHERE Main_DB_Name.schema.TEMPORARY_WORK_LOCATION.from_date <= @the_date
                                      and Main_DB_Name.schema.TEMPORARY_WORK_LOCATION.to_date >= @the_date) twl
                            ON d.user_id = twl.requester
                  LEFT JOIN (SELECT employee, status
                             from Main_DB_Name.schema.LEAVE_INVOIRE
                                 WHERE Main_DB_Name.schema.LEAVE_INVOIRE.leave_date = @the_date) li ON d.user_id = li.employee
                  LEFT JOIN (SELECT requester, status
                             from Main_DB_Name.schema.TRAVEL
                                 WHERE Main_DB_Name.schema.TRAVEL.from_date <= @the_date
                                      and Main_DB_Name.schema.TRAVEL.until_date >= @the_date) ctaf
                            ON d.user_id = ctaf.requester
     ) y

И мои модели / отношения следующие:

class Building extends Model {
  /* Properties */

   public function floors()
    {
        return $this->hasMany(Floor::class);
    }
}

class Floor extends Model {
  /* Properties */
   public function building()
    {
        return $this->belongsTo(Building::class);
    }

   public function seats()
    {
        return $this->hasMany(Seat::class);
    }
}

class Seat extends Model {
  /* Properties */
   public function floor()
    {
        return $this->belongsTo(Floor::class);
    }

   public function booked()
    {
        return $this->hasMany(BookedSeat::class);
    }
}

class BookedSeat extends Model {
  /* Properties */
   public function user()
    {
        return $this->belongsTo(User::class);
    }

   public function seat()
    {
        return $this->belongsTo(Seat::class);
    }
}

1 Ответ

0 голосов
/ 25 апреля 2019

Проблема довольно сложная.Я застрял в этом, пытаясь делать разные вещи больше недели, но не мог найти хорошего способа сделать это.

В итоге я воспользовался предложением @Jonas Staudenmeir, вручную отобразив все мои вложенные отношения, а затем установив соответствующее отношение booked в моих экземплярах модели Seat из коллекции, полученной с использованием BookedSeat::hydrate(), с результатами.из необработанного запроса в качестве аргумента.

$availableSeats = new AvailableSeatsQuerySimple($format);

        // Map through all the Buildings
        $this->template['buildings'] = $this->template['buildings']
            ->map(static function (Building $building) use ($availableSeats) {

                // Map through all the Floors in a Building
                $floors = $building->floors->map(static function (Floor $floor) use ($availableSeats) {
                    /** @var BookedSeat|Collection $booked */
                    $booked = $availableSeats->execute($floor->id); // execute the raw query and get the results

                    if(count($booked) > 0) {

                        // Map through all the Seats in a Floor
                        $seats = $floor->seats->map(static function (Seat $seat) use ($booked) {

                            // Select the BookedSeat for the corresponding Seat
                            /** @var BookedSeat $bookedSeatForRelation */
                            $bookedSeatForRelation = $booked->filter(static function (BookedSeat $bookedSeat) use ($seat) {
                                return $bookedSeat->seat_identifier === $seat->id;
                            })->first();

                            // Attach the BookedSeat to the Seat only if the Status IS NOT 0
                            if($bookedSeatForRelation !== null && $bookedSeatForRelation->Status !== 0) {
                                return $seat->setRelation('booked', $bookedSeatForRelation);
                            }

                            return $seat->setRelation('booked', null);
                        });

                        return $floor->setRelation('seats', $seats);
                    }

                    return $floor;
                });

                return $building->setRelation('floors', $floors);
            });
...