hwmOptimalRepairAtMarket

Цена боя и оптимальный слом на рынке

< Feedback on hwmOptimalRepairAtMarket

Review: Good - script works

§
Posted: 02.12.2023

// ==UserScript==
// @name hwmOptimalRepairAtMarket
// @author Tamozhnya1
// @namespace Tamozhnya1
// @homepage https://greasyfork.org/ru/scripts/463346
// @description Цена боя и оптимальный слом на рынке
// @version 16.1
// @encoding utf-8
// @include /^(https?:\/\/(www\.(heroeswm|lordswm)\.(ru|com)\/(auction|art_info|mod_workbench|inventory|object-info|shop|ecostat|ecostat_details|home|pl_info|\*)))/
// @grant GM_deleteValue, GM_getValue, GM_setValue, GM_log, GM_openInTab, GM.xmlHttpRequest, GM.notification
// @license MIT
// @downloadURL https://update.greasyfork.org/scripts/463346/hwmOptimalRepairAtMarket.user.js
// @updateURL https://update.greasyfork.org/scripts/463346/hwmOptimalRepairAtMarket.meta.js
// ==/UserScript==

§
Posted: 02.12.2023
Edited: 02.12.2023

исправление ошибки выше, нельзя объединить к сожалению, нужно использовать разные строчки:
// @grant GM_deleteValue
// @grant GM_getValue
// @grant GM_setValue
// @grant GM_log
// @grant GM_openInTab
// @grant GM.xmlHttpRequest
// @grant GM.notification

Tamozhnya1Author
§
Posted: 04.12.2023

Пожалуйста опишите проблему, которую решают ваши изменения.

§
Posted: 04.12.2023

придаёт стиль шапке скрипта. добавляет упоминание адреса место расположения скрипта. позволяет сократить общий объём кода и строк в скрипте.

Tamozhnya1Author
§
Posted: 05.12.2023

1) По поводу @downloadURL. Эта информация не относится к скрипту, и скрипт можете быть размещен в нескольких источниках. С установкой и обновлением хорошо справляются скрипт-менеджеры.
Вот если бы без этого скрипт не обновлялся, тогда конечно имело бы смысл поместить в него эту лишнюю информацию
2) По поводу регулярных выражений в @include. Я считаю, что это дело вкуса. Лично я предпочитаю не усложнять, где - не надо.
А вот огромным минусом регулярного выражения в @include будет то, что скрипт не будет идентифицироваться, как относящийся к сайту heroeswm.ru:
https://greasyfork.org/ru/scripts/by-site/heroeswm.ru

§
Posted: 05.12.2023
Edited: 05.12.2023

https://greasyfork.org/ru/scripts/14055-gn-market/code
https://greasyfork.org/ru/scripts/14055
Работает на heroeswm.ru, 178.248.235.15
Обновлён 20 мая 2022 г.

код:
// ==UserScript==
// @name GN_Market
// @namespace Gradient
// @description Улучшенный рынок
// @include /^https{0,1}:\/\/(www\.heroeswm\.ru|178\.248\.235\.15)\/auction\.php.*/
// @exclude /.+cat=(res|elements|skeletons|dom|cert|part).*/
// @version 1.0.12
// ==/UserScript==

@downloadURL не нужен. я бы удалил эту строку вообще. эта строка - является частью твоего кода. поэтому я здесь её и оставил.

// @homepage https://greasyfork.org/ru/scripts/463346
по этой строке пользователь смотрит страницу js и его описание.

§
Posted: 05.12.2023
Edited: 05.12.2023

правка:
@downloadURL не нужен. это добавил в коде мой плагин тамперманки при обновлении твоего скрипта на следующую версию. в итоге я перепутал источники.

везде в js указывать 178\.248\.235\.15 теперь не требуется. ибо такого домена нет.
https://www.heroeswm.ru/ob-igre
сейчас известно 3 адреса игры: www.heroeswm.ru, my.lordswm.com, www.lordswm.com

в текущей версии кода 16.3 в шапке скрипта есть лишние пробелы. их надо удалить. и не хватает пары строк:

1) // @homepage https://greasyfork.org/ru/scripts/463346

2) // @include /^(https?:\/\/(www\.(heroeswm|lordswm)\.(ru|com)\/(auction|art_info|mod_workbench|inventory|object-info|shop|ecostat|ecostat_details|home|pl_info|\*)))/
это пример. определи сам какие адреса страниц гвд тебе нужны для работы кода, и пропиши их тут.

3) // @exclude /^https:\/\/((www|my)\.(heroeswm|lordswm)\.(ru|com))\/(home|pl_info|inventory|sklad_info|arts_arenda|mod_workbench|house_info)\.php.*/
это пример. определи страницы гвд, где ты не хочешь запускать код никогда. например war, auction и другие.
обрати внимание что я указал оба зеркала игры.
и версию зеркала игры my.lirdswm.com -> "my." отличает его от прочих доменов игры.

§
Posted: 09.12.2023
Edited: 09.12.2023

"А вот огромным минусом регулярного выражения в @include будет то, что скрипт не будет идентифицироваться, как относящийся к сайту heroeswm.ru"
почему не видишь мой ответ на это? посмотри выше.

просьба в любые js добавлять:

- строку адреса к скрипту. например:
// @homepage https://greasyfork.org/ru/scripts/463346

- строку включающую адреса страниц, где должен работать скрипт
// @include /^https{0,1}:\/\/((www|qrator|my)\.(heroeswm|lordswm)\.(ru|com)|178\.248\.235\.15)\/(war|warlog|leader_guild|leader_army|inventory).php(?!.?setkamarmy)/

- строку с адресами страниц, где скрипт работать НЕ должен
// @exclude /^https{0,1}:\/\/((www|qrator)\.(heroeswm|lordswm)\.(ru|com)|178\.248\.235\.15)\/(login|war|cgame|campaign|frames|chat|chatonline|ch_box|chat_line|ticker|chatpost|rightcol|brd|frames)\.php.*/
спасибо.

Post reply

Sign in to post a reply.