opm list -remote #18
opm list -remote #18
Conversation
src/Классы/КэшПакетовХаба.os
Outdated
There was a problem hiding this comment.
Не стоит использовать стрчислострок и стрполучитьстроку. Внутри у них крайне не оптимальный алгоритм. В том числе и в родной 1С
There was a problem hiding this comment.
Лучше прочитать один раз текстовым документом или ЧтениемТекста
There was a problem hiding this comment.
@EvilBeaver ты что-то уж слишком придираешься.
Какой процент от общего выполнения заберет эта строка? 0,001? Там же хттп запрос перед этим
There was a problem hiding this comment.
Ну так @EvilBeaver и не затребовал изменений :)
В любом случае, я переделаю файл на json. Поговорили вчера с Лешей. Там по-любому будут добавляться поля, лучше сразу сделать расширяемым
There was a problem hiding this comment.
Там в первом варианте вообще темпфайл сохранялся =)
Сейчас исправил.
Как будет Json, поменяем на чтениеJson.
|
|
||
| СписокТестов = Новый Массив; | ||
|
|
||
| СписокТестов.Добавить("ТестДолжен_ПолучитьПакетыХаба"); |
src/Классы/КэшПакетовХаба.os
Outdated
There was a problem hiding this comment.
Исправил. Теперь ТекстовыйДокумент
There was a problem hiding this comment.
А потом приду я и будет #Использовать JSON :D
There was a problem hiding this comment.
@nixel2007 сначала был HTML =)
Потом пришел ты и HTML разложился в структуру.
Потом ты опять пришел и появился txt
Ждем третьего пришествия =)
artbear
left a comment
artbear
left a comment
There was a problem hiding this comment.
У меня только одно замечание по формату файла-списка.
Нужно договориться, будет ли это txt или json-формат
|
JSON, JSON. |
|
Тогда формат опиши и тогда @asosnoviy сможет доработать свой PR |
|
@artbear это PR в oscript-library:feature/list мы же на нем и договариваемся. |
|
@asosnoviy поменял base-branch. словил конфликт =/ сорри. В общем, можно ребейзится на девелопе |
|
Пока нет json мержу так. назрело. |
5 participants
#13