Laravel-datatables: すべての行にグローバル変数を割り当てるにはどうすればよいですか。

作成日 2017年07月04日  ·  16コメント  ·  ソース: yajra/laravel-datatables

問題または機能のリクエストの概要

addColumn()関数とeditColumn()関数を使用する場合は、データテーブルの反復ごとに1つの変数を設定する必要があります。

問題のコードスニペット

何かのようなもの
`` `
Datatables :: of($ users)を返します
-> setGlobal(function($ user){
$ status = getUserStatus($ user-> id) ;
})
-> addColumn( 'status'、function($ user){
// getUserStatus($ user-> id);を返します
$ statusを返します。
});
-> editColumn( 'is_authorized'、function($ user){
// getUserStatus($ user-> id)を返しますか? 真/偽;
戻り値($ status)? 真/偽;


so, each time I don't need to call this **getUserStatus($user->id)** function.
<!--
If applicable, please include a copy of your code
which triggers the suspected bug.

You may use the markdown php code tags to format your paste:

```php
$params = ['foo'];

->

システムの詳細

  • オペレーティング・システム
  • PHPバージョン
  • Laravelバージョン
  • Laravel-Datatablesバージョン
question

最も参考になるコメント

グローバル? クロージャで変数を渡すだけですか?

$status = getUserStatus($user->id);

return Datatables::of($users)
->addColumn('status', function ($user) use ($status) {
  return $status;
});

全てのコメント16件

グローバル? クロージャで変数を渡すだけですか?

$status = getUserStatus($user->id);

return Datatables::of($users)
->addColumn('status', function ($user) use ($status) {
  return $status;
});

反復ごとにステータスを変更したい
ユーザーごとに変更する必要があります。

