Если было бы больше контекста, то можно было бы что-то увереннее советовать. По этому куску кода можно дать только такие комментарии (считая что Get* методы и итераторы не имеют сайд эффектов):
var someItems = _someService.GetList();
foreach (var item in someItems) // item внутри не используется, можно убрать этот foreach
{
var additionalItems = _additionItemsSerivce.GetList();
foreach (var additionalItem in additionalItems)
{
var properties = additionalItem.Properties;
foreach (var property in properties)
{
property.First = true;
property.First = true; // зачем делать одно и тоже два раза?
_additionItemsSerivce.UpdateProperies(additionalItem); // property не используется, значит можно вынести за этот foreach
}
}
}
Если по этим комментариям убрать лишнее, получится что-то такое:
var additionalItems = _additionItemsSerivce.GetList();
foreach (var additionalItem in additionalItems)
{
var properties = additionalItem.Properties;
foreach (var property in properties)
property.First = true;
_additionItemsSerivce.UpdateProperies(additionalItem);
}
Так же странно что все проперти у вас получают флаг First, возможно вместо foreach по properties стоит получить только первый элемент и ему установить First = true.
