Laravel 4-发送数据控制器以查看

Question

概述：

我有此方法getDailyTimeRecordEntry()

public function getDailyTimeRecordEntry($record_id = false) {

    $user_id = Auth::user()->id;

    $dtr =  DailyTimeRecord::where('id', '=', $record_id)
            ->where('user_id', '=', $user_id);

    if ($dtr->count() > 0) { 
        $dtr = $dtr->first();
    } else if ($dtr->count() == 0 && $record_id != false) {

        // If other user accessing someone's record id, will redirect
        return  Redirect::route('dashboard-shinra')
                ->with('failure', 'The hell are you accessing someone\'s record?');
    }

    // if the dtr->count() is 0 or has a value it still send the data to view
    return  View::make('dashboard.shinra.dtr_entry')
            ->with('dtr', $dtr);
}

因此我们知道url是否具有记录ID或空白的参数，它仍会发送我传递给查看的dtr值

没有参数： www.mydomain.com/dashboard/shinra/daily-time-record-entry

使用参数： www.mydomain.com/dashboard/shinra/daily-time-record-entry/2

因此，在视图dashboard.shinra.dtr_entry

我有这行代码中有一个dtr变量。

<input type="text" name="date" value="{{ ($dtr->date) ? $dtr->date : '' }}">

因此，如果我们访问

www.mydomain.com/dashboard/shinra/daily-time-record-entry/1

所有记录都将在文本字段中输出，就像上面的输入一样。 因为参数1在数据库中有一条记录。

所以如果我访问

www.mydomain.com/dashboard/shinra/daily-time-record-entry

没有参数，我会遇到错误

我知道是因为这个

{{ ($dtr->date) ? $dtr->date : '' }}

但是，对此有任何解决方法吗？

我什至试过

{{ (isset($dtr->date)) ? $dtr->date : '' }}

仍然不起作用。

Answer 1

在方法getDailyTimeRecordEntry() ，还应该处理$record_id为false时的情况($dtr->count() == 0 && $record_id == false) 。 否则，您可能会遇到以下情况：没有get()或first() $dtr查询。

因此，您应该具有：

public function getDailyTimeRecordEntry($record_id = false) {

    $user_id = Auth::user()->id;

    $dtr =  DailyTimeRecord::where('id', '=', $record_id)
            ->where('user_id', '=', $user_id);

    if ($dtr->count() > 0) { 
        $dtr = $dtr->first();
    } else if ($dtr->count() == 0 && $record_id != false) {

        // If other user accessing someone's record id, will redirect
        return  Redirect::route('dashboard-shinra')
                ->with('failure', 'The hell are you accessing someone\'s record?');
    } else {
        // return something or make sure you do $dtr = $dtr->first() here
        $dtr = $dtr->first();
    }

    // if the dtr->count() is 0 or has a value it still send the data to view
    return  View::make('dashboard.shinra.dtr_entry')
            ->with('dtr', $dtr);
}

另外，您可以对Blade中的三元运算符使用更简单的语法。 而不是写作

{{ (isset($dtr->date)) ? $dtr->date : '' }}

你可以写

{{ $dtr->date or '' }}

Answer 2

问题是，当count为0并且您从不执行查询时，会出现边缘情况。

因此，首先我建议适当命名变量：

$query =  DailyTimeRecord::where('id', '=', $record_id)
        ->where('user_id', '=', $user_id);
...

$dtr = $query->first(); // or whatever
...
return View::make()->with('dtr', $dtr);

然后您将已经知道$dtr未设置，因此调试起来非常容易。

现在，您的代码将执行不必要的查询：

// 1st query
if ($dtr->count() > 0) { 
// 2nd query if passed
    $dtr = $dtr->first();

// 2nd query if didn't pass
} else if ($dtr->count() == 0 && $record_id != false) {

尽管您可以像这样简单地做到这一点：

$dtr = DailyTimeRecord::where('id', '=', $record_id)
        ->where('user_id', '=', $user_id)
        ->first();

if ( ! count($dtr))
{
  return ... // ERROR
}

return ... // correct row returned

Answer 3

对我来说，如果没有参数，则id设置为0。这意味着您在$ dtr中得到一个空集合，并将该空集合传递给视图。 比您尝试访问您认为的模型属性更重要的是实际上它是一个集合。

当没有参数传递给controllers操作时，根据数据库返回的内容，可能会出现这种情况。