Чем плох этот код ?

Question

[RUSgrigoryev]

Ниже написал код, который при клике на внутренний скрывает родительский элемент. Вопрос : это один из паттернов JS или нет ?

<div id="first"><p class="inner">Lorem ipsum dolor sit amet, consectetur adipisicing elit. Fuga non sit ratione atque minus voluptates tempora aliquid aut perspiciatis impedit harum id et sed molestiae ad asperiores iusto nobis alias.</p></div>

<div id="second"><p class="inner2">Lorem ipsum dolor sit amet, consectetur adipisicing elit. Voluptatem odit molestias velit consectetur molestiae. Minima obcaecati similique odio incidunt recusandae consequuntur ipsam provident inventore nesciunt numquam dignissimos nihil harum eveniet.</p></div>

function closeElem (elem, parent) {
    var obj = {
        otps : {
            elem  : elem,
            parent: parent
        },
        init: function () {
            $(elem).on('click', function () {
                $(this).closest(parent).hide();
            });
        }
    };
    obj.init();
    return obj;
}

var closediv = closeElem('.inner', '#first');
var closediv = closeElem('.inner2', '#second');

Answer 1

[aumk]

Это лучший код, который я когда-либо видел. БРАВО!

Comments

[RUSgrigoryev]

Без сомнений. Он идеален.

Answer 2

[Сергей Протько]

Даже не знаю с чего начать... Всем... Как минимум нету смысла в проперте opts, у вас и так переменные elem и parent будут доступны в родительском скоупе. Использовали бы хотя бы полноценные объекты, хотя опять же не понимаю смысла. Как-то слишком много кода...

Что должно быть в closedDiv? И зачем такие сложности? Мне кажется плох не код, а подход.