laravel のトランザクションの成功メッセージと一緒にデータをDBに登録したことでできたIDも一緒に送りたい

Question

前提・実現したいこと

Laravelで受け取ったリクエストの内容をDBに登録するアプリケーションを作っています
その成功メッセージと一緒にDBに値を登録したことでできたIDを返したいです

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

{
  "message": {
    "headers": {},
    "original": {
      "error": "false"
    },
    "exception": null
  }
}

該当のソースコード

public function post(Request $request){
     try{
         $r_response = DB::transaction(function() use($request) {
             //リクエストで送られてきたものの登録処理
             $user_id = $request->user_id;
             global $user_id;

             //テーブルから対象IDを取得する処理
             global $goods_id;

         });

        $r_response['message'] = response()->json([
        'result' => 'success register',
        'user_id' => $user_id,
        'goods_id' => $goods_id
        ]);
            return $r_response;

     }catch (\Throwable $exception) {
        $r_response['message'] = response()->json([
            'error' => 'false'
        ]);
        return $r_response;
    }
}

試したこと

public function post(Request $request){
の直下に
$user_id = [];
$goods_id = [];と書き足しましたがそうすると

  "message": {
    "headers": {},
    "original": {
      "message": "success register",
       "user_id": [],
       "goods_id ": []
    },

となり、入れたい場所が空になっていました

また、別のやり方としてはIDを取得した直下に
$this->Session->write('セッション名', $変数);
とかき

$r_response['message'] = response()->json([
        'result' => 'success register',
        'user_id' => $user_id,
        'goods_id' => $goods_id
        ]);

の直前で
$user_id = $this->Session->read('userIdSession');
としましたが値が取れないようでエラーメッセージが帰ってきてしまいます

ちなみに値をセッションに格納するやり方ですと、そもそも処理がとまりDBにインサートできませんがその前にただ、グローバル変数に入れて一緒に送ろうとする書き方の場合DBへの登録はできていました

idを送ろうとしないで成功メッセージだけの場合はエラーメッセージのほうが来ず、成功メッセージがちゃんと来ます

Answer 1

これが果たして適切なのかどうかは分かりませんが、今回やりたかったことは下記のようなコードで実現できました

まずそもそもuser_idは外部からのリクエストで取得しているのでトランザクション終了場所で$user_id = $request->user_id;のコードを記入することで取得可能でした

$goods_idに関してはグローバルに指定せずとも下記のようにして取得できました。
このコードで問題ないかは若干不明ですが・・・・。

public function post(Request $request){
     try{
         $r_response = DB::transaction(function() use($request) {

            $goods_id　＝　テーブルから対象IDを取得する処理

　　　　　　　return $goods_id;

         });

　　　　　$user_id = $request->user_id;
         $goods_id = $r_response;

        $r_response['message'] = response()->json([
        'result' => 'success register',
        'user_id' => $user_id,
        'goods_id' => $goods_id
        ]);
            return $r_response;

     }catch (\Throwable $exception) {
        $r_response['message'] = response()->json([
            'error' => 'false'
        ]);
        return $r_response;
    }
}

Answer 2

DB::transaction内での変数代入をその外でも利用したい場合は、一例として引数に利用しているクロージャに対して、useで変数を追加で渡してみて下さい。

$user_id = null;
$goods_id = null;
$r_response = DB::transaction(function() use($request,$user_id,$goods_id) {
             //リクエストで送られてきたものの登録処理
             $user_id = $request->user_id;

             //テーブルから対象IDを取得する処理
             $goods_id = XXXXX;

         });

変数へのglobalキーワードや、同一リクエスト処理で値を受け渡すためだけのセッション利用等は、必要の無い箇所で使うとコードが複雑になったり、不具合の元になります。
一度PHPの基本を再整理してみてください。

Comments

[lkjgffh]

ご指摘くださいましてありがとうございます。
続けざまに質問してしまい申し訳ないのですが、上記のように設定した変数はどのように参照すればよいのでしょうか？

ご指摘いただいたとおり書き換えた後、トランザクション終了タグの後ろで$goods_id= $r_response($goods_id);
というようにすれば値を参照できると思ったのですが取得できませんでした。グローバル変数の取得方法上記のような形式ではないのでしょうか？

[aro10]

DB::transaction内では省略されている何らかのデータベース処理があるかと思いますが、グローバル変数の取得は必要ありますか？
```
#例
$user_id = $request->user_id;
$goods_id = Good::first();
```

[lkjgffh]

このアプリケーションはスマートフォンと連携しているアプリで、スマートフォンから送った値がどういったものか（登録されたgoods詳細画面）へ移行するのに$goods_idを使いたいのでグローバル化して取得したいと思ったしだいです。

[lkjgffh]

aro10様のご指摘どおりトランザクション内で上記のような操作をしています

[lkjgffh]

すみません、希望している操作ができました。
このたびはご助言くださいましてまことにありがとうございました