0

Здравствуйте!

Подскажите, пожалуйста, есть ли смысл выносить код в отдельную функцию если он встречается на странице только один раз? Влияет ли это на производительность или только на удобство чтения кода? Может есть какие-то подводные камни?

Никак не могу найти статьи в интернете на эту тему, везде встречается фраза - плохой стиль.

Есть кнопка, при нажатии которой выполняются определенные действия:

<button class="details" onclick="
var x = document.getElementsByClassName('details'); 
for (;x[0];) { 
x[0].className = 'det';}">Все характеристики</button>

Есть вариант вынести весь код в отдельную функцию и написать так:

<button class="details" onclick="my_func()">Все характеристики</button>

И всё-таки как лучше сделать? Заранее спасибо за ответ!

13
  • 3
    функция оф корз. в кнопке я б лично даже onclick="my_func()" не оставлял бы. всё в коде биндил бы.........или только на удобство чтения кода - не только читаемость, но и разделять надо. а еще и тестировать код некоторые любят) Commented 16 нояб. 2016 в 18:17
  • 1
    если код 100% не будет никогда меняться - без разницы
    – Grundy
    Commented 16 нояб. 2016 в 18:18
  • @Grundy ты сам в такое веришь-то? %) Commented 16 нояб. 2016 в 18:19
  • @АлексейШиманский, зачёт сдал и забыл?))
    – Qwertiy
    Commented 16 нояб. 2016 в 18:20
  • Плохой стиль - это очень серьезный аргумент, если Вы занимаетесь программированием профессионально или постоянно-любительски, и вообще не имеет значения, если вам лабу сдать и на всю оставшуюся жизнь забить. Commented 16 нояб. 2016 в 18:20

1 ответ 1

0

Короткий ответ: вынести в отдельную функцию.

Длинный ответ.

Лично я предпочитаю следовать такому порядку разработки:

  1. Сделайте, чтобы работало (но никому это не показывайте). На этом шаге можно написать что-то похожее на то, что у вас сейчас.
  2. Сделайте, чтобы ваш код было удобно читать и понимать. Здесь вы выносите код в отдельную функцию, понятным образом ее называете, форматируете отступами, навешиваете обработчик через javascript, а не в атрибуте html и т. д. Получается вариант, который уже можно куда-то выкладывать или сдавать работу.
  3. Если осталось желание или есть необходимость, начинайте оптимизировать производительность. Скорее всего, этот пункт плавно перейдет в новую итерацию пункта 1.

Вы сейчас находитесь на шаге 1. Двигайтесь дальше!

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.