Если вы конвертируете в конце представления из ViewData обратно в свой словарь, то да, производительность снизится.
Кроме того, теперь вы вводите код в представление. В представлениях должно быть мало или совсем нет кода.
Ваш вид может наследовать от класса по причине. используй это. ViewData следует использовать только для небольших строк, таких как mayby заголовков страниц или что-то в этом роде.
Я просто исправляю проект, в котором ни одно представление не наследуется от модели, а вместо этого использует ViewData для списков и тому подобное. Иногда 15 из них в одном представлении. Чертовски медленно, но, заменяя ViewData моделями, мы возвращаем скорость.