Не работает onclick функция?

Question

[Kuper132]

var blogContainer = document.getElementsByClassName('blog-container__blog-items')[0].clientHeight;
var buttonShow = document.getElementsByClassName('blog-container__button-show')[0];

function blogItems(container, show) {
    if (container > (container / 2))  {
        show.onclick = function onHide() {
            document.getElementsByClassName('blog-container__blog-items')[0].classList.add('overflow_blog-item');
            return false
        }
        } else {
        show.onclick = function onNice() {
           document.getElementsByClassName('blog-container__blog-items')[0].classList.remove('overflow_blog-item');
           return false
            }
        }
    }

Срабатывает только первый if. Потом как будто все умирает не могу понять в чем причина

Comments

[twobomb]

Мне кажется или тут всегда будет true?
if(container > (container / 2))

[Kuper132]

twobomb, ну после первого клика высота уже в половину меньше

[ThunderCat]

Kuper132,

ну после первого клика высота уже в половину меньше

и вы ее сравниваете с половиной себя... Логично, логично...

Answer 1

[Владимир]

Начнём с того, что container > (container / 2) выполняется всегда, за исключением случая, когда
container == 0

Comments

[Kuper132]

пробовал else if писать не работает. Что в таком случае делать?

[Владимир]

Kuper132, правильно написать условие if прежде всего, иначе ветвь else никогда не выполнится

[Kuper132]

Владимир,

var blogContainer = document.getElementsByClassName('blog-container__blog-items')[0];
var buttonShow = document.getElementsByClassName('blog-container__button-show')[0];

function blogItems(container, show) {
    if (blogContainer.classList.contains('overflow_blog-item'))  {
        show.onclick = function onNice() {
            document.getElementsByClassName('blog-container__blog-items')[0].classList.remove('overflow_blog-item');
            console.log('adasd')
            return false
        }
        } else {
        show.onclick = function onHide() {
            document.getElementsByClassName('blog-container__blog-items')[0].classList.add('overflow_blog-item');
            console.log('lol')
            return false

        }

    }

    }

теперь наоборот else срабатывает и невыходит

[Kuper132]

Владимир, получилось решить так я так понимаю через рекурсию

var blogContainer = document.getElementsByClassName('blog-container__blog-items')[0].clientHeight;
var buttonShow = document.getElementsByClassName('blog-container__button-show')[0];

console.log(child)
function blogItems(container, show) {
    if (blogContainer > (container / 2) )  {
     show.onclick = function onHide() {
            document.getElementsByClassName('blog-container__blog-items')[0].classList.add('overflow_blog-item');
            return  show.onclick = function onNice() {
                document.getElementsByClassName('blog-container__blog-items')[0].classList.remove('overflow_blog-item');
                return blogItems(container,show)
            }
          }
        }
    }
blogItems(blogContainer,buttonShow)

Возможно подскажите лучший вариант?