orderByを使用するとエラーが出る

Question

orderByを使用すると以下のようなエラーメッセージが出ます。

SQLSTATE[42S22]: Column not found: 1054 Unknown column '' in 'order clause' (SQL: select * from `people` order by `` asc limit 5 offset 0)

以下は，orderByを使用しているコードです。
HelloController.php

    public function index(Request $request){
        $sort = $request->sort;
        $items = Person::orderBy($sort, 'asc')
            ->paginate(5);
        $param = ['items' => $items, 'sort' => $sort];
        return view('hello.index', $param);
    }

viewに関するコードはindex.blade.phpに表記しています。

@extends('layouts.helloapp')

<style>
    .pagination { font-size:10pt; }
    .pagination li { display:inline-block }
    tr th a:link { color: white; }
    tr th a:visited { color: white; }
    tr th a:hover { color: white; }
    tr th a:active { color: white; }
</style>

@section('title', 'Index')

@section('menubar')
    @parent
    インデックスぺージ
@endsection

@section('content')
    <table>
        <tr>
            <th><a href="/hello?sort=name">name</a></th>
            <th><a href="/hello?sort=mail">mail</a></th>
            <th><a href="/hello?sort=age">age</a></th>
        </tr>
        @foreach ($items as $item)
            <tr>
                <td>{{$item->name}}</td>
                <td>{{$item->mail}}</td>
                <td>{{$item->age}}</td>
            </tr>
        @endforeach
    </table>
    {{ $items->appends(['sort' => $sort])->links() }}
@endsection

先ほど，下記のコードに問題があると思い調べたところ，$sortが空ということが分かりました。
ですが，現状そこからどうしたらよいかわかりません。ご助言いただけると幸いです。

        $sort = $request->sort;
        $items = Person::orderBy($sort, 'asc')
            ->paginate(5);

以上です。何卒よろしくお願い申し上げます。。

Answer 1

原因が調べても分からずという状態です。私は以前，peopleテーブル以外にboardsテーブルを作成して，リレーションを使用したことがあります。それが原因ということはあるのでしょうか？？

エラーメッセージを読む限り、無関係ですね。

SQLSTATE[42S22]: Column not found: 1054 Unknown column '' in 'order clause' (SQL: select * from people order by `` asc limit 5 offset 0)

書いてあることをしっかり読みましょう

Comments

[tanukitii]

ご回答ありがとうございます。
申し訳ございません，リレーションは関係なかったです。
調べているうちに問題が何かよくわからなくなっていました。
本文を修正いたします。

[terminator]

$sort = $request->sort;
ここがおかしいということは特定できているんでしょうか？
こちらのドキュメント読んでいますか？
https://readouble.com/laravel/5.7/ja/requests.html

$request->sort が「空」なんじゃなくてNullなんだと思いますが、Null のとき「初期値」設定することで回避できませんか？

[tanukitii]

ようやく理解できました。
$request->sortの中身はNullでした。Nullのときに初期値を設定するという発想が
なぜか出てきませんでした…。まだまだ勉強不足です。精進します。

そして，この度はご回答いただき誠にありがとうございます。

Answer 2

terminator さんがおっしゃっているのはこんな感じでしょうかね。

        if(!$request->sort) {
            $sort = 'id';
        } else {
            $sort = $request->sort;
        }
        $items = Person::orderBy($sort, 'asc')
            ->paginate(5);

コードの書き方の良し悪しは別として、orderBy($sort, 'asc')の$sortに何の値も入っていないと、MySQLのクエリエラーになるので、何もない場合は、かならずあるであろうtableのid カラムで並び替えをする。という事ですね。ご参考になれば幸いです。

Comments

[tanukitii]

ご回答いただきありがとうございます。

「コードの書き方の良しあしは別として」とありますが，
どの辺が気になったか，よろしければ教えてはいただけないでしょうか？？
私は現在，書籍で勉強しているのですが，その書籍のコードでも十分きれいだと感じています。
ですが，もし，もっと良いコードの書き方があるならばそちらのほうを参考にしたいのです。
何卒よろしくお願い申し上げます。

[sakamata]

すみません、主語がぬけていました。「コードの良し悪し」は私が例題で書いたコードの事を示しており、まあ…自虐です。慣れた人はifで私が書いた部分を三項演算子等を使って1行で書く方もいると思うので。
あまりお気になされませんよう。ひとまず解決したようで何よりです。