Блог

Недавно у нас возникла проблема, когда мы вызывали словарь для извлечения значения, ожидая, что ключ будет присутствовать. Это не привело к прерыванию процесса.

Есть ли у ReSharper функциональность для проверки этого, как я слышал, для нулевых объектов?

Вот пример, подтверждающий то, о чем я говорю:

 Dictionary<String, Entity> allEntities = 
      new Dictionary<String, Entity>(SringComparer.OrdinalIgnoreCase);

allEntities.AddMany(db.GetAllEntities());

Entity thisEntity = allEntities[entityID]; 
      // <-- error here as EntityID isn't in all entities...
  

Я бы хотел, чтобы ReSharper мог сказать, что я не проверял словарь таким образом:

 if (allEntities.ContainsKey(entityID))
    ...
  

К вашему сведению, у меня нет ReSharper, но это было бы еще одним дополнением к бизнес-обоснованию, чтобы получить его для всех разработчиков.

Даже если бы это было возможно (чего я не думаю, что это так), все, что он сделает, это предупредит вас, что ключа может не быть в словаре — точно так же, как он предупреждает вас, что объект может быть нулевым. На вас по-прежнему лежит ответственность за добавление кода, чтобы проверить, считаете ли вы это необходимым.

Я думаю, что такая функция была бы скорее бременем, чем благословением, если бы она предупреждала вас о каждом вашем доступе к словарю. Тогда вы могли бы утверждать, что хотели, чтобы он предупреждал вас о любом другом типе доступа к коллекции или о любом исключении, которое может быть вызвано. Основное сообщение здесь заключается в том, что вам следует больше полагаться на модульное тестирование для выявления таких проблем, а не на инструменты статического анализа

Редактировать:

Поскольку ReSharper не обладает такой функциональностью, если вы действительно этого хотите, вы можете подумать о том, чтобы написать его самостоятельно с помощью новых API Roslyn. Смотрите Эту статью для примера того, как написать анализатор кода

В настоящее время я установил ReSharper 6, и, по крайней мере, с моими настройками, он не предупреждает меня о проверке a Dictionary ContainsKey перед доступом к его значению.

Resharper — это инструмент, используемый во время компиляции. Как он может узнать, какие значения будут помещены в словарь во время выполнения?

Вы можете проверить словарь, чтобы увидеть, имеет ли он нужное вам значение, прежде чем что-либо с ним делать.

 Entity thisEntity;
if (allEntities.TryGetValue(entityID, out thisEntity)){
    //DoStuff with thisEntity
}
  

или просто используйте if (allEntities.ContainsKey(entityID)){}