Как переписать эту лапшу?

Question

[codharma]

Как сделать этот код более DRY?

showButton () {
      if (this.$route.name === 'new-article') {
        if (this.article.section && this.article.title && this.article.description) {
          return true
        } else return false
      }
      if (this.$route.name === 'update-article') {
        if (this.show && this.article.section && this.article.title && this.article.description) {
          return true
        } else return false
      }
    }

Есть ощущение что можно обойтись без повторения блоков и излишней вложенности

Answer 1

[Ivan Bogachev]

showButton () {
    switch (this.$route.name) {
        case 'new-article':
            return this.article.section &&
                   this.article.title &&
                   this.article.description;
        case 'update-article':
            return this.show &&
                   this.article.section &&
                   this.article.title &&
                   this.article.description;
    }
}

Comments

[Igor Makarov]

Пойдет.
Но вынести бы еще повтор проверки в отдельный метод.

spoiler

showButton () {
    switch (this.$route.name) {
        case 'new-article':
            return this.isArticleHasMetaOrWhateverItIs();
        case 'update-article':
            return this.show && this.isArticleHasMetaOrWhateverItIs();
    }
}

isArticleHasMetaOrWhateverItIs() {
   return this.article.section &&
                   this.article.title &&
                   this.article.description;
}

[Ivan Bogachev]

Igor Makarov,

повтор проверки в отдельный метод

Или даже сделать у самой статьи свойство "hasMeta", которое будет реактивно рассчитываться в глубинах фреймворка, который использует автор (что-то подсказывает, что роутер у него там не сам по себе висит).

[Антон Спирин]

Igor Makarov, Пойдет.
Но привести бы еще результат выполнения isArticleHasMetaOrWhateverItIs() к boolean:

spoiler

isArticleHasMetaOrWhateverItIs() {
   return Boolean(
     this.article.section &&
     this.article.title &&
     this.article.description
   );
}

или

isArticleHasMetaOrWhateverItIs() {
   return !!(
     this.article.section &&
     this.article.title &&
     this.article.description
   );
}

[Igor Makarov]

Антон Спирин, хорошее замечание

Answer 2

[TheAthlete]

Решение в лоб:

showButton () {
  if (this.$route.name === 'new-article') {
    return !!(this.article.section && this.article.title && this.article.description);
  }
  if (this.$route.name === 'update-article') {
    return !!(this.show && this.article.section && this.article.title && this.article.description);
  }
}

с выносом в отдельную переменую повторяющихся частей

showButton() {
  var is_sect_title_descr = !!(this.article.section && this.article.title && this.article.description);
  if (this.$route.name === 'new-article')    { return is_sect_title_descr; }
  if (this.$route.name === 'update-article') { return this.show && is_sect_title_descr; }
}

Answer 3

[voronkovich]

showButton () {
    if (!this.article.section || !this.article.title || !this.article.description) {
        return false;
    }

    if (this.$route.name === 'new-article') {
        return true;
    }

    if (this.$route.name === 'update-article' && this.show) {
        return true;
    }

    return false;
}

Answer 4

[TDerden]

showButton () {
return (((this.$route.name === 'update-article')?this.show:true) && this.article.section && this.article.title && this.article.description);
}

Comments

[Anton Mashletov]

Нечитаемый звиздец. Уж лучше как автор писать.

[TDerden]

Мне ифы тоже больше нравятся, но спрашивали как сократить. Я максимально сократил, думаю из этого легко раскрутить уровни вложенности обратно и понять на каком уровне необходимо остановиться.
Еще можно {this.article.section && this.article.title && this.article.description} вынести в функцию
showButton () {
function isSplitAtr(){
return this.article.section && this.article.title && this.article.description
};

if (this.$route.name === 'new-article') {
return(isSplitAtr);
}
if (this.$route.name === 'update-article') {
return (this.show && isSplitAtr);
}
}

Answer 5

[MrTimon]

Зачем эти повторения и ф-ции не могу понять. Как по мне проще сделать так

showButton () {
    return in_array(this.$route.name, ['new-article','update-article'] ) && this.article.section && this.article.title && this.article.description;
}

Comments

[MrTimon]

Ахх, блин, извиняюсь. Только сейчас понял что во втором варианте есть еще одно условие, ну тогда без вариантов, только з if но думаю тоже можно обєденит через or. Как то так

showButton() {
  if (this.$route.name === 'new-article' || this.$route.name === 'update-article' && this.show)    { 
     return  !!(this.article.section && this.article.title && this.article.description);
  }
  return false;
}