1. JavaScript / Говнокод #25338

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    let tableCellHamburgers = Array.prototype.slice.call(document.getElementsByClassName('js-listen__letters-table__hamburger'));
  tableCellHamburgers.forEach((tableCellHamburger) => {
	let code = Array.prototype.slice.call(tableCellHamburger.classList).filter(elem => elem.indexOf('js-table__cell__hamburger__') === 0)[0].replace('js-table__cell__hamburger__', '');
	let searchInput = document.querySelector(`.js-table__cell__search-input__${code}`);
	if (!(searchInput && searchInput.value)) {
	  tableCellHamburger.style.filter = 'none';
	}
	let dropdownContent = document.querySelector(`.js-table__cell__dropdown-content__${code}`);
	if (dropdownContent) {
	  dropdownContent.style.display = 'none';
	}
  });

    Автор этого кода (full-stack программист) собирался выпилить jQuery из проекта, т.к. всю работу с DOM давно можно красиво писать на чистом ES6. К счастью, он уволился, не осуществив свой замысел.

    Запостил: webviewer, 30 Января 2019

    Комментарии (24) RSS

    • ava guest8 30.01.2019 14:32 # −999

      показать все, что скрытоvanished
      Ответить

      • ava JIyHHblu_nemyx 30.01.2019 15:25 # 0

        На Жуквери можно было бы написать чуть менее многословно:
        $('.js-listen__letters-table__hamburger')each(function(){
    var code = $(this).find('.js-table__cell__hamburger__:first').removeClass('js-table__cell__hamburger__');
    var searchInput = $(.js-table__cell__search-input__'+code);
    if (!(searchInput && searchInput.value)) {
	  $(this).css('filter', 'none');
    }
    $('.js-table__cell__dropdown-content__'+code).hide();
));


        Представь, сколько буков можно сократить!
        Ответить

        • ava guest8 30.01.2019 22:54 # −999

          показать все, что скрытоvanished
          Ответить

          • ava JIyHHblu_nemyx 30.01.2019 23:45 # 0

            Покажите класс!
            Ответить

            • ava bootcamp_dropout 31.01.2019 00:10 # 0

              Подержите мое п̶и̶в̶о̶ смузи!
              let tableCellHamburgers = Array.from(document.getElementsByClassName('js-listen__letters-table__hamburger'))
	.map(tableCellHamburger => ((code => (
		((selector, style) => {style.filter = selector && selector.value ? style.filter : 'none'})
		(document.querySelector(`.js-table__cell__search-input__${code}`), tableCellHamburger.style),
		(dropdownContent => {if(dropdownContent) dropdownContent.style.display = 'none'})
		(document.querySelector(`.js-table__cell__dropdown-content__${code}`))
	))(Array.from(tableCellHamburger.classList).find(elem => !elem.indexOf('js-table__cell__hamburger__')).replace('js-table__cell__hamburger__', '')),tableCellHamburger));
              Ответить

    • ava JIyHHblu_nemyx 30.01.2019 15:07 # 0

      Да тут и ES6 не нужен:
      var tableCellHamburgers = document.getElementsByClassName('js-listen__letters-table__hamburger');
for(var i = 0; i < tableCellHamburgers.length; ++i) {
	var code = tableCellHamburgers[i].classList.filter(function(elem){return elem.indexOf('js-table__cell__hamburger__') === 0;})[0].replace('js-table__cell__hamburger__', '');
	var searchInput = document.querySelector('.js-table__cell__search-input__'+code);
	if (!(searchInput && searchInput.value)) {
	  tableCellHamburgers[i].style.filter = 'none';
	}
	var dropdownContent = document.querySelector('.js-table__cell__dropdown-content__'+code);
	if (dropdownContent) {
	  dropdownContent.style.display = 'none';
	}
  }


      Кстати, какая польза от Эррей.прототайп.слайс.кал с одним аргументом, которым так пестрит код?
      Ответить

    • ava JIyHHblu_nemyx 30.01.2019 15:27 # 0

      А нет ли здесь говна в пятой строчке? searchInput –— это же массив, поэтому searchInput.value будет означать какую-то фигню.
      Ответить

      • ava guest8 30.01.2019 15:54 # −999

        показать все, что скрытоvanished
        Ответить

        • ava JIyHHblu_nemyx 30.01.2019 15:57 # +1

          Кокококонечно! Если бы тупизация была статической, кокококомпилятор сразу бы зарубил кококод и программист не смог бы сдать работу. А при динамической ошибка может обнаружиться только через несколько месяцев эксплуатации, кокококогда программист уже получил свою порцию «Роллтона».
          Ответить

    • ava gost 30.01.2019 15:48 # +1

      > Array.prototype.slice.call
      Я хуею с этого дизайна.
      Ответить

    • ava bootcamp_dropout 30.01.2019 22:21 # +2 

      Array.prototype.slice.call(tableCellHamburger.classList).filter(elem => elem.indexOf('js-table__cell__hamburger__') === 0)[0]

      Человек Array.prototype.find не знает, а уже переводить на ES6 хотел?
      Ответить

    • ava webviewer 31.01.2019 13:48 # +1

      Этот код на jQuery (с небольшим рефакторингом верстки)
      $('.js-hamburger').each((index, el) => {
        const $hamburger = $(el);
        const $cell = $hamburger.closest('.js-table-cell');
        const $clearFilterButton = $cell.find(`.js-clear-filter-button`);
        const isFilterValueExist = !!$cell.find('.js-filter-search').val();
        if (!isFilterValueExist) {
          $hamburger.removeClass('filter-hamburger-active');
          $clearFilterButton.hide();
        } else {
          $clearFilterButton.show();
        }
        $cell.find('.js-drop-down-content').hide();
      });
      Ответить

    Добавить комментарий