サクセスメッセージで動的に表示内容を変えたいです。

前提・実現したいこと

サクセスメッセージで動的に表示内容を変えたいです。
一つ目のif文でチェックして空白なら次へ、
空白じゃなかったら"$message.が更新されました。"としたいです。

二つ目も同様にチェックし、
"$message.が更新されました。"としたいです。

両方更新された場合には、間に”と”を入れて両方更新されたことを伝える様にしたいです。

該当のソースコード

    public function update(Request $request, Account $account)
    {
        $request->validate([
            'user_account_id' => ['integer'],
            'post_account_id' => ['integer']
        ]);

        try {
            $message = '';

            if ($request->get('user_account_id')) {
                $account->user_account_id = $request->get('user_account_id');

            }

            if ($request->get('post_account_id')) {
                $account->post_account_id = $request->get('post_account_id');

            }

            $account->save();

            return redirect()->back()->with('success', $message.'が更新されました。');
        } catch (\Exception $e) {
            logs()->error($e);
            return redirect()->back()->with('failure', $e->getMessage());
        }
    }

試したこと

ここに問題に対して試したことを記載してください。

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

laravel8

行動規範の内容に同意します

回答1件

あまりイケてる書き方ではないかもしれないですが、こちらでどうですか？

php
1try {
2    // 更新されたidのメッセージを配列に入れるために準備
3    $message_array = [];
4
5    if ($request->get('user_account_id')) {
6        $account->user_account_id = $request->get('user_account_id');
7        $message_array[] = 'user_account_id_message';
8    }
9
10    if ($request->get('post_account_id')) {
11        $account->post_account_id = $request->get('post_account_id');
12        $message_array[] = 'post_account_id_message';
13    }
14
15    $account->save();
16
17    if (count($message_array) === 1) {
18        // 配列の要素が一つ（片方しか更新されていない）の場合は配列の1つ目の要素を表示
19        $message = $message_array[0];
20    } else {
21        // 配列の要素が二つ以上（両方更新されている）の場合は各要素を"と"で結合する
22        $message = implode('と', $message_array);
23    }
24
25    return redirect()->back()->with('success', $message.'が更新されました。');
26}