Телеграмм чат группы javascript

14:24пожаловаться #1

Dima Nazdratenko in JavaScript.Ninja

просто в первом повторения ненужные, в втором случае лишняя вложенность 🤷

14:25пожаловаться #2

МВ

Мопсим Вертухаев... in JavaScript.Ninja

подскажите по чистоте кода как лучше из 2 вариантов:
1)

if (condition 1) {
  same code

  different code

  same code
} else if (condition 2) {
  same code

  different code

  same code
}

2)
if

 (condition 1 || condition 2) {
  same code
  
  if (condition 1) different code
  else if (condition 2) different code
 
  same code
}

Второй же вариант лучше?)

Как по мне 1 более читабельный, а вообще зависит от код стайла команды и настройки линтера.

14:26пожаловаться #3

Sergey Pavlenko in JavaScript.Ninja

node 12.8.3

package.json
"dependencies": {
"@mdi/font": "^5.0.45",
"@nuxtjs/axios": "^5.3.6",
"cross-env": "^5.2.0",
"express": "^4.16.4",
"express-session": "^1.17.0",
"moment": "^2.24.0",
"nuxt": "^2.0.0",
"nuxt-i18n": "^6.6.1",
"session-file-store": "^1.4.0",
"vuelidate": "^0.7.5"
},
"devDependencies": {
"@nuxtjs/eslint-config": "^2.0.0",
"@nuxtjs/eslint-module": "^1.0.0",
"@nuxtjs/vuetify": "^1.0.0",
"babel-eslint": "^10.0.1",
"eslint": "^6.1.0",
"eslint-plugin-nuxt": ">=0.4.2",
"eslint-plugin-vue": "^6.2.2",
"nodemon": "^1.18.9"
}

Всем привет! Были ли у кого такие проблемы с данным стеком? На сервере всё ок, т.е. скорее всего такое проявляется в дев режиме.

14:36пожаловаться #4

Taras in JavaScript.Ninja

подскажите по чистоте кода как лучше из 2 вариантов:
1)

if (condition 1) {
  same code

  different code

  same code
} else if (condition 2) {
  same code

  different code

  same code
}

2)
if

 (condition 1 || condition 2) {
  same code
  
  if (condition 1) different code
  else if (condition 2) different code
 
  same code
}

Второй же вариант лучше?)

мне первый больше. Лучше повторение, зато прозрачная логика

14:36пожаловаться #5

Dima Nazdratenko in JavaScript.Ninja

Taras

мне первый больше. Лучше повторение, зато прозрачная логика

принято, спасибо)

14:36пожаловаться #6

подскажите по чистоте кода как лучше из 2 вариантов:
1)

if (condition 1) {
  same code

  different code

  same code
} else if (condition 2) {
  same code

  different code

  same code
}

2)
if

 (condition 1 || condition 2) {
  same code
  
  if (condition 1) different code
  else if (condition 2) different code
 
  same code
}

Второй же вариант лучше?)

same code

codeThatShouldExecute(condition)

same code

14:48пожаловаться #7

Даже не так.

14:49пожаловаться #8

Можешь условие между ними написать, а не выносить куда-либо.

14:49пожаловаться #9

Dima Nazdratenko in JavaScript.Ninja

th.witness

Можешь условие между ними написать, а не выносить куда-либо.

дык мой второй вариант вроде такой же

14:50пожаловаться #10

дык мой второй вариант вроде такой же

Ага, я не вник.

14:51пожаловаться #11

Sergey Nobiev in JavaScript.Ninja

Какие потенциальные проблемы могут быть если затащить в react rxjs?

Дмитрий Дивин... in JavaScript.Ninja

17:47пожаловаться #12

ДД

Sergey Nobiev

Какие потенциальные проблемы могут быть если затащить в react rxjs?

Программировать придётся :(

17:55пожаловаться #13

vasilich in JavaScript.Ninja

Дмитрий Дивин

Программировать придётся :(

Ору)

Глеб Рябцев... in JavaScript.Ninja

17:55пожаловаться #14

ГР

А какие могут быть проблемы помимо увеличения бандла

17:56пожаловаться #15

vasilich in JavaScript.Ninja

Sergey Nobiev

Какие потенциальные проблемы могут быть если затащить в react rxjs?

Порог входа для новичков, например.

17:56пожаловаться #16

Sergey Nobiev in JavaScript.Ninja

если не учитывать порог входа)

17:57пожаловаться #17

Sergey Nobiev in JavaScript.Ninja

тут же более прикладной момент интересует) скажем rxjs человек знает

17:57пожаловаться #18

vasilich in JavaScript.Ninja

Да что ж таке)
Все не такие и не таки проблемы)))

Глеб Рябцев... in JavaScript.Ninja

17:57пожаловаться #19

ГР

Если человек хочет включить либу в проект, то скорее всего он знает как с ней обращаться, иначе лучше не тащить а сначала поексперементировать и самому ответить