200 рублей спасут Отца Демократии?

[Ritsuka]

Помните сценку из строго любимого фильма? Все в этом мире повторяется.

 

В 3.0.3 был введен лимит в 50 BBCode тегов на один форумный пост. Его собственноручно вкрутил сам великий и ужасный автор форума bfarber. Дальше началось шоу. Совершенно понятно, что 50 тегов на один пост - лимит слишком маленький для больших досок. Посыпались варианты и предложения в стиле "кто больше?". И вот, в версии 3.0.4 нам как бы говорят: "а 500 постов спасут ваш форум?".

 

Знаете, когда я учился программированию на C, первое, за что препод отрывал пальцы, это использование хардкодед переменных в коде. Если у тебя есть определенный много где используемый параметр (более 6(!) раз в пределах класса BBCode) - вынеси его в константы, а еще лучше, вынеси его в настройки программы! Так почему же вместо того, чтобы всюду "намертво" залепить число 500 не была вынесена константа в conf_global.php? Почему не сделана дополнительная настройка в группе "Производительность" под названием "Максимальное количество тегов одного типа на пост"? Почему не сделать эту настройку специфичной для каждой из групп пользователей, чтобы, к примеру, снимать лимит для админов полностью, а для гостей ставить минимум тегов?

 

Ответ простой: непрофессионализм + лень. Просто от нас отмахиваются. "50 не подойдет? А 500 рублей спасут Отца Демократии?". Зачем думать и стараться, главное по-быстрее отмахнуться....

 

Вывод тоже простой: рано, рано была выпущена тройка, не готовы разработчики к её развитию и даже примитивному обслуживанию. Их просто задрали, и они только и делают, что отмахиваются от назойливых пользователей, клепая заплатки как попало и куда попало.

[dengol]

Ritsuka, хорошо что есть такие люди как вы, которые что-то понимают (а не просто любители-администраторы) и дают возможность нам понять все плюсы, минусы и недостатки!

[Evrika]

Ritsuka, хорошо что есть такие люди как вы, которые что-то понимают (а не просто любители-администраторы) и дают возможность нам понять все плюсы, минусы и недостатки!

Пожалуй подпишусь под каждым словом

[Sannis]

Со временем так происходит с каждым програмным продуктом. Даже с китами. Другое дело, раньше или позже, можно было бы развивать 2.3.х дольше, чем проживёт тройка.

[WildRAID]

Из class_attach:

			foreach( $rows as $_attach_id => $row )
		{
			//-----------------------------------------
			// INIT
			//-----------------------------------------

			$row = $rows[ $_attach_id ];

 

Legacy-код зажигает конечно.)

[Alex/AT]

Мда, жестоко. Интересно, как можно было до такого додуматься. На ум приходит только одно - было какое-то перечисление индексов без foreach или из другого массива, строчку убрать тупо забыли.

[Ritsuka]

Мое любимое - это "private function _learnIP()" на 1057 строке в файле admin/applications/members/modules_admin/members/tools.php. Просто просмотрите все от начала и до конца.

 

Кстати, оно не работает в 3.0.4, вполне резонно выдает ошибку:

mySQL query error: SELECT * FROM ibf_members WHERE member_id IN (author_id)

SQL error: Unknown column 'author_id' in 'where clause'
SQL error code: 1054
Date: Sunday 01st o  November 2009 04:00:08 PM

[saman]

Знаете, когда я учился программированию на C, первое, за что препод отрывал пальцы, это использование хардкодед переменных в коде. Если у тебя есть определенный много где используемый параметр (более 6(!) раз в пределах класса BBCode) - вынеси его в константы, а еще лучше, вынеси его в настройки программы! Так почему же вместо того, чтобы всюду "намертво" залепить число 500 не была вынесена константа в conf_global.php? Почему не сделана дополнительная настройка в группе "Производительность" под названием "Максимальное количество тегов одного типа на пост"?

тебя услышали

в версии 3.1 вынесут этот параметр в настройки, так bfarber написал:

 

Added a setting for 3.1 (default to 2000).

[rozalion]

Мда, жестоко. Интересно, как можно было до такого додуматься. На ум приходит только одно - было какое-то перечисление индексов без foreach или из другого массива, строчку убрать тупо забыли.пенопласт

Скорее всего так и произошло!