Смешивание считают шаблоном разработки?

Есть несколько проблем с вашим кодом (всегда проверяйте консоль на наличие ошибок).

Во-первых, поймите разницу между узлом и набором узлов. Такие функции, как getElementsByClassName, возвращают набор узлов , который представляет собой массив узлов. Поэтому вы не можете запускать методы узлов на них как есть, т. Е. Это приведет к ошибке:

boxDiv.nextSiblings(... //boxDiv is not a node; it's a nodeset

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

Далее, nextSiblings и previousSiblings не являются допустимыми методами; они должны быть в единственном числе, а не во множественном числе.

Далее, вы предполагаете, что есть следующий и предыдущий братья и сестры. Это приведет к ошибке в первом и последнем .box div по очевидным причинам.

Кроме того, пока мы здесь:

• , как правило, лучше избегать DOM-событий «уровня 0», например. .onclick и вместо этого используйте addEventListener(). Есть несколько причин для этого, которые выходят за рамки этого ответа.
• В наши дни, если вам не нужна поддержка старых браузеров, лучше использовать let, чем var

Давайте все соберем вместе

let
parent = document.querySelector('#row'),
boxDiv = document.querySelectorAll('.box'),
btnCh = document.querySelector('#changeColor')
;

btnCh.addEventListener('click', evt => {
    boxDiv.forEach(el => {
        if (el.classList.contains('green')) {
            if (el.nextSibling) el.nextSibling.classList.add('green');
            if (el.previousSibling) el.previousSibling.classList.add('green');
        }
    });
}, false);