- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
- 10
- 11
- 12
function catGetCategoryById($categoryID)
{
$categoryID = (int)$categoryID;
$q = db_query("select categoryID, name, parent, products_count, description, picture, ".
" products_count_admin, sort_order, viewed_times".
", allow_products_comparison".
", allow_products_search".
", show_subcategories_products, meta_description, meta_keywords ".
" from ".CATEGORIES_TABLE.
" where categoryID=$categoryID");
return db_fetch_row($q);
}
снова ShopScript...
пересчитал ради интереса количество полей в таблице - как и в запросе 14. вот хотел я добавить еще одно поле. хорошо, что посмотрел, как берутся данные
вообще для меня тут больше играет роль что нужно добавлять в список полей еще одно, что не очень удобно + код читабельнее со звездой все же.
Если в таблица добавятся столбцы неизвестно куда, то этот код будет работать.
Да к тому же сразу видишь где какой столбец вернулся, не надо в базу глядеть.
Негодую, не говнокод, минусую.
Если речь идёт о том, чтобы вынуть два - три поля из десяти -- конечно нужно. А если вынуть все поля, тогда банально вы гоняете по сети лишние байты текста, кроме того, я уверен, что во всех СУБД есть оптимизации выборки по скорости всех полей.
Плюс, в некоторых случаях, звёздочка является элементом проектировки. Например, если таблица представляет структурированные записи, и вы пишите запрос по выборке определённых структур, например данных в узлах циклического подграфа, при этом сама программа по коллекционированию данных ничего не знает о том, что именно может хранится в узлах.
Программист, который занимает одну позицию -- плохой программист. Проектировка -- это всегда выбор. В разных ситуациях выбор может пасть на различные решения.
плюс к этому меня, как пользователя функции вообще не должна заботить ее начинка, и если функция называется catGetCategoryById то она должна действительно взять ВСЮ категорию, а не часть данных, даже если изменилась структура БД.
Не говнокод - минуснул.
а 5% процентов меня не волнуют на самом деле, мне важно меньше кода затрагивать при добавлении функционала.
>Да к тому же сразу видишь где какой столбец вернулся, не надо в базу глядеть.
вовсе не видишь, если выборка запехана в отдельную функцию, не видную сразу же. тут уже становится проще и надежнее посмотреть непосредственно на таблицу в СУБД.
во-вторых, я как раз хочу чтобы собрание сочинений таки было доступно. если не нужно будет - я это поле из перечисления уберу.
в-третьих снова повторяю, для особо слоупочных, что в этом примере из таблицы выбираются все поля, независимо от типа поля.
и, в-четвертых, каким образом звездочка мешает ORM?
Касательно ORM и звездочки вообще молчу - RTFM, одним словом.
я согласен с этим, но тогда вообще незачем было писать универсальную функцию.
Здесь верный подход, кроме как этой куевой тучи конкотенаций.
Хотя в таких случаях её использование можно оправдать разве что ленью 🙂
А ленивые программисты пока будут повторно используя код выходить на рынок быстрее.
Чего это я вас отговариваю названия писать? Пишите, пишите.
А я за это время успею свою звёздочку продать без потери производительности.