Merlin777
Есть такое выражение: "В погоне за улучшениями есть риск ухудшить то, что уже и так хорошо."
Изменение, которое внесли ChatGPT и ваш "джун" ("теперь ошибки  при отсутствии интернета, обрыве соединения и т.п. не выводятся пользователям на страницу,  а пишутся в консоль браузера") является вредным и неразумным. Большинство пользователей не в курсе о существовании консоли браузера и как та открывается. Консоль больше нужна разработчикам, чем пользователям, в скрипте нет отладочной информации. Вывод ошибок, связанный с отсутствием интернета, обрывом соединения и подобного ChatGPT связал с недоступностью API, что логично, но неверно интерпретировано. Пользователь, после обновления страницы, и сам будет видеть, что у него нет соединения с интернетом, в отличие от вероятных проблем с недоступностью API, о чем пользователь может быть не в курсе, ожидая момент, когда загрузится список тем. Всю отладочную информацию я специально убрал из скрипта для облегчения кода, сделав вывод ошибки во встроенный в сервис jGrowl (для админов), а в случае каких-либо проблем с jGrowl или jQuery, вывод ошибок для всех дублируется в блок вывода списка тем. А так вы лишили своих (и чужих) пользователей информации о возможных ошибках.

Я добавил "костыль" сервисного API в виде увеличения лимита по той причине, что не знаю как одним запросом вывести только выделенные темы (а лучше одним запросом ‒ сначала выделенные, а за ними все остальные). Если в нашем API такое возможно, то ребята подскажут как это сделать. Насколько сильно увеличится нагрузка при увеличении лимита, сказать сложно, но это явно не в пользу оптимизации. Сейчас запрос работает таким образом: он отсортировывает темы текущего форумИд по дате последнего сообщения по убыванию. Затем в процесс впрягается скрипт, вытягивая сначала выделенные темы (sticky: "1"), а затем все остальные. Есть большая вероятность, что при одновременном накоплении тем в разделе в количестве более 100 штук, одна или несколько выделенных тем, созданных в далеком прошлом, могут выпасть из списка до тех пор, пока в них не появится новое сообщение. Выделенные темы могут висеть закрытыми годами (такие темы являются правилами разделов, они созданы давно и закрыты от написания ответов). А вот обычные темы в разделе копятся. С увеличением лимита мы лишь временно купировали возникновение ошибки. Именно поэтому необходимо поработать с изменением запроса, если это возможно. В ином случае придется вновь возвращаться к парсингу.

У параметра sort_by нет параметра limit, оба параметра взаимосвязаны методами их использования.

Верно отметил Alex_63 про глобальные переменные. Я переписывал код по образу кода Fover, и уже после опубликования скрипта возникла та же мысль за переменную count ‒ из-за её значимого названия она много где может использоваться, однако я не стал её переименовывать, потому как за продолжительное время существования скрипта ни у кого не возникло проблем (но это не точно). Лучшим решением в данном случае будет сгруппировать все параметры в одном объекте, это позволит вынести логику скрипта в файл, а параметры настройки оставить рядом. Тогда переменные не пересекутся с другими скриптами, соответственно от использования анонимной функции можно отказаться, и наш код дозволяется использовать как в файле с отдельным выносом настроек, так и совместно, не пряча настройки скрипта в глубине.

Рекомендую также обновить стиль скрипта: ещё в прошлый период (через 2 дня после публикации) были внесены поправки для обеспечения корректного отображения на портативных устройствах.

Частично использовал логику вашего кода с кэшированием в 10 минут.
В случае отсутствия настроек скрипта добавил в логику стандартные значения (count = 10, useSession = 0, selectAct = 'click').

Обновил скрипт в теме Скрипты от пользователей 3

Отредактировано Reysler (Пт, 13 Сен 2024 11:15:19)

Merlin777
Хорошие новости
Благодаря нашим мегаребятам ‒ @kernel и @Alex_63, удалось сократить код, подобрав правильный запрос для API, тем самым теперь избегая ненужной сортировки по выделенным и обычным темам в скрипте, а это значит, что можно со спокойной душой урезать бессмысленные огромные лимиты до необходимого нам количества выводимых тем. Большущий поклон ребятам Мы движемся в правильном направлении!
Добавил классы "tid{topic.id}" для возможного индивидуального оформления ссылки в списке тем (не знаю зачем, пусть будут )
Немного переработал логику скрипта, теперь всё работает именно так, как и задумывалось, уряяяяяя

Скрипт обновлён в файле Скрипты от пользователей 3

Для быстрого обновления кэша у всех пользователей следует добавить (или увеличить цифру на 1) к окончанию файла (выделенное красным):

Уважаемые Reysler и Alex_63,

на моем форуме та же проблема с отображением "Последних N тем форума".

Напишите, пожалуйста, готовый код, заменяющий старый скрипт.

Форум:   https://biorezonans.3bb.ru