return Datatables::of($users) ->setGlobal(function ($user){ //for $user->id = 1 //status = true //for $user->id = 2 // status = false **$status** = **getUserStatus($user->id)**; }) ->addColumn('status', function ($user) { // return getUserStatus($user->id); return $status; }); ->editColumn('is_approved', function ($user) { //return getUserStatus($user->id) ? true : false; return ($status) ? true : false;

開発者のレビューの前に問題を閉じないでください。

各反復で必要な場合は、 GLOBALではありません。

->addColumn('status', function ($user) {
    return getUserStatus($user->id);
});
->addColumn('status', function ($user) {
    return getUserStatus($user->id);
})
->editColumn('xyz', function ($user) {
    return getUserStatus($user->id);
});

追加または編集する列ごとに、同じ関数を呼び出す必要があります。 これによりMySqlクエリが呼び出されるため、この同じ関数を使用する5つの列がある場合、1回の反復で同じMySqlクエリを5回呼び出し、1000レコードの場合は1000 * 5 = 5000重複クエリになります。

ページングを使用する場合、最低20レコードに対しても20 * 5 = 100の重複クエリ。

N + 1の問題を解決したい場合は、熱心な読み込みを使用してください。

参照については、このデモを参照してください//datatables.yajrabox.com/relation/belongs-to

または、結合ステートメントを使用して、クエリにステータスを含めます。

https://datatables.yajrabox.com/fluent/joins

$assignments = DB::table('assignments as a')
        ->when($round_id != "" || $round_id != NULL, function ($query) use ($round_id) {
            return $query->where('a.round_id', '=', $round_id);
        })
        ->leftJoin('rounds as r', 'a.round_id', '=', 'r.id')
        ->leftJoin('surveys as su', 'a.id', '=', 'su.assignment_id')
        ->leftJoin('projects as p', 'r.project_id', '=', 'p.id')
        ->leftJoin('sites as s',  'a.site_id', '=', 's.id')
        ->leftJoin('fieldreps as f', 'a.fieldrep_id', '=', 'f.id')
        ->leftJoin('chains as ch','p.chain_id','=','ch.id')
        ->leftJoin('clients as c','ch.client_id','=','c.id')
        ->when(is_string($status) && $status != "", function ($query) use ($status, $current_date) {
            if($status == 'late'){
                return $query->where(function ($query) use ($current_date) {
                    $query->where(function ($query)  {
                        $query->where(DB::raw('IFNULL( DATE_FORMAT(a.deadline_date,"%Y-%c-%e"), DATE_FORMAT(r.deadline_date,"%Y-%c-%e"))'), '<', DB::raw('CURDATE()'));
                    })
                    ->where(function ($query) use ($current_date) {
                        $query->where('a.is_scheduled', '=', true)
                        ->where('a.is_reported', '=', false)
                        ->where('a.is_partial', '=', false);
                    });
                });
            }else if($status == 'scheduled'){
                return $query->where(function ($query) use ($current_date) {
                    $query->where(function ($query)  {
                        $query->where(DB::raw('IFNULL( DATE_FORMAT(a.deadline_date,"%Y-%c-%e"), DATE_FORMAT(r.deadline_date,"%Y-%c-%e"))'), '>=', DB::raw('CURDATE()'));
                    })
                    ->where(function ($query) use ($current_date) {                     
                        $query->where('a.is_scheduled', '=', true)
                        ->where('a.is_reported', '=', false)
                        ->where('a.is_partial', '=', false);
                    });
                });
            }
            else if($status == 'pending'){
                return $query->leftJoin('assignments_offers as ao', function($join)
                {
                    $join->on('a.id', '=', 'ao.assignment_id');
                })
                ->where('a.is_scheduled', '=', false)
                ->where(DB::raw('(SELECT count(id) as offer_count FROM assignments_offers where assignment_id = a.id and is_accepted is null)'), '<=', '0')
                ->groupBy('a.id');
            }else if($status == 'offered'){
                return $query->leftJoin('assignments_offers as ao', function($join)
                {
                    $join->on('a.id', '=', 'ao.assignment_id');
                })
                ->where(['a.is_scheduled' => false, 'a.is_offered' => true])->where(['ao.is_accepted' => NULL])
                ->groupBy('ao.assignment_id');
            }else if($status == 'reported'){
                return $query->where(['a.is_reported' => true, 'is_approved' => false]);
            }else if($status == 'partial'){
                return $query->where('a.is_partial', true);
            }else if($status == 'completed' || $status == 'approved'){
                return $query->where('a.is_approved', true);
            }
        })
        ->select([
            'a.id',
            'a.fieldrep_id',
            'a.deadline_date',
            'a.is_scheduled',
            'a.is_reported',
            'a.is_partial',
            'a.is_offered',
            'a.is_approved',
            'c.client_logo',
            'p.id as project_id',
            'p.project_name',
            'r.id as round_id',
            'r.round_name',
            's.site_code',
            's.site_name',
            's.city',
            's.state',
            's.zipcode',
            'su.id as survey_id',
            DB::raw("CONCAT(IFNULL( DATE_FORMAT(a.schedule_date,'%d %b %Y'), DATE_FORMAT(r.schedule_date,'%d %b %Y')), ' ' , IFNULL(TIME_FORMAT(a.start_time, '%h:%i %p'), TIME_FORMAT(r.start_time,'%h:%i %p'))) as assignment_scheduled"),
            DB::raw("CONCAT(IFNULL( DATE_FORMAT(a.deadline_date,'%d %b %Y'), DATE_FORMAT(r.deadline_date,'%d %b %Y')), ' ' , IFNULL(TIME_FORMAT(a.start_time, '%h:%i %p'), TIME_FORMAT(r.deadline_time,'%h:%i %p'))) as assignment_end"),
            DB::raw('CONCAT(f.first_name," ",f.last_name) as schedule_to'),
            DB::raw('(select COUNT(id) as offer_count from assignments_offers where assignment_id = a.id and is_accepted is null group by assignment_id) as offer_count'),
            ]);

クエリはこれほど複雑です。 積極的な読み込みが可能になるとは思わない

列の追加と編集でgetXYZ関数とsetXYZ関数を使用するには、Eloquentのオブジェクトが必須です。

それほど複雑なクエリは期待していませんでした。 ^ _ ^

それでは、キャッシュの使用を検討できますか? ステータスがあまり頻繁に変更されない場合は、より長い期間設定することもできます。

-編集-
多分次のようなものです:
`` `php
function getUserStatus($ id){
Cache :: Remember( 'user.status。'。$ id、1、function($ id){を返す
Status :: find($ id);を返します。
});
}

->setGlobal(function ($user){
//for $user->id = 1 //status = true
//for $user->id = 2 // status = false
$status = getUserStatus($user->id);
})

このようなものがそこにあることを願っています

ステータスを取得できるようにするには、各ユーザーを反復処理する必要があるため、提案setGlobalについてはよくわかりません。したがって、複数の関数呼び出しが引き続き実行されます。

私は本当に何百もの重複したクエリを取り除く方法を探しています。 getXYZとsetXYZを複数の列で呼び出すためです。

私も同じ問題を抱えており、グローバル(共通)変数を使用して、反復ごとに共通変数として使用できるようにしたいと考えています
@ruchisheth何かあったら、答えを教えてください

事前に助けてくれてありがとう.. :)

同じ@ruchisheth

@rajanjain@haratmalliこれに対する解決策はありません。

こんにちは、最初の列に入れてみましたか

データ->彫像= 4;

2番目のコールバックでそれを呼び出しますか?

このページは役に立ちましたか?
0 / 5 - 0 評価