ループせずに何千もの個別のメールを送信する

Question

私は、人々が販売するものを置くサイトを持っています。 毎月、何か売り物を持っているすべてのユーザーは、Windowsサービスから、そのアイテムが売れたかどうかを尋ねる電子メールを送信し、クリックするカスタムリンクを与えます（これは、販売されているサイトに残るアイテム）。

まず、すべての未販売アイテムと関連ユーザーのメールを取得するためにクエリを実行する必要があります。 現在、私はこれらのすべてをループしており、各自のカスタムメールを生成し、個々のメールとして送信しています。

foreach (Item unsoldItem in unsoldItemsCollection) 
     { 
      //generate email 

      string email = GenerateUnsoldEmail(itemName, itemPrice); 
      Utils.Sendemail(unsoldItem .UserEmail, "no-reply@website.com", "Unsold Item", email); 

     } 

（これは擬似コードの一種であるが、これは私がやっているほとんどである）

私の問題は、（それが離陸する場合:)何百万人も多分、）おそらく数千人があるかもしれないですがそのループ内のアイテムはすべて電子メールを必要としますが、これは問題を引き起こすと思います。

これ以外の方法は何ですか？

Bexの

Answer 1

あなたは、コレクション内のすべてのアイテムのために何かをしたい場合は、あなたが（再帰のような、または同等のもの）ループのいくつかの種類を使用する必要があります。その周りに方法はありません。

潜在的な問題については心配する必要はありません。このコードがパフォーマンス上の問題と思われる場合は、そのコードを測定し、実際の速度と処理する電子メールの数を確認してください。

すべてのメールを送信するには数時間かかることがありますか？それは実際に問題ですか？どうして？

また、あまりにも多くのメモリが必要になると心配です。一度にすべてのアイテムをメモリにまとめることができない場合は、一度に1000を取得して処理してから別の1000を取得してください。

Answer 2

現在の電子メールが送信されるまでプログラムが停止されないように、私は非同期的にこれを行うだろう。

すべてのメールが異なるため、差し込み印刷はできません。電子メールの本文にユーザーが強制的にログインするような種類のランディングページがある場合を除いて、ユーザーのすべてのアイテムがリストされている特定のページにユーザーをリダイレクトして、尋ねられない限り既に販売されているものを取り除く。その場合、数千ではなく電子メールを送信することができます。

SmtpClient has an async version of the Send method

Answer 3

Windowsサービスを作成して送信しますすべてのあなたの電子メールを1日1回。

さらに単純なコンソールユーティリティを作成して、それを実行してスケジュールされたタスクとして実行することもできます。