Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   Фигурные скобки без условий для группировки кода - это говнокод? (https://javascript.ru/forum/misc/58165-figurnye-skobki-bez-uslovijj-dlya-gruppirovki-koda-ehto-govnokod.html)

Momon 08.09.2015 13:30

Фигурные скобки без условий для группировки кода - это говнокод?
 
Просто использую скобки {} для наглядности, обычно так:
// Описание что делает код ниже
{
   //...
}

// Описание что делает код ниже
{
   //...
}

// Описание что делает код ниже
{
   //...
}
Мне просто так удобно. Потом это удобно сворачивать и разворачивать. Это сильно плохо?

tsigel 08.09.2015 13:34

http://habrahabr.ru/post/176951/
Восновном там соль в комментах. По крайней мере я их поддерживаю.

alex.vv 08.09.2015 13:36

Скорее, это несоответствие общепринятому code style.
Вообще если получается большая "простыня" кода, нередко рекомендуют делать extract method и тому подобное.

Momon 08.09.2015 14:27

Я должен пояснить, что использую такой финт с пустыми скобками {} только если не получается использовать стандартный блок. Т.е. например какой то логически связанный блок кода содержит несколько условий if, которые ну никак не объединить в одни скобочки.
// бла бла бла
{
    if {
    }
    if {
    }
}
Ну это же действительно более наглядно чем самому додумывать что эти два условия логически связаны, находясь при этом на одном уровне вложенности. Т.е. так ты сразу видишь скобки и комментарий и понимаешь что к чему. А так нужно будет прочитать первое и второе условие (или ещё комментарии к ним) и додумать что они вместе работают. Можно шлёпнуть комментарий и просто так, но тогда нельзя будет всё это дело свернуть плюсиком в редакторе, а это очень удобно :)

ruslan_mart 08.09.2015 15:05

Momon, проще просто табуляцию сделать.

tsigel 08.09.2015 15:06

Momon,
Обычно я такие вещи выношу в отдельные методы. И оставляю вызов с человекопонятным названием. И все условия вношу в метод который возвращает булеан, так всегда нагляднее.

что-то типа:
if (this.hasClassActive()) {

  }

  if (this.isSome()) {

  }

Momon 09.09.2015 11:41

tsigel,
По-моему всё в подряд выносить в методы/отдельные функции это перебор. Хотя может быть я не прав.

Всем спасибо! :)


Часовой пояс GMT +3, время: 12:32.