單獨的功能沒有重複的代碼

Question

我目前正在使用兩個不同的函數來遍歷數字列表以查找總和和特定值的程序。這裏是我已經實現

class Program 
{ 
    static int i, sum; 
    static List<int> store = new List<int>(); 

    static void Main(string[] args) 
    { 


     for (i = 0; i < 100; i++) 
     { 
      store.Add(i); 
     } 

     i = 0; 
     TraverseList(); 
     Console.ReadLine(); 
    } 

    static void TraverseList() 
    { 
     while (i < store.Count) 
     { 
      FindValue(); 
      FindSum(); 
      i++; 
     } 

     Console.WriteLine("The sum is {0}", sum); 
    } 

    static void FindValue() 
    {   
     if (store[i] == 40) 
     { 
      Console.WriteLine("Value is 40"); 
     }     
    } 

    static void FindSum() 
    { 
     sum = sum + store[i]; 
    } 
} 

我想FindSum和FindValue分成兩個不同的功能，而不是叫他們TraverseList的代碼。有沒有做這件事，而在重複這兩個函數列表遍歷的通用代碼，因爲我在這裏做

class Program 
{ 
    static int i, sum; 
    static List<int> store = new List<int>(); 

    static void Main(string[] args) 
    { 


     for (i = 0; i < 100; i++) 
     { 
      store.Add(i); 
     } 

     i = 0; 
     FindValue(); 
     i = 0; 
     FindSum(); 

     Console.ReadLine(); 
    } 

    static void FindValue() 
    { 
     while (i < store.Count) 
     { 
      if (store[i] == 40) 
      { 
       Console.WriteLine("Value is 40"); 
      } 
      i++; 
     } 
    } 

    static void FindSum() 
    { 
     while (i < store.Count) 
     { 
      sum = sum + store[i]; 
      i++; 
     } 

     Console.WriteLine("The sum is {0}", sum); 
    } 
} 

Answer 1

要查找的一系列數字，您可以使用簡單的LINQ功能的總和的任何其他方式：

List<int> numbers = new List<int>(); 
int sum = numbers.Sum(); 

我不確定你的意思是找到一個值。如果你想檢查是否在一系列數字中的一個等於一定值時，你可以使用LINQ功能Any：

int myValue = 40; 
bool hasMyValue = numbers.Any(i => i == myValue); 

它使用一個lambda表達式，集合中執行函數，並將每個元素到功能。該函數返回true或false來表示該元素與Any測試匹配。

相反，如果你想檢查序列中的許多號如何搭配一定值時，你可以改用Count功能，像這樣：

int numberOfMatches = numbers.Count(i => i == myValue); 

Answer 2

第一件事 - 我會用foreach代替while ，關於重複的代碼（假設你沒有使用Linq） - 我認爲它很好

Linq可以簡化你的代碼的味道：

var FindSum = store.Sum(); 
var FindValue = store.FindAll(x => x == 40); 

Answer 3

我不能強調i和sum作爲類成員有多糟。特別是i。它會讓你的代碼非常脆弱，而且很難處理。嘗試使每種方法儘可能與代碼的其餘部分隔離開來。

嘗試這樣代替：

static void Main(string[] args) 
{ 
    List<int> store = new List<int>(); 

    for(int i = 0; i < 100; i++) 
     store.Add(i); 

    FindValue(store); 
    FindSum(store); 

    Console.ReadLine(); 
} 

static void FindValue(List<int> list) 
{ 
    for(int i = 0; i < list.Count; i++) 
    { 
     if(list[i] == 40) 
      Console.WriteLine("Value is 40"); 
    } 
} 

static void FindSum(List<int> list) 
{ 
    int sum = 0; 
    for(int i = 0; i < list.Count; i++) 
     sum += list[i]; 

    Console.WriteLine("The sum is {0}", sum); 
} 

這是完全正常（正常）複製的循環，它只是一個單行。你也可以在這裏使用foreach。

另外，無視每個人告訴你使用LINQ。你顯然是編程新手，你應該先學習基礎知識。