Call to a member function save() on nullの解決

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
PHP(CakePHP)でマスタ画面を作っています。
マスタの新規登録時にに以下のエラーメッセージが発生しました。

前提として、MtShiftPatternとMtShiftPatternsの二つがあり、モデル名やテーブル名などで使い分けないといけないと決まっているみたいです。
初心者のためわかる範囲での質問であることをお許しください。

発生している問題・エラーメッセージ

Fatal Error
Error: Call to a member function save() on null
File: C:\xampp\htdocs\car-maintenance\app\Controller\MtShiftPatternsController.php
Line: 59

Notice: If you want to customize this error message, create app\View\Errors\fatal_error.ctp

該当のソースコード

<?php
App::uses('AppController', 'Controller');

class MtShiftPatternsController extends AppController {

    public function index() {

    }


    public function search() {

        $this->autoRender = false;
	$this->response->type('json');

        if ($this->request->is('get')) {

            $shift = $this->shift_patterns();

            return json_encode(['success' => 'success', 'data' => $shift ]);
        }


        $this->response->statusCode(403);
        return json_encode(['error' => ['message' => '仕様で定義されていないMETHODです。']]);
    }

    public function detail($id = 0) {
        $this->autoRender = false;
	$this->response->type('json');

        // $idの確認
        if (! ctype_digit($id) && $id !== 0) {
            $this->response->statusCode(403);
            return json_encode(['error' => ['message' => 'ID以外の値が渡されました。'. $id]]);
        }


        if ($this->request->is('get')) {

            $shift = $this->MtShiftPattern->find('first', ['conditions' => ['id' => $id]]);

            if (empty($shift)) {
                $this->response->statusCode(404);
                return json_encode(['error' => ['message' => '未登録のIDが指定されました。'. $id]]);
            } else {
                return json_encode(['success' => 'success', 'data' => $shift]);
            }
        } elseif ($this->request->is('post')) {

            if ($id > 0) {
                $this->request->data['MtShiftPattern']['id'] = $id;
                $this->MtShiftPattern->id = $id;
                $data['id'] = $id;
            } else {
                $this->MtShiftPattern->create();
            }

            if ($this->MtShiftPatterns->save($this->data)) {
                if ($id <= 0) {
                    $id = $this->MtShiftPattern->id;
                }

                return json_encode(['success' => 'success', 'data' => $this->request->data]);
            } else {
                $this->response->statusCode(500);
                return json_encode(['error' => ['message' => '保存に失敗しました。']]);
            }
        } elseif ($this->request->is('delete')) {
            // 削除

            if ($id > 0) {
                /* if ($this->User->find('count', ['conditions' => ['MtOffice_id' => $id]]) > 0) {
                 *     $this->response->statusCode(403);
                 *     return json_encode(['error' => ['message' => '使用中の権限です。未使用にしてから削除してください。']]);
                 * } */

                $this->MtShiftPattern->id = $id;
                if ($this->MtShiftPattern->delete($id, true)) {
                    return json_encode(['success' => 'success', 'data' => []]);
                } else {
                    $this->response->statusCode(500);
                    return json_encode(['error' => ['message' => '削除に失敗しました。']]);
                }

            } else {
                $this->response->statusCode(403);
                return json_encode(['error' => ['message' => 'IDを送信してください。']]);
            }
        }

        $this->response->statusCode(403);
        return json_encode(['error' => ['message' => '仕様で定義されていないMETHODです。']]);
    }


    private function shift_patterns() {
        return $this->MtShiftPattern->find('all', ['order' => 'order_no']);
    }
}

試したこと

定義されていない変数名を使おうとしているからエラーが出ていると思ったのですが、どの変数がそれにあたるのかがわかりませんでした。

補足情報（FW/ツールのバージョンなど）

Answer 1

定義されていない変数名を使おうとしているからエラーが出ていると思ったのですが、どの変数がそれにあたるのかがわかりませんでした。

エラーの解釈は違いますが「どの変数が」かはエラーに書いてあります。

Error: Call to a member function save() on null

File: C:\xampp\htdocs\car-maintenance\app\Controller\MtShiftPatternsController.php
Line: 59

CakePHP特有のものでもありません。

大方、下記が混ざってるのが原因でしょう。
MtShiftPattern
MtShiftPatterns

どちらかしか存在しないのか、両方存在するのかで対応は違いますが、まずはデバッグしてみては。

エラーメッセージの読み方と対処, 検索や質問の原則

Comments

[kuuhaku4262]

どちらも存在しています。
とりあえずデバックしてみます。

[m.ts10806]

>どちらも存在しています。
質問内容に提示されたコードからはそれが読み取れませんでした。
読み取れるように質問に追記願います

[kuuhaku4262]

もう一方のコメントに回答させていただきました。
そちらをご覧いただくようお願いいたします。

[m.ts10806]

流れもあるのでこちらに書いていただくべきと思います

[kuuhaku4262]

テーブル名やモデル名など単数形ではいけないや複数形ではなくてはいけないということが前提として決められていると先輩が言っていました。
何分、初心者のため理解している部分が少なくわかる範囲でのコメントとなることをお許しください。

[m.ts10806]

いえ、きちんと書いてもらえればそこも考慮できるので。
特に社内ルールやプロジェクト固有ルールがあったりするとそこは他者がわからない領域となります。
CakePHP自身のネーミングルールに則っていればあとは開発者次第なので、ある程度説明があった方が助かります

[kuuhaku4262]

アドバイスありがとうございます。
社内のルールなどがまだわかっていない状態なので、確認してみます。

[m.ts10806]

そこはあまり外に出してはいけない部分かもしれないので質問する際には注意が必要です。

[kuuhaku4262]

了解しました。

Answer 2

nullなのでsave関数は実行できまへん、というエラーなので、心当たりのある（というよりエラーの出た）行に出てくる変数の値をチェックしてみれば。

Comments

[kuuhaku4262]

試してみます。

Answer 3

59行目で「MtShiftPattern」と書かないといけないところを「MtShiftPatterns」と書いていたため、「s」を消したら治りました。

Comments

[m.ts10806]

私の回答の通りの結果ですがコメントにあった「両方ある」は結局どう言うことだったのでしょうか。
(これで自己解決はちょっと納得いかないですね)

[kuuhaku4262]

モデル名やテーブル名などでどちらを使うのか元から決められているようでした。
使い方をまだ理解できていなく、せっかくアドバイスをもらい解決できたにもかかわらず自己解決という形になってしまい大変嫌な思いをさせてしまいました。
申し訳ございませんでした。
一度未解決にして、ベストアンサーにさせていただきます。
プログラマー初心者のためこの場でたくさん質問すると思いますので、その時は少しでもアドバイスをしていただけると幸いです。
今後とも、よろしくお願いいたします。

[m.ts10806]

>モデル名やテーブル名などでどちらを使うのか元から決められているようでした。

ちょっとよくわかりませんが、前提や背景、経緯があるのでしたらそれはきちんと書かれないと誰にも正しく伝わりませんし考慮はできません。
回答する限りはそれなりの知見を持ってはいますが質問者さんの傍らで状況を見てるわけでもエスパーでもないので、持ちうる情報は基本的に全て前もって出すようにしてくださいね

[kuuhaku4262]

大変失礼いたしました。
次回から、持ちうる情報を全て記入するようにいたします。
アドバイス、ありがとうございました。

[m.ts10806]

質問は編集できますので、適宜対応するようにしてくださいね

[kuuhaku4262]

編集致しました。
確認していただいて、もし何か気になるようなことがありましたらアドバイスお願いします。