Size: a a a

2021 March 04

МТ

Марк Танащук... in Svelte [svelt]
Dan Kozlov
Я все еще ничего не понял. В примере не понял, что из этих трех вариантов хорошо, а что плохо.
Почитай книжечку фаулера мартина про рефакторинг js
источник

МТ

Марк Танащук... in Svelte [svelt]
Там все по полочкам и ясненько расписано - отличная книга
источник

МТ

Марк Танащук... in Svelte [svelt]
Я бы даже сказал маст хев каждому
источник

МТ

Марк Танащук... in Svelte [svelt]
Я так как он не распишу))
источник

DK

Dan Kozlov in Svelte [svelt]
Мне комфортно без чтения таких книг и без знания паттернов. Я ж не программист, сне дозволено говорить то, что у всех на уме.
источник

AI

Aleksey Immũzov in Svelte [svelt]
Марк Танащук
// Код подсвечивающий выделенный текст
let anotherVar = variableName.invert()

vs

let anotherVar = highlight(variableName);
function highlight(color) {
 return color.invert()
}

или

const highlight = color => color.invert()
Удваиваю. В этом случае ты легко меняешь реализацию не трогая бизнес-логику.
источник

МТ

Марк Танащук... in Svelte [svelt]
Dan Kozlov
Мне комфортно без чтения таких книг и без знания паттернов. Я ж не программист, сне дозволено говорить то, что у всех на уме.
Я просто ссылку дал на более ясное объяснение, если ты не хочешь ее читать, то это твое право =)
источник

DK

Dan Kozlov in Svelte [svelt]
Марк Танащук
// Код подсвечивающий выделенный текст
let anotherVar = variableName.invert()

vs

let anotherVar = highlight(variableName);
function highlight(color) {
 return color.invert()
}

или

const highlight = color => color.invert()
Так а что из этих трех вариантов хорошо, а что плохо?
источник

МТ

Марк Танащук... in Svelte [svelt]
Dan Kozlov
Так а что из этих трех вариантов хорошо, а что плохо?
Если ты от меня ждешь краткого ответа - второй вариант с функцией лучше =)
источник

DK

Dan Kozlov in Svelte [svelt]
Eye opening. У меня во всей кодбазе не наберется и пяти упоминаний этого слова. Я что-то делаю не так.
источник

AS

Alexey Schebelev in Svelte [svelt]
а чем плохо в этом примере если ты перееменные как и функции осмысленно назовешь? let blackColor = whiteColor.invert()
источник

МТ

Марк Танащук... in Svelte [svelt]
Alexey Schebelev
а чем плохо в этом примере если ты перееменные как и функции осмысленно назовешь? let blackColor = whiteColor.invert()
А не лучше ли все сразу? :)
источник

МТ

Марк Танащук... in Svelte [svelt]
Один рефакторинг не может без остальных, всегда лучше, когда на понимание того, что делает твой код тебе достаточно посмотреть на название функции или первые несколько строк тела, чем читать длинный комментарий с объяснением тела фунции или смысла переменной
источник

МТ

Марк Танащук... in Svelte [svelt]
Комментарии хороши для IDE'шек чтобы наведя на функцию можно было увидеть что она возвращает, типы переменных(если без тса) и т.д. и т.п.(особенно при огромной кодовой базе), но когда они начинают объяснять тело функции то есть смысл(имхо) начать думать над рефакторингом этого куска кода(если время есть конечно)
источник

МТ

Марк Танащук... in Svelte [svelt]
Или когда код высокопроизводительный(базовый пример - chevrotain с его высокопроизводительным парсингом), то как раз описание комментарием функций и её тела просто спасает
источник

A

Arushwl in Svelte [svelt]
источник

NK

ID:0 in Svelte [svelt]
источник

A

Arushwl in Svelte [svelt]
Продолжаются эксперименты с бандлерами... когда там появится esbuild...🤔
источник

AB

Alex Bekhtin in Svelte [svelt]
Пока сыро, запускается с ошибками, и не работает
источник

A

Arushwl in Svelte [svelt]
Вот кстати укомплектованный  бойлералейт на esbuild + derver + pathfinder + ts  + prettier + stylelint + eslint – https://github.com/tonyrewin/svelte-boilerplate
источник