Есть вот такой код:
...
Match match;
foreach (var value in egeQuestionValues)
{
match = Regex.Match(value, $@"(?<={number}\()\d+(?=.*%\))");
if (!match.Success)
throw new ArgumentException($"The record {value} is incorrect", nameof(egeQuestionValues));
egeQuestionIntValues.Add(int.Parse(match.Value));
}
return egeQuestionIntValues.Average();
Я создал переменную match
специально вне цикла foreach
, чтобы в рантайме не создать ее при каждой итерации. Делал, думаю что это затратно. Но ReSharper мне советует отрефакторить этот код таким образом:
...
foreach (var value in egeQuestionValues)
{
var match = Regex.Match(value, $@"(?<={number}\()\d+(?=.*%\))");
if (!match.Success)
throw new ArgumentException($"The record {value} is incorrect", nameof(egeQuestionValues));
egeQuestionIntValues.Add(int.Parse(match.Value));
}
return egeQuestionIntValues.Average();
т.е. сейчас при каждой итерации переменная match
будет создаваться каждый раз. Разве это не затратно? ReSharper советует поступать так не только с типом Match
и с другими типами так было.