0

Доброго времени суток!

Делаю бота для торговой площадки, который через сокет сканирует новые, добавленные вещи и скупает их если название и цена совпадают с теми что я указываю, все вроде работает, но бот не успевает купить вещь, так как проходит слишком много времени (порядка 250-300мс), не хитрым кодом я проверил время выполнения скрипта и понял что во всем виновата функция парсинга информации о вещи. Найти решение мне так и не удалось, пробовал найти альтернативу request, но не вышло. Вся надежда на вас

PS. Пробую открыть ссылку из функции в браузере, консоль показывает время выполнения 100-180мс.

Функция парсинга информации:

function getItemsInfo( classid, instanceid, infoCallback ) {
    var rOptions = {
        url: 'https://market.csgo.com/api/ItemInfo/' +classid+ '_' +instanceid+ '/ru/?key=' +apiKey
    };
    request( rOptions, function( err, resp, body ) {
        if( err && resp.statusCode !== 200 ) {
            loggerMess( 'Ошибка!: ' +resp.err, 'error' );
            return;
        }
        if( typeof( body ) === 'string' ) {
            fs.writeFileSync( 'logs/error_getItemsInfo.txt', body );
            infoCallback( JSON.parse( body ) );
        }
    } );
}
function newItems( data ) {
    var time = new Date().getTime();
    if( typeof( data ) === 'string' ) {
        data = JSON.parse( data );  
        var objData = JSON.parse( data.data );
    }
    var app = objData.app;
    if( app == 'go' ) {
        var market_hash_name = objData.i_market_hash_name;
        var classid = objData.i_classid;
        var instanceid = objData.i_instanceid;
        var uiprice = objData.ui_price; 
        for( items in OrderItems ) {
            //console.log( items );
            if( market_hash_name === OrderItems[items].name && uiprice <= OrderItems[items].price ) {
                var OItemsName = OrderItems[items].name;
                var OItemsPrice = OrderItems[items].price;
                var OItemsAutobuy = OrderItems[items].autobuy;
                getItemsInfo( classid, instanceid, function( info ) {
                    //console.log( info );
                    if( info.stickers != null ) {
                        var stickerNum = info.stickers.split( '|' );
                        if( stickerNum.length >= 1 && stickerNum.length <= 3 ) {
                            if( OItemsAutobuy === 'yes' ) {
                                buyItems( classid, instanceid, uiprice, info.hash );
                            }
                            console.log( 'M: ' +market_hash_name+ ' (' +uiprice+ ') - O: ' +OItemsName+ ' S: ' +stickerNum.length+ ' A: ' +OItemsAutobuy );
                            console.log( 'https://market.csgo.com/item/' +classid+ '-' +instanceid );
                        } else if( stickerNum.length == 4 ) {
                            buyItems( classid, instanceid, uiprice, info.hash );
                            console.log( 'M: ' +market_hash_name+ ' (' +uiprice+ ' ) - O: ' +OItemsName+ ' S: ' +stickerNum.length+ ' A: ' +OItemsAutobuy );
                            console.log( 'https://market.csgo.com/item/' +classid+ '-' +instanceid );
                        }
                        var resTime = new Date().getTime() - time;
                        loggerMess( resTime, 'info' );
                        //if( soundAlert ) { console.log( '\x07' ); }                       
                    }
                } );
            }
            if( purchased ) {
                break;  
            }
        }
    }
}

Заранее спасибо за ответ!

4
  • 1
    1. "и понял что во всем виновата функция парсинга информации о вещи" - ну так её и надо было показывать. 2. Мне кажется, что наличие fs.writeFileSync гораздо более виновато. 3. http-запрос - это не сокеты.
    – Qwertiy
    12 июл 2017 в 17:22
  • Выше же эта функция. fs.writeFileSync - без него тоже самое, добавил я ее для отлова html документа что бы сделать проверку на ошибку при использовании JSON.parse. Я знаю что не сокеты, суть в чем, бот, подключается через WebSocket к wss (апи торговой площадки), от туда получает данные типа string, потом парсит их в json-obj и уже сверяет с моим массивом (имя, цена и т.д) если совпало, то отправляется уже http-запрос для получения полной информации, если все подходит, то происходит покупка вещи. И вот он не успевает купить так как проходит много времени. 12 июл 2017 в 17:33
  • Скорее всего у тебя неэффективный алгоритм проверки надо покупать вещь, или нет.
    – Qwertiy
    12 июл 2017 в 17:35
  • Вот основная функция где должна "твориться магия" - pastebin.com/vb5vrU7G Что тут не так? 12 июл 2017 в 17:44

2 ответа 2

1

Убери fs.writeFileSync и if typeof, поставь try catch, замерь время.
Убери запрос на ItemInfo, тебе от куда нужно только hash, сделай разовые запросы для каждого предмета из твоего списка предметов и занеси в массив под каждый предмет.
Hash не меняется со временем, он статичен и уникален для каждого предмета.
Так что не вижу смысла каждый раз просить его.

Я как писал бота для отправки предметов, перед покупкой я проверял в соответствующей таблицу есть ли у меня для этого предмета hash, если не было, отправлял запрос и заносил в бд. Таким образом сокращал кол. обращений на csgo.tm

Увидел что ты добавил еще код. Убери console.log - он выполняется синхронно, используй вместо него какой-то асинхронный вывод. Пример: winston

14
  • try catch - так себе совет. try-catch не оптимизируется, точнее, не оптимизировался до недавнего времени.
    – Qwertiy
    12 июл 2017 в 17:53
  • Разве у одинакового оружия с разными стикерами, hash не меняется? 12 июл 2017 в 18:06
  • @Бесенок, На сколько я помню classId и instanceId меняется вместе со стикерами 12 июл 2017 в 18:21
  • @Qwertiy, разве try catch хуже чем писать в файл ? Можно погуглить другие проверки на валидность JSON 12 июл 2017 в 18:22
  • Так мне в любом случае нужно посылать запрос к ItemInfo что бы узнать есть ли на оружке стикер или нет 12 июл 2017 в 19:05
0
  1. Убери fs.writeFileSync
  2. Замени for( items in OrderItems ) { на нормальный цикл
  3. Не вижу, чтобы коллбэк getItemsInfo как-то использовал item (кроме критерия покупать или нет, что вообще странно) - вероятно, его не надо делать в цикле.
  4. Замени new Date() на performance.now().

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.