Более короткий способ написать следующую функцию в C#?

Question

У меня есть эта функция -

public int GetAvgResult() 
{ 
var weeklyvalues=GetWeeklyValues();//gets list of weekly values. 
if (weeklyvalues.Count == 0) 
       return 0; 
      return (weeklyvalues.Sum()/weeklyvalues.Count); 
} 

Есть более короткий путь, чтобы написать это с помощью ?: или, может быть что-то еще?

Answer 1

public double GetAvgResult() 
{ 
    // Assumes GetWeeklyValues() never returns null. 
    return GetWeeklyValues().DefaultIfEmpty().Average(); 
} 

Обратите внимание, что это возвращает double, который я предполагаю, что вы на самом деле покупке (в среднем кучу целых чисел логически не является целым числом). Вы можете привести его к int, если это необходимо, или если вы хотите придерживаться целой математики всего пути:

var seq = GetWeeklyValues().DefaultIfEmpty(); 
return seq.Sum()/seq.Count(); 

Answer 2

public int GetAvgResult() 
{ 
    var weeklyvalues = GetWeeklyValues(); 
    return (weeklyvalues.Count != 0) ? (weeklyvalues.Sum()/weeklyvalues.Count) : 0; 
} 

или:

public int GetAvgResult() 
{ 
    return GetWeeklyValues().DefaultIfEmpty().Average(); 
} 

Answer 3

public int GetAvgResult() 
{ 
    var weeklyvalues = GetWeeklyValues(); //gets list of weekly values. 
    return weeklyvalues.Count == 0 ? 0 : weeklyvalues.Sum()/weeklyvalues.Count; 
} 

Это столь же коротким, как я бы попытаться сделать это. Есть ли какая-то конкретная причина (кроме кода для гольфа), которую вы пытаетесь использовать для низкого количества символов?

Answer 4

public int GetAvgResult() 
{ 
var weeklyvalues = GetWeeklyValues();//gets list of weekly values. 
return (weeklyvalues.Count == 0) ? 0 : (weeklyvalues.Sum()/weeklyvalues.Count); 
} 

Answer 5

public int GetAvgResult() 
{ 
    var weeklyvalues=GetWeeklyValues();//gets list of weekly values. 
    return weeklyvalues.Count == 0 
     ? 0 
     : (weeklyvalues.Sum()/weeklyvalues.Count); 
}