ループ全体

Question

でprevValue！= currValue事は、我々がリストを持っている{、、、B、B、C、C}

我々はリストをループにしたいとのいくつかの種類を作ると言うなら項目値が変更されたときに変更します。たとえば、

prevEmployer = String.empty; 
foreach(Person p in PersonList){ 
    if(p.Employer != prevEmployer){ 
    doSomething(); 
    prevEmployer = p.Employer; 
    } 
    ... more code 
} 

これには代替手段がありますか？それはちょうど私にクールジーに見えます。

編集：コードを手元の問題にもっと現実的にしました。

Answer 1

これは実際には、残りのコードで何をしようとしているかによって異なります。 doSomething（）がリスト要素が変更されたという事実に基づいていくつかの状態を設定するだけで、リスト内の各要素に対して操作を実行している場合、@Marc Gravellの応答は正しいです。リスト（または別の目的のために個々の要素を必要とする）、あなたのメソッドは完全に妥当と思われる。

Answer 2

個別の値が必要ですか？すなわち、{a、a、b、b、a、c、c、a}が存在するだろうか？ない場合は、LINQを使用することができます。あなたの更新日時

foreach(string s in theList.Distinct()) { 
    doSomething(); // with s 
} 

---

を。おそらくDistinctByのようなものを使用します。

foreach(var item in data.DistinctBy(x=>x.Foo)) { 
    Console.WriteLine(item.Bar); 
} 

public static IEnumerable<TSource> DistinctBy<TSource,TValue>(
     this IEnumerable<TSource> source, Func<TSource,TValue> selector) { 
    var set = new HashSet<TValue>(); 
    foreach (var item in source) { 
     if (set.Add(selector(item))) { 
      yield return item; 
     } 
    } 
} 

Answer 3

は限りString.Emptyのはあなたのリストの要素になることはありませんし、あなたのリストがソートされているように私にはよさそうです。 theListが他のスレッドによって変更されていないことを確認したいかもしれません。リスト内の遷移に関する作業を処理するためのかなりストックな方法のように見えます。

Answer 4

私は以下の解決策をとることに決めました。これははるかにきれいですが、より多くのdbコールを生成します。

foreach(Employer employer in EmployerList){ 
    ProcessEmployees(employer); 
} 

ProcessEmployees(Employer employer){ 
    EmployeeList = GetEmployees(employer); 
    foreach(Employee employee in EmployeeList){ 
    doStuff... 
    } 
} 

ので、代わりに{A、B、B、C、C}、Iは、処理されます{、A}、{B、B}、{C、C}で始まるの