Заключение переменных в двойные кавычки, нормально или грех?
Можно. Я, например, использую и не вижу проблем. И лучше использовать синтаксис с усами — как по мне, так повышает читабельность помимо прочих плюшек:echo "Ответ {$res}";
Мое мнение тут главная ошибка в echo . На весь проект такой оператор должен быть один.
Вторая ошибка как я подозреваю контролеры смешиваются с представлениями.
Третья, не задумываетесь об многоязычности.
А спорить можно или нельзя можно долго. Только в правильном проекте такого вообще не должно встречаться.
А если нужно, собирать строки например для смс уведомления $mes =  sprintf("Уважаемый, %s! Стоимость вашего заказа %d руб.",$username,$cost);
просто сравните на сколько проще потом будет менять тексты, его можно даже вынести в таблицу