PHP.SU

Программирование на PHP, MySQL и другие веб-технологии

PHP.SU Портал

Главная

Помощь

Поиск

Поиск Яндекс Вакансии

Пользователи

Здравствуйте, Гость

( Вход · Регистрация · Правила форума )

Забыли пароль?

Потребление памяти против читабельности

Форумы портала PHP.SU » PHP » Программирование на PHP » Вопросы новичков (Модераторы: OrmaJever, Саныч, Строитель)

Страниц (1): [1]

Описание: Что лучше - делать код читабельным или быстрым и компактным

Поиск в теме | Версия для печати

igosja	Отправлено: 05 Марта, 2015 - 10:17:48
Гость Покинул форум Сообщений всего: 77 Дата рег-ции: Март 2013 Откуда: https://vhol.org Помог: 3 раз(а)	Скажем, есть два варианта одной функции: PHP: скопировать код в буфер обмена function f_test_1($a,$b) { return number_format(round(pow($a,2)pow($b,3)+sqrt($a+$b),0),0,","," "); } function f_test_2($a, $b) { $a_pow = pow($a, 2); $b_pow = pow($b, 3); $mul_pow = $a_pow $b_pow; $sum_a_b = $a + $b; $sqrt_sum = sqrt($sum_a_b); $sum_total = $mul_pow + $sqrt_sum; $sum_total = round($sum_total, 0); $sum_total = number_format($sum_total, 0, ",", " "); return $sum_total; } В первом случае код занимает меньше места и чуть быстрее работает, но, чем больше в него будет напихано вычислений, тем сложнее его будет читать и вносить туда правки. Во втором случае код лучше читается и поддерживается, но работает чуток медленнее и занимает больше места. Какой вариант записи кода считается более правильным? (Отредактировано автором: 05 Марта, 2015 - 10:28:29)

RickMan	Отправлено: 05 Марта, 2015 - 10:20:40
Участник Покинул форум Сообщений всего: 1033 Дата рег-ции: Май 2012 Помог: 30 раз(а)	Что мешает первый вариант разбить на пару строк и добавить пробелы (особенно после запятых)? Это добавит ему читабельности, но создавать 100500 переменных не прийдется...

igosja	Отправлено: 05 Марта, 2015 - 10:26:26
Гость Покинул форум Сообщений всего: 77 Дата рег-ции: Март 2013 Откуда: https://vhol.org Помог: 3 раз(а)	RickMan пишет: Что мешает первый вариант разбить на пару строк и добавить пробелы (особенно после запятых)? Это добавит ему читабельности, но создавать 100500 переменных не придется... Я специально вдался в крайности, чтобы вопрос был наглядней. Я так понимаю, что лучше держаться "золотой середины"?

RickMan	Отправлено: 05 Марта, 2015 - 10:29:41
Участник Покинул форум Сообщений всего: 1033 Дата рег-ции: Май 2012 Помог: 30 раз(а)	Ну мое мнение наврядле подойдет под показатель "мнение эксперта", но я бы держался золотой середины. На самом деле, те пару байт особо погоды в нынешнее время не делают, но и второй вариант рассписан прям как для слабоумных. Я бы просто в первом варианте форматирование сделал бы более приятное, в описании логики вынес бы в комментарий (аннотации) к функции.

Мелкий	Отправлено: 05 Марта, 2015 - 11:05:31
Активный участник Покинул форум Сообщений всего: 11926 Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург Помог: 618 раз(а)	igosja пишет: работает чуток медленнее и занимает больше места. Это доказано профилировщиком? Какой версии PHP? А на актуальной версии PHP? Как насчёт PHP7? Делайте код сначала читабельным (оба ваших варианта такими не являются, при том второй гораздо хуже первого именно из-за размера в LOC), а потом профилируйте. С.Макконнелл со своим совершенным кодом неплохо вправляет мозг, почитайте. Особенно в контексте этого вопроса интересен опыт автора в ручной оптимизации кода с получением сложного куска кода, который очень тяжело поддерживать, но не давшего ничего по производительности - компилятор реализовал свои оптимизации. ----- PostgreSQL DBA

igosja	Отправлено: 07 Марта, 2015 - 15:41:02
Гость Покинул форум Сообщений всего: 77 Дата рег-ции: Март 2013 Откуда: https://vhol.org Помог: 3 раз(а)	Мелкий пишет: С.Макконнелл со своим совершенным кодом неплохо вправляет мозг, почитайте. Спасибо, скачал книгу. Пробежался глазами - действительно вправляет мозг. Буду читать ее основательно. (Отредактировано автором: 07 Марта, 2015 - 15:41:39)

Поиск в теме | Версия для печати

Страниц (1): [1]

Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)

« Вопросы новичков »

Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.