Что лучше: обработать исключение или не вызывать его?

Question

[Тимур Покровский]

Допустим у меня есть список
var list = new List<int> { 1, 2, 3, 4 };
и как будет правильнее сделать?
Так:

int getValue(int index) {
    try {
        return list[index];
    }
    catch (ArgumentOutOfRangeException ex) {
        return -1;
    }
}

или так:

int getValue(int index) {
    if (index < list.Count) {
        return list[index];
    }
    else {
        return -1;
    }
}

или разницы нет?

Comments

[Hemul GM]

Во втором случае тоже может быть исключение, если аргумент index будет меньше ноля.

Answer 1

[#]

практикуйте замеры времени исполнения.. и вопросы отпадут

ps для async схем все может сильно меняться

Answer 2

[freeExec]

Тут нет лучше, вопрос что вам нужно. Исключение вам всё равно придётся где-то обработать, и вы сами решаете на каком уровней логики не критично не получить результат.
Тот же List не может подсунуть вам фейк в виде -1, потому что он не знает, что это какое-то не валидное значение в вашем случае.

Answer 3

[Владимир Коротенко]

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