Почему возвращаемый объект функции orm ограничен 100? - PullRequest
1 голос
/ 22 октября 2019

В проекте Node.js я использую oracle-orm-zn для связи с моей базой данных Oracle.

В какой-то момент я вызываю функцию findAllandCount(), чтобы получитьвсе записи в таблице:

...
tipm.findAllandCount(params).then(function (r) {
    if (r.all > 0) {
        locals.data.righe = r.rows;

        console.log("+allRows:", r.all);
        console.log("+NumRows:", r.rows.length);
    }
...

Где params - Объект:

// var params = { attributes: viewOrder, order: sortOrder, limit: 9999 };
var params = { attributes: viewOrder, order: sortOrder};

Как указано в findAllandCount () документация без ограничения (илис пределом = 9999) Я должен получить все записи таблицы! Вместо этого я получаю только 100 записей, как я вижу из оператора console.log("+NumRows:", r.rows.length).

Оператор console.log("+allRows:", r.all) дает мне правильное количество записей.

Я подумал, что проблема заключается вв findAllandCount() функции модуля, то есть:

Model.prototype.findAllandCount = function(options){
    var pro = Q.defer(), self = this;
    var sql_count = 'select count(*) as count from ' + self.table_name;
    var sql = 'select * from (select rownum rownum_,t.* from ', start = 0, end = 100;
    if(!options){
        sql += self.table_name+ ' where rownum <= ' + end ;
    }else{
        sql += '(';
        var _col = '', _tab = '', _where = '';
        var makeCTW = function(opt, par_tn){
            if(!opt.model ) { pro.reject(ecode.lack_para());return pro.promise;}
            var sub_tn = opt.model.table_name;
            //make column sql (has ',' last)
            if(opt.attributes && opt.attributes.length){
                for(var i=0;i<opt.attributes.length;i++){
                    _col += par_tn? sub_tn+'.'+opt.attributes[i]+' as '+sub_tn+'#'+opt.attributes[i]+',' : sub_tn+'.'+opt.attributes[i]+',';
                }
            }else{
                if(par_tn){
                    for(var colkey in opt.model.col_def){
                        _col += sub_tn+'.'+colkey+' as '+sub_tn+'#'+colkey+',';
                    }
                }else
                    _col += sub_tn+'.*,';
            }
            //make table sql (has ',' front)
            if(opt.notRequired && par_tn){
                if(!opt.use||!opt.on){ pro.reject(ecode.lack_para());return pro.promise;}
                _tab += ' left join '+sub_tn+' on '+par_tn+'.'+opt.use+'='+sub_tn+'.'+opt.on;
            }else{
                _tab += ','+sub_tn;
            }
            //make where sql (has ' and ' last)
            if(opt.where){
                _where += lib.whereObjToSqlStr(opt.where,opt.model.col_def,sub_tn)+' and ';
            }
            if(!opt.notRequired && par_tn){
                if(!opt.use||!opt.on){
                    pro.reject(ecode.lack_para());return pro.promise;
                }
                _where += par_tn+'.'+opt.use+'='+sub_tn+'.'+opt.on+' and ';
            }
            if(opt.include && opt.include.length){
                for(var j= 0; j<opt.include.length; j++) makeCTW(opt.include[j],sub_tn);
            }
        };
        options.model = self;
        makeCTW(options);
        sql += 'select '+_col.substring(0,_col.length-1)+' from '+_tab.substring(1,_tab.length)+
            (_where==''?'':(' where '+_where.substring(0,_where.length-4)))+
            ') t where rownum<='+end+' ) where rownum_>='+start;
        if(options.order){
            if(typeof(options.order) == 'string') sql += ' order by '+options.order;
            else if(options.order.length){
                sql += ' order by ';
                for(var i= 0,l= options.order.length;i<l;i++) sql += options.order[i]+',';
                sql = sql.substring(0,sql.length-1);
            }
        }
    }
    self.orm.getCon().then(function(con){
        var _pro = Q.defer();
        con.execute(sql_count,{},self.orm.opt,function(e,r){
            if(e) _pro.reject({c:con,e:'FindAllandCount count error: '+ e.message});
            else _pro.resolve({c:con,r: r.rows[0].COUNT});
        });
        return _pro.promise;
    }).then(function(res){
        var con = res.c, count = res.r;
        var _pro = Q.defer();
        con.execute(sql,{},self.orm.opt,function(e,r){
            if(e) _pro.reject({c:con,e:'FindAllandCount select error: '+ e.message});
            else _pro.resolve({c:con,r: r.rows.length? r.rows: null, count: count});
        });
        return _pro.promise;
    }).then(function(res){
        res.c.release();
        if(res.r && res.r.length){
            for(var i= 0,l=res.r.length;i<l;i++){
                delete res.r[i].ROWNUM_;
                for(var k in res.r[i]){
                    var ks = k.split('#');
                    if(ks.length ==2 && ks[0] && ks[1]){
                        if(!res.r[i][ks[0]]) res.r[i][ks[0]] = {};
                        res.r[i][ks[0]][ks[1]] = res.r[i][k];
                        delete res.r[i][k];
                    }
                }
            }
        }
        pro.resolve({
            all: res.count,
            rows: res.r
        });
    }).catch(function(e){
        e.c.release();
        pro.reject(ecode.sql_execute(e.e));
    });
    return pro.promise;
};

И, если только не мои ошибки, кажется, что все зависит от этого end = 100. Который, кажется, не устанавливается динамически.

Это действительно так? Должен ли я открыть вопрос на странице github oracle-orm-zn?

Могу ли я провести еще несколько тестов? Есть идеи?

1 Ответ

0 голосов
/ 22 октября 2019

я думаю, что это какая-то ошибка, потому что в коде функции нет проверки для опции "limit", и они используют переменные "start" и "end" при создании запроса sql, а эти переменные нене изменяется динамически, вы можете проверить это, просто установите «предел» как случайное число

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...