[C#] returnする場所が分からない

Question

前提・実現したいこと

C#で電文を出力するシステムを作っています。
初心者なりに書いては失敗してを繰り返し、上司に質問したりもしているのですが、今日に限ってその上司が休みで八方塞がりなので助けてください。

発生している問題

var response = new RealResponse<HogeResponseBody>();
上記のreturn場所が分からない

該当のソースコード

        /// <summary>
        /// 実行処理
        /// </summary>
        /// <param name="request">リクエストデータ</param>
        /// <returns>RESTリクエスト</returns>
        [AutoTransaction]
        public RealResponse<HogeResponseBody> Execute(RealRequest<HogeRequestBody> request)
        {
            var response = new RealResponse<HogeResponseBody>();

            // ヘッダー情報よりコード設定
            request.Message.Huga = request.Huga;

            // ユーザチェック
            if (_userCheckService.Validate(request.Message, out var userCheck))
            {
                // 更新処理
                // 区分　＝　"0" の場合のみ
                userCheck = _entryFacade.EntryRequest(request.Message, userCheck);
            }
            else
            {
                // 業務ロジックでエラーのケース
                response.WithAppError();
            }

            // 履歴出力
            userCheck = _entryService.CreateTable(request.Message, userCheck);

            return new RealResponse<HogeResponseBody>
            {
                Message = userCheck.ToHogeResponseBody(request.Message)
            };
        }

試したこと

もちろんelse{}の下に書くと履歴出力の処理には到達できないので、最後のnewをreturn responseに変更してみたり、最初のnewの下に{}を配置し、その中にすべて入れてみたり、と試してみました。

Comments

[m.ts10806]

１日休まれたくらいでどうにもならなくなるようなものを作ってるんでしょうか？
その「システム」が業務上のものであれば、結局仕様を握っているのはリーダーでしょうし、世界中からアクセス可能な場所にコードを公開して問題ないのでしょうか。

[Zuishin]

総当たりコーディングはやめましょう。書いては失敗しを繰り返すのではなく、まずロジックを整理するところから。

[dodox86]

> 上記のreturn場所が分からない

？？？？質問自体が謎なのですが、execute()メソッドの冒頭でnewしたRealResponseインスタンスを使うとすると、メソッド末尾の以下のコードとどう折り合いをつけたらよいのか、というような質問でしょうか。

> return new RealResponse<HogeResponseBody>
> {
> Message = userCheck.ToAHR090ResponseBody(request.Message)
> };

※質問者さんらが独自に作っている電文システムとやらの各クラス間のインターフェースに依存するでしょうから、第三者の回答者には正確な回答はできない気がするのですが。

[new5450]

はい、その通りで、どう折り合いをつければ良いのかが分かりません…

[new5450]

> m.ts10806様

業務ロジックなどと書いてはいますが、こちらはあくまでも社員同士で試しに作っているようなものなので業務に直接影響するものではありません、心配していただきありがとうございます。

[Zuishin]

折り合いもなにも、new するのをやめればいいだけのように思います。むしろなぜ最後に new しているのか意味不明なので、適当にキーボードを叩くのではなく考えてから書きましょう。

[dodox86]

RealResponseやHogeResponseBodyは質問者さんが取り組んでいるプロジェクトの独自クラスと推察します。そうであればそのクラスのインスタンスの適切な使い方、return先の呼び出し側が求めるインスタンスの完成形は第三者には正確には分かりません。この質問は言うなればあなたの家に他の者が知らない独自ルールがあり、ある局面で「どう守ったらよいの？」と唐突に赤の他人に聞いているようなことなのですが、その違和感が伝わりますでしょうか。具体的なコードでの回答は、「こんなのでいいんじゃない？（知らんけど）」くらいしか提示できないと思います。そんな回答にある修正方法ではあとでお母さんに怒られるかもしれません。結局はZuishinさんのご指摘のように、そのシステムを知っている質問者さんが「適当にキーボードを叩くのではなく考えてから書く」しかないはずです。

[new5450]

>dodox86様

とても図々しいことをお聞きしたこと、理解しました…。
大変失礼しました、ありがとうございます。

[new5450]

>Zuishin様

これ別の方から引き継いだものなので最後のreturnは必須だと思ってましたが、たしかに変なことしてました、ご指摘ありがとうございます。

[m.ts10806]

コードから考えるのではなく仕様書から考えては。
設計書すらないのでしたらちょっと。

[Zuishin]

引き継いだ時にはコードを読んで何をしているか処理を追ってから改修するのをおすすめします。userCheck も上書きしていますが、これもしていいことのようには思えません。

[m.ts10806]

というか
>社員同士で試しに作っているようなものなので業務に直接影響するものではありません

なおさら、上司待てば良いのでは。
引き継いだもの…等々、他人には目的やスケジュール感が不明です。

「悩んでる時間を初心者脱却に充てよう」くらいのアドバイスならできますが。