凝聚行代碼

Question

如果我有下面的代碼行下面有什麼辦法，我可以凝結成1或2行，並使用循環或類似的東西？有超過100個索引，所以它是大量的文字寫出來。

unrlzdPnl = 
(((Positions[0].GetProfitLoss(Closes[0][0], PerformanceUnit.Points))*200) 
+ ((Positions[1].GetProfitLoss(Closes[1][0], PerformanceUnit.Points))*200) 
+ ((Positions[2].GetProfitLoss(Closes[2][0], PerformanceUnit.Points))*200) 
+ ((Positions[3].GetProfitLoss(Closes[3][0], PerformanceUnit.Points))*200) 
+ ((Positions[4].GetProfitLoss(Closes[4][0], PerformanceUnit.Points))*200) 
+ ((Positions[5].GetProfitLoss(Closes[5][0], PerformanceUnit.Points))*200) 
+ ((Positions[6].GetProfitLoss(Closes[6][0], PerformanceUnit.Points))*200) 
); 

Answer 1

也許我不明白這一點，但我會說：

int unrlzPnl = 0; 
for (int i = 0; i < nb_indices; i++) { 
    unrlzdPnl += Positions[i].GetProfitLoss(Closes[i][0], 
              PerformanceUnit.Points); 
} 
unrlzPnl *= 200; 

編輯＃1：

不要忘記循環之前初始化unrlzPnl。如果你想看到在每次迭代發生的事情（假設你使用Java）：

int unrlzPnl = 0; 
for (int i = 0; i < nb_indices; i++) { 
    System.out.println("Iteration " + i + ":"); 
    System.out.println("unrlzdPnl (old): " + unrlzdPnl); 
    unrlzdPnl += Positions[i].GetProfitLoss(Closes[i][0], 
              PerformanceUnit.Points); 
    System.out.println("unrlzdPnl (new): " + unrlzdPnl + "\n"); 
} 
return unrlzPnl * 200; 
System.out.println("unrlzdPnl (final value): " + unrlzdPnl); 

---

編輯＃2：

好吧，如果你真的希望它儘可能的緊湊並避免每次調用函數時進行初始化：

int MyFunction(int nb_indices) { 
    if (1 > nb_indices) 
     return 0; 

    unrlzdPnl = Positions[0].GetProfitLoss(Closes[0][0], 
              PerformanceUnit.Points); 

    for (int i = 1; i < nb_indices; i++) 
     unrlzdPnl += Positions[i].GetProfitLoss(Closes[i][0], 
               PerformanceUnit.Points); 

    return unrlzPnl * 200; 
}