C# "lock" в многопотоке.

[PROforum]

Здравствуйте. Подскажите Нужно ли в этом коде ставить lock для многопотока или нет? Если да то в каком варианте правильно

lock(SyncObjects.TableSyncer) //Блокируем таблицу для работы в многопотоке
{

IZennoTable table = project.Tables["Аккаунты"];


for(int i = 0; i < table.RowCount; i++)
{
    if (table.GetCell("A", i).Contains(project.Variables["account"].Value))
    {
        return i;
    }
}
throw new Exception("Строка не найдена"); //ошибка, если не найден
}

IZennoTable table = project.Tables["Аккаунты"];

lock(SyncObjects.TableSyncer) //Блокируем таблицу для работы в многопотоке
{

for(int i = 0; i < table.RowCount; i++)
{
    if (table.GetCell("A", i).Contains(project.Variables["account"].Value))
    {
        return i;
    }
}
throw new Exception("Строка не найдена"); //ошибка, если не найден
}

 

[nicanil]

Тут лок ставить не обязательно, т.к. Вы только считываете данные, без их модификации.

 

[PROforum]

А если бы в коде происходила запись в таблицу привязанную к шаблону, то какой вариант правильный?

 

[nicanil]

Я особой разницы не вижу. Можно использовать оба варианта.

 

[volody00]

А если бы в коде происходила запись в таблицу привязанную к шаблону, то какой вариант правильный?

Лучше 2-й, т.к. в первом случае у тебя объявление таблицы идет в {} и в дальнейшем не сможешь к ней обратиться (т.е. придется её опять объявлять)

 

[Phoenix78]

Желательно всегда использовать Лок. Потому что тут возможно ты только считыватель данные, а другой шаблон начал какие то изменения. В результате получаешь кашу.
Лок ресурсов не жрёт дополнительных, а если вдруг надо поработать без лока, так лучше специально потом убрать Лок в нужном месте и не беспокоится , поставил ли я где Лок или забыл.