ループをもう一度繰り返しますか？

Question

int iterationMax = 1; 
double actualMax = 0; 
int A = 3; 

List<double> tmp = new List<double> { 400, 0, -300, 400, 600 ,300, 400, 
             1200, 900, 400, 1200, 1500}; 
List<double> listMax = new List<double>(); 
for (int i = 0; i < tmp.Count; i++) 
{ 
    if (iterationMax < A) // A == 3 
    { 
     if (tmp[i] > actualMax) 
     { 
      actualMax = tmp[i]; 
     } 
     iterationMax++; 
    } 
    else 
    { 
     listMax.Add(actualMax); 
     actualMax = 0; 
     iterationMax = 1; 
    } 
} 

Console.WriteLine("\nMaxs: "); 
foreach (double max in listMax) 
{ 
    Console.Write(max + ", "); 
} 

リストtmpは= { 400,0,-300|400,600,300|400,1200,900|400,1200,1500}を保持し、 プロデュース400, 600, 1200, 1200、 は400, 600, 1200, 1500であるべき。私はリストに1500を追加するためにelse文を入力する方法を知らない。

3つの要素ごとにmaxを取得したいだけです。

Answer 1

クイックフィックスは、次のようになります。他の人のよう

var A = 3; 

int iterationMax = 0; 
double actualMax = 0; 

List<double> tmp = new List<double> {400,0,-300,400,600,300,400,1200,900,400,1200,1500}; 
List<double> listMax = new List<double>(); 
for (int i = 0; i < tmp.Count; i++) 
{ 
    if (iterationMax < A) // A == 3 
    { 
     if (tmp[i] > actualMax) 
     { 
      actualMax = tmp[i]; 
     } 
     iterationMax++; 
    } 

    if (iterationMax == A) 
    { 
     listMax.Add(actualMax); 
     actualMax = 0; 
     iterationMax = 0; 
    } 
} 

Console.WriteLine("\nMaxs: "); 
foreach (double max in listMax) 
{ 
    Console.Write(max + ", "); 
} 

0からiterationMaxを開始し、if (iterationMax == A)にそのelseを回す、と述べています。

Answer 2

コレクションの操作を行う必要があるときは、Linqを使用するのが何回もうまくいきます。

index/3にGroupByを使用してください。intであるため、以下の項目はそれぞれkeyとなります。次に、各グループについて、最大値を選択します。

var items = new int[] { 400, 0, -300, 400, 600, 300, 400, 1200, 900 }; 

var results = items.Select((item, index) => new { item, index }) 
        .GroupBy(item => item.index/3) 
        .Select(group => group.Max(item => item.item)); 
//400, 600, 1200 

Answer 3

初期化でiterationMaxを0に設定すると、問題が解決するはずです。

現在、if構造体は3つの要素のうち最初の2つのみをチェックします。 1500は要素＃3なので、チェックされません。あなたのコードのための

Answer 4

iterationMaxが3に達すると、tempの値で何もしないと、そのループは失われます。

for (int i = 0; i < tmp.Count; i++) 
{ 
    if (tmp[i] > actualMax) 
    { 
     actualMax = tmp[i]; 
    } 
    iterationMax++; 

    if (iterationMax > A) 
    { 
     listMax.Add(actualMax); 
     actualMax = 0; 
     iterationMax = 1; 
    } 
}