(22.04.2020 10:29)Algor Wrote:  

(22.04.2020 10:23)Algor Wrote:  Berserker, хм, выглядит вкусно, надо переползать на 2.9.13 (я ща на 2.9.12 сижу)

Zur13, глянул только одно место, есть пара вопросов (Click to View)

1. Нет обработки отрицательных х1. Хотя, если моддер ссзб, то лучше, наверное, пусть выдаст ошибку.
2. Зачем передавать размер списка, если ты этот размер потом принудительно обнуляешь? Или просто пропущено условие и принудительного обнуления в !!SN:Mv1/0/0; не нужно?

по остальному коду: не используй пожалста, SN:G и loop'ы. Есть же нормальный !!re.

Кстати, общестратегический вопрос по функциям:
если в функцию передаются неверные параметры, что лучше делать?
а) ничего не делать, пусть вылетает ошибка и моддер чешет репу
б) молча отфильтровывать некорректные параметры и пусть моддер чешет репу
в) выводить предупреждение через IF:L/IF:M
г) выводить сообщение через спец.функцию Fun_Warning, которая на основании какого-нить значения типа Warnings=# в ini/json действует по одному из вышеперечисленных правил

2) Список это массив размера М у которого первый элемент хранит количество элементов в списке L, причем всегда M > L, так как есть нулевой элемент недоступный модеру. В данном случае initial size задает изначальный размер массива М, там кстати недоработка нужно бы так:


Далее при операциях вставки размер массива периодически увеличивается чтобы он был на 20-30 элементов больше чем размер самого списка (не при каждой вставке а с шагом по 10 элементов), при удалении наоборот уменьшается шагом по 10 элементов, плюс перекрытие (threshold) между добавлением и удалением чтобы избежать ресайзов когда модер добавляет/удаляет приграничный элемент несколько раз, тут еще надо проверить и подкрутить если что.

По ошибкам и варнингам, а) не очень пдходит, потому что у меня например есть кастомные ошибки, например попытка доступа к нулевому элементу массива/списка, сама по себе ошибку не выбросит. Я тут пошел по сценарию, что некритичные ошибки отфильтровываются, а для остальных ситуаций есть статус операции, сложно сказать что более правильно. Возможно Fun_Warning более правильное решение, но при разбивке по файлам модеру нужно будет к себе забирать не только нужный файл но и базовый файл в котором описана Fun_Warning.

Правка: если товарищ Berserker, добавит АПИ для выброса кастомных исключений, то всё станет намного проще, выбрасываем исключения для ошибок и всё, это самый правильный подход, потому что тогда модер сразу видит где он лажает. Эх если бы еще в окне ЕРМ ошибки показывали кусок стека вызовов функций ЕРМ, цены бы ему не было. Вот такое небольшое дополнение к окошку с ЕРМ ошибкой намного всё упростило бы: