「まだ行われていない場合に何かを行う」という用語（または「パターン」？）[クローズ]

かなり基本的に聞こえますが、最近、同僚に、呼び出されたメソッドstartHttpServerがまだ実行されていない場合にサーバーを起動するだけなので、理解するには複雑すぎると言われました。「真剣に？私はこれを何十年も続けてきました。これはプログラミングの一般的なパターンです」と答えたとき、私はトラブルに陥ります。私が認めようと思うよりも頻繁に、彼は、プログラミングコミュニティ全体が彼の視点の背後にあることを示す文書化された証拠とともに戻ってきて、私はひどい気持ちになります。

質問：必要なアクションが既に有効になっている場合、ノーオペレーションであるメソッドの概念の背後にある文書化された設計パターンはありますか？または、パターンでない場合、名前もありますか？そうでない場合、この方法でメソッドを記述することを検討するのが複雑すぎると考える理由はありますか？

NickWilliams がすでに言ったように：OPが記述する概念は、べき等（Idempotency）と呼ばれます。特に高レベルのAPIでは、実際に一般的な方法です。

ただし、関数の名前を変更します。

またはをstartHttpServer呼び出す代わりに。makeSureHttpServerIsRunningensureHttpServerIsRunning

関数が呼び出されるとstartHttpServer、読者はHTTPサーバーを起動することを期待します。連続して10回呼び出されると、10台のサーバーが実行されます。ほとんどの場合、関数はそれを行いません。さらに、「start」の付いた名前は、1つのサーバーのみを実行する場合、関数が既に呼び出されているかどうかを追跡する必要があることを示しています。

関数が呼び出されるとき、makeSureHttpServerIsRunningHTTPサーバーが実行されていることを確認するために必要なことを行うと思います。ほとんどの場合、既に実行されているかどうかを確認し、そうでない場合は開始します。また、この関数は、サーバーが実際に実行されていることを確認すると想定しています（サーバーの起動には、まだ完全に実行されていない時間がかかる場合があります）。

名前をに変更しEnsureServerRunningます。

完全に明確であり、再起動を暗示することなく実行されていることを確認します（実行されていない場合）。

（代替：StartServerIfNotRunning？）

実際には設計パターンではありませんが、私はあなたのメソッドをべき等と呼びます。通常、この用語はリモートコールを指すために使用されますが、説明はあなたがしていることと一致するようです。

べき等メソッド。メソッドは、（エラーまたは有効期限の問題を除いて）N> 0の同一リクエストの副作用が単一リクエストの場合と同じであるという点で、「べき等性」のプロパティを持つこともできます。（W3.orgから）

ここでのサーバー側の効果は、メソッドが呼び出されるとhttpサーバーが開始されることです。これを行うメソッドに問題はありません。

デザインパターンが必要な場合は、httpServerを初期化時に開始されるシングルトンとして公開できると思います。

この実装一つとしてツールを、startHttpServerあなたはそれが最も簡単な滑らかで作ろうとしなければならないシームレスに使用します...

関数のロジック

技術的には、分割することによりstartHttpServer、2つの'関数にsのロジックをし、それらを呼び出す別に、あなたがされないものをすべて移動 startHttpServerの冪等を、あなたが何をするかである第三の機能（の両方のロジックをラップしない限り、さらに...代わりに、両方の機能を呼び出すコードにstartHttpServerそもそも）、これにより、DRYされていないコードを記述し、を呼び出す必要のあるすべての場所で指数関数的に複製しますstartHttpServer。要するに、関数自体startHttpServer を呼び出す必要がありisHttpServerRunningます。

だから私のポイントは：

• isHttpServerRunningとにかくこれは独立して必要になる可能性があるため、関数を実装します...
• それに応じて、次のアクションを定義するためstartHttpServerに使用isHttpServerRunningするように実装します...

それでも、startHttpServerこの関数のユーザーが必要とする可能性のある任意の値を返すことができます。例えば：

• 0 =>サーバーの起動失敗
• 1 =>サーバー起動成功
• 2 =>サーバーはすでに開始されています

関数の命名

まず、ユーザーの主な目標は何ですか？するHTTPサーバを起動し、右？

基本的に、すでに開始されているもの、AKAを開始しようとすることで問題はありません1*1=1。それで、少なくとも私にとって、それを「ensureHttpServerIsRunning」と呼ぶことはそれほど必要ではないように思えます。関数の名前がどれだけ長く、自然で記憶に残るかをもっと気にしたいと思います。

フードの下の関数の詳細な作業方法を知りたい場合は、そのためのドキュメントまたはコードソースがあります。ライブラリ/フレームワーク/ API /などの他の関数のように...

あなたはそれを複数回書いている間に一度あなたは関数を学びます...

とにかく、私startHttpServerはに比べて短く、シンプルで、より明示的であることに固執しensureHttpServerIsRunningます。

私はあなたの同僚がそれstartHttpServerをやりすぎていることを意味すると思います：

• サーバーが既に実行されているかどうかを確認し、
• 必要に応じてサーバーを起動します。

これらは、コードの2つの無関係な部分です。たとえば、デスクトップアプリケーションが起動時にまだ実行されていないことを確認する必要がある場合、同様の状況が存在します。アプリインスタンスを処理するコードの一部（たとえば、ミューテックスを使用）と、アプリケーションメッセージループを開始するコードがあります。

つまり、1つではなく、少なくとも1つの2つのメソッドが必要です。

• isHttpServerRunning: boolean
• startHttpServer

アプリケーションのエントリポイントは最初のメソッドを呼び出し、戻り値がの場合は2番目のメソッドを呼び出しますfalse。現在、すべてのメソッドは1つのことを実行しており、理解しやすいものです。

^{¹サーバーが既に実行されているかどうかを知るために必要なロジックが複雑すぎる場合、複数のメソッドにさらに分離する必要があります。}

言語を指定しないので、JavaScriptの多くのライブラリには「1回」関数（たとえば、アンダースコア）があります。そのため、それが馴染みのある場合は、「1回」パターンと呼び、メソッドの名前を変更する可能性があります。

私自身、Javaから来たもので、「キャッシング」または「遅延評価」という用語が思い浮かびます。「べき等」は技術的に正確であり、特に良い選択です。より機能的な背景がある場合。

私は好むstartHttpServerIfNotIsRunning。

このように、条件はメソッド名にすでに明記されています。EnsureあるいはmakeSure、それは技術的な表現ではないとして、私には少しあいまいなようです。何が起こるのか正確にはわからないようです。

あなたの同僚があなたに言ったことは、あなたはその方法を書いているビジネスがないということです。それはすでに何度も書かれており、あなたが書こうとするよりもよく書かれています。例：http : //docs.ansible.com/ansible/latest/systemd_module.html https://docs.saltstack.com/en/latest/ref/states/all/salt.states.service.html

アーキテクチャの観点から見ると、Webサーバーを管理するコードの任意のビットが悪夢のようなものです。サービスの管理があなたのコードが独占的に行うことでない限り。しかし、私はあなたがmonit（またはkubernetesまたは...）を書いていなかったと推測しています。