Какая наилучшая практика для работы с объектами для циклов foreach или foreach? Должны ли мы создать один объект за пределами циклов и воссоздать его снова (используя новый...) или создать новый для каждой итерации цикла?
Пример:
foreach(var a in collection)
{
SomeClass sc = new SomeClass();
sc.id = a;
sc.Insert();
}
или
SomeClass sc = null;
foreach(var a in collection)
{
sc = new SomeClass();
sc.id = a;
sc.Insert();
}
Что лучше?
Первый способ лучше, поскольку он более четко передает предполагаемую область действия переменной и предотвращает случайные ошибки при использовании объекта вне предполагаемой области действия.
Одна из причин желания использовать вторую форму - это если вы хотите выйти из цикла и все еще иметь ссылку на объект, который вы последний раз достигли в цикле.
Плохая причина выбора второй формы - это производительность. На первый взгляд может показаться, что второй метод использует меньше ресурсов или что вы только создаете один объект и повторно используете его. Здесь не так. Повторное объявление переменной внутри цикла не потребляет дополнительных ресурсов или тактовых циклов, поэтому вы не получаете какой-либо выгоды от вытягивания декларации вне цикла.
Во-первых, я отмечаю, что вы имеете в виду "создание переменных", когда вы говорите "создание объектов". Ссылки на объекты идут в переменных, но они не являются самими переменными.
Обратите внимание, что описанный сценарий вводит смысловую разницу, когда цикл содержит анонимную функцию, а переменная является замкнутой внешней разновидностью анонимной функции. См
http://ericlippert.com/2009/11/12/closing-over-the-loop-variable-considered-harmful-part-one/
для деталей.
Я уверен, что кто-то может вытолкнуть анализ MSIL, но практически нет заметной разницы в исполнении или производительности. Единственное, что вы затрагиваете, это хранение ссылки на объект.
Я говорю, держите его чистым и простым; объявить переменную внутри цикла. Это обеспечивает принцип открытого/закрытого на практике, поэтому вы знаете область применения переменной и не используются повторно в других местах. В следующем цикле переменная теряет область действия и автоматически инициализируется.
Вы создаете новый объект в каждой итерации цикла в обоих случаях (так как вы вызываете new SomeClass()).
В предыдущем подходе ясно, что sc используется только внутри цикла, что может быть преимуществом с точки зрения обслуживания.
Я думаю, что это не важно для производительности, но я предпочитаю первую. Я всегда стараюсь, если возможно, вместе декларировать и создавать экземпляры.
Я бы пошел с опцией 2, чтобы быть аккуратным, чтобы сохранить все объявления в одном месте. Вы можете сказать, что "объекты должны быть объявлены только там, где и когда они нужны" но ваша петля, вероятно, будет иметь собственный небольшой метод.
Я бы использовал первый, но для компилятора это то же самое, потому что компилятор вытесняет объявление переменных из циклов. Уверен, что после компиляции код будет выглядеть как второй.