Ограниченно допустимо. Когда вы такую последовательность будете обходить в цикле foreach - то перед входом в цикл блокировка будет взята, а после выхода - освобождена. Также блокировка будет освобождена при любом прерывании цикла.
Однако, использование IEnumerable не ограничивается простыми циклами. При использовании блокировок всегда важно небольшое время удержания блокировки - а тут вы его растягиваете на не зависящее от вас время! Так можно легко и до взаимоблокировки доиграться.
Обычно лучше всего под блокировкой получить копию данных, и работать в дальнейшем уже с ней:
public IEnumerable<SomeObject> Get()
{
IEnumerable<KeyValuePair<SomeKey, SomeValue>> d;
_lock.EnterReadLock();
try
{
d = _dictionary.ToList();
}
finally
{
_lock.ExitReadLock();
}
foreach (var item in d)
{
yield return new SomeObject(item.Key, item.Value);
}
}
Или даже вот так:
public IEnumerable<SomeObject> Get()
{
_lock.EnterReadLock();
try
{
return _dictionary.Select(item => new SomeObject(item.Key, item.Value)).ToList();
}
finally
{
_lock.ExitReadLock();
}
}