используйте css
www.w3schools.com/cssref/sel_focus.asp , зачем лишний раз юзать JS.
input:focus{
outline: solid 2px red;
}
И зачем лишние операции, зачем превращать NodeList в массив? Перемудрили для такой простой задачи.
А насчет эффективности - если вы знаете примерно как работает код, то не важно, что под капотом, вы должны понимать, что это лишние операции - браузер сначала ищет все элементы удовлетворяющие условию(querySelector), потом все это дело превращает в массив, потом проходит по каждому элементу и вешает обработчик