このコードは不明ですが、コードの臭いがあります！

Question

このコードは私に迷惑をかけています

別のオブジェクトEventDistanceを作成しましたが、おそらくそれは必要ではないと思われます。距離はイベントの部分クラスに追加したプロパティです。基本的には、すべてのイベントを返信し、距離順に並べ替える必要があります。

私はforeachの余分なラウンドがおそらく必要ではないと考えています。

public IQueryable<Event> FindByLocation(float latitude, float longitude) 
     { 

      var eventsList = from ev in GetAllEvents() 
          join i in db.NearestEvents(latitude, longitude) 
          on ev.ID equals i.ID 
          select new EventDistance() { TheEvent = ev, Distance = i.Distance }; 


      foreach (var item in eventsList) 
      { 
       item.TheEvent.Distance = item.Distance; 
      } 
      return eventsList.OrderBy(e => e.Distance).Select(e => e.TheEvent); 
     } 

Answer 1

私に見えるものから、EventDistanceクラスは必要ありません。単にそれを省略して、あなたのLINQの式の中で任意の型を指定しないことによって、匿名型を使用します。

public IQueryable<Event> FindByLocation(float latitude, float longitude) 
    { 

     var eventsList = from ev in GetAllEvents() 
         join i in db.NearestEvents(latitude, longitude) 
         on ev.ID equals i.ID 
         select new { TheEvent = ev, Distance = i.Distance }; 


     foreach (var item in eventsList) 
     { 
      item.TheEvent.Distance = item.Distance; 
     } 
     return eventsList.OrderBy(e => e.Distance).Select(e => e.TheEvent); 
    } 

Answer 2

私はLINQの構文で無知だけど、それは本当に最初に距離値を移入することは可能であるべきselect。何かのように

var eventsList = from ev in GetAllEvents() 
       join i in db.NearestEvents(latitude, longitude) 
       on ev.ID equals i.ID 
       orderby i.Distance 
       select (ev => { ev.Distance = i.Distance; return ev; }); 

？または、

var eventsList = GetAllEvents() 
       .Join(db.NearestEvents(latitude, longitude), 
         ev => ev.ID, 
         i => i.ID, 
         (ev, i) => { ev.Distance = i.Distance; return ev; }) 
       .OrderBy(ev => ev.Distance); 

---

N.B. :-)適切に書かれましたこれ以降のqstarinコメントは、SQLとして評価されるORMオブジェクトではなく、この時点でオブジェクトに作用している場合にのみ可能です。それは言った、あなたはおそらくまだ

1. 初期にORDERBYを行うことができますにオブジェクトを結合する単一の選択を使用し
2. （私はこれを行うための最善の方法を忘れてしまった）メモリにオブジェクトを強制
3. をフェッチEventオブジェクト上記のようにではなく、ループ

Answer 3

あなたがのIQueryableとして、それを維持したい場合は、あなたがこれを行うことができます。

 public IQueryable<Event> FindByLocation(float latitude, float longitude) 
     { 

      return from ev in GetAllEvents() 
        join i in db.NearestEvents(latitude, longitude) 
        on ev.ID equals i.ID 
        order by i.Distance 
        select ev; 
     } 

GetAllEvents（）メソッドがIQueryableを返す限り、そうです。

編集：私は今実際に以下のRupの提案を試しました。このメソッドでdb呼び出しを呼び出さずにIQueryableを返すことができるはずです。このように：

 public IQueryable<EventDistance> FindByLocation(float latitude, float longitude) 
     { 
       return from ev in GetAllEvents() 
        join i in db.NearestEvents(latitude, longitude) 
        on ev.ID equals i.ID 
        orderby i.Distance 
        select new EventDistance 
        { 
         Event = ev, 
         Distance = i.Distance 
        }; 
     } 

EventDistanceクラスはマッピングする必要はありません。

Answer 4

このようなことはできますか？