Alex_63
Прошу прощенья, неверно поняла запрос, для которого Reysler сделал код. Оказывается, моя проблема совсем другая. У меня обрезаются названия тем в списке "Последних тем форума" (не в описании). Причем справа много свободного места.
Если можно, посоветуйте что-нибудь.

Отредактировано Dango (Вт, 17 Сен 2024 22:31:09)

Принял замечания и в свободное время значительно переработал скрипт Первые N тем в описании форума (новая версия)

Из интересного: скрипт на модульной основе; оптимизировал загрузку там, где это требуется; переработал систему случайного изменения параметров скрипта, чтобы это не привело к полной его поломке; появился перевод для английской версии форума (переключается автоматически при изменении языка в настройках форума или профиле);

добавил форму в профиль (Профиль - Отображение) для индивидуального изменения настроек скрипта с сохранением (ключи настроек хранятся на сервере форума), то есть ваши пользователи смогут выбрать удобный для них вариант отображения (click или hover) и количество тем (от 1 до 25);

и прочие изменения.
Текстовые настройки скрипта, которые вы указываете в Администрирование - Формы, имеют пониженный приоритет перед индивидуальными настройками профиля пользователя ‒ они скорее для гостей и пользователей, ещё не сохранивших настройки.

Прошу тестировать (никак не могу привыкнуть к кодировке страниц на сервисе ‒ требуется посмотреть на разных устройствах, заодно узнаем, как сервис работает с модульными скриптами). Желающих детально разобраться со скриптом я оставил неминимизированные версии (90901.js-90904.js ‒ постарался сохранить нарицательные именования функций и объекты, вроде даже осталась какая-то отладочная информация).

Выглядит весьма интересно, и местами даже непонятно  

Хочу попросить, по возможности,  ещё добавить:

1) В настройках, кроме количества тем и способа отображения, сделать галочку "не показывать список тем" - для ретроградов    А то не всем юзерам это по душе

2) Добавить в меню выбора вариантов отображения показ не только по картинке, но и по стрелочке ">" (как было раньше).
В идеале,  сделать  блоки настроек и для десктопной и для мобильной версии (с выбором по наведению, по картинке и со стрелочкой) 

3) Сделать переменную для отключения алертов о проблемах при работе скрипта  (я до сих пор считаю, что алерты совершенно бессмысленны для "простых" юзеров)   

P.S. Поставил скрипт себе на форум, работает хорошо и заметно стабильнее прошлых версий.

Отредактировано Merlin777 (Пт, 20 Сен 2024 03:37:49)

@Romych
Up!

На самом деле это может реагировать на "тормозящий" интернет, когда нет связи по несколько секунд, а потом она возникает. Не только может, но и реагирует   Впрочем, анализировать ещё и это, в данном случае уже наверное чрезмерно 

У меня один товарищ пожаловался, что его браузер неправильно воспринимает вложенные определения, после разбиения на два отдельных .clickt и .clickt img заработало как положено.

.clickt {
    cursor: pointer;
    user-select: none;
        }
.clickt img {
        border: 0;
        line-height: 1;
        max-height: 16px;
        max-width: 16px;
        vertical-align: middle;
            }

Merlin777
Скрипт регулирует время ответа сервера, чтобы клиент бесконечно не ждал загрузку, в надежде получить данные с недоступного сервера. Может быть эта проверка не нужна, вероятно в будущем её уберу. А пока сделал дополнительные 2 попытки переподключения к API, рекурсивно увеличив время по таймауту. Добавил параметр touchDevice, позволяющий администраторам активировать принудительный вариант показа блока 'click' для пользователей с сенсорными устройствами (работает независимо от выбранного пользователем типа открытия блока у себя в профиле). Добавил возможность индивидуального отключения скрипта в профиле.
Синтаксис вложенной структуры CSS стал поддерживаться относительно недавно ‒ почти во всех современных браузерах с начала текущего года. Кстати, у вашего товарища и цветоделение тем (важные, закрытые) не должны были работать, они на той же nested-конструкции. Изменил в теме Скрипты от пользователей 3 стили под стандартную конструкцию.
Благодаря пользователям, протестировали на нескольких форумах, выловили все возможные ошибки. За последние несколько дней, во время пути, в двух частях регионов Сибири тестировал на своих мобильных операторах с E-шкой, так и не удалось словить ошибку с "тормозящим" интернетом (а на трассе ловит крайне плохо и периодически пропадает). Ошибку удалось искусственно воспроизвести дома, значительно урезав скорость канала с отключенным кэшем браузера и задержками, практически попадающими под условия полного дисконнекта, в таком положении загрузка страницы по времени существенно увеличилась. В любом случае необходимо тестировать, смотреть. При возникновения подобной ошибки во всплывающем окне, которое видят только администраторы, при нажатии на ссылку "Error Log", появится отладочная информация прямиком из консоли браузера ‒ сделано это как раз с целью, чтобы мало-мальски разбирающийся администратор мог принести ошибку на форум поддержки для изучения и исправления.
PS: Во всех старых браузерах много чего не будет работать Именно поэтому следует своевременно обновлять браузеры, технологии не стоят на одном месте.