Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #1 (permalink)  
Старый 08.09.2015, 13:30
Аватар для Momon
Аспирант
Отправить личное сообщение для Momon Посмотреть профиль Найти все сообщения от Momon
 
Регистрация: 02.12.2014
Сообщений: 72

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

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

// Описание что делает код ниже
{
   //...
}
Мне просто так удобно. Потом это удобно сворачивать и разворачивать. Это сильно плохо?
Ответить с цитированием
  #2 (permalink)  
Старый 08.09.2015, 13:34
Профессор
Отправить личное сообщение для tsigel Посмотреть профиль Найти все сообщения от tsigel
 
Регистрация: 12.12.2012
Сообщений: 1,398

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

Последний раз редактировалось tsigel, 08.09.2015 в 13:38.
Ответить с цитированием
  #3 (permalink)  
Старый 08.09.2015, 13:36
Профессор
Отправить личное сообщение для alex.vv Посмотреть профиль Найти все сообщения от alex.vv
 
Регистрация: 15.04.2014
Сообщений: 176

Скорее, это несоответствие общепринятому code style.
Вообще если получается большая "простыня" кода, нередко рекомендуют делать extract method и тому подобное.
Ответить с цитированием
  #4 (permalink)  
Старый 08.09.2015, 14:27
Аватар для Momon
Аспирант
Отправить личное сообщение для Momon Посмотреть профиль Найти все сообщения от Momon
 
Регистрация: 02.12.2014
Сообщений: 72

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

Последний раз редактировалось Momon, 08.09.2015 в 14:36.
Ответить с цитированием
  #5 (permalink)  
Старый 08.09.2015, 15:05
Аватар для ruslan_mart
Профессор
Отправить личное сообщение для ruslan_mart Посмотреть профиль Найти все сообщения от ruslan_mart
 
Регистрация: 30.04.2012
Сообщений: 3,018

Momon, проще просто табуляцию сделать.
Ответить с цитированием
  #6 (permalink)  
Старый 08.09.2015, 15:06
Профессор
Отправить личное сообщение для tsigel Посмотреть профиль Найти все сообщения от tsigel
 
Регистрация: 12.12.2012
Сообщений: 1,398

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

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

  }

  if (this.isSome()) {

  }
Ответить с цитированием
  #7 (permalink)  
Старый 09.09.2015, 11:41
Аватар для Momon
Аспирант
Отправить личное сообщение для Momon Посмотреть профиль Найти все сообщения от Momon
 
Регистрация: 02.12.2014
Сообщений: 72

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

Всем спасибо!
Ответить с цитированием
Ответ



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Баги Opera пишем сюда devote Оффтопик 101 08.08.2013 05:56
поиск классов внутри тега yozuul jQuery 24 14.06.2013 22:00
Решение проблемы кодировок для AJAX и PHP без iconv (cp1251 в AJAX) Serge Ageyev AJAX и COMET 10 24.04.2013 20:48
http://gigalit.info - Мои контакты Маэстро Ваши сайты и скрипты 70 08.12.2011 03:12