jQuery Grid Plugin - jqGrid - Topic: small improvment of the performance of working with jqGrid methods http://www.trirand.com/blog/?page_id=393/feature-request/small-improvment-of-the-performance-of-working-with-jqgrid-methods Simple:Press Version 5.7.5.3 tony on small improvment of the performance of working with jqGrid methods http://www.trirand.com/blog/?page_id=393/feature-request/small-improvment-of-the-performance-of-working-with-jqgrid-methods#p27966 Feature Request http://www.trirand.com/blog/?page_id=393/feature-request/small-improvment-of-the-performance-of-working-with-jqgrid-methods#p27966 Hello Oleg,

I have implemented your idea, extended to grouping too (code is in GitHub). Also i do not think that we need the code when the pin=undefined.

We can easy do (after the grid is created)

var $gird = $("#grid") and then use use it  or

var $gird = $("#grid").jqGrid({…}) when it is created.

Best Regards

Tony

]]> Fri, 28 Dec 2012 12:46:13 +0200 tony on small improvment of the performance of working with jqGrid methods http://www.trirand.com/blog/?page_id=393/feature-request/small-improvment-of-the-performance-of-working-with-jqgrid-methods#p27884 Feature Request http://www.trirand.com/blog/?page_id=393/feature-request/small-improvment-of-the-performance-of-working-with-jqgrid-methods#p27884 Oleg,

This is very interesting. Will perform tests in order to get feedback.

Best Regards

Tony

]]> Mon, 10 Dec 2012 12:27:08 +0200 OlegK on small improvment of the performance of working with jqGrid methods http://www.trirand.com/blog/?page_id=393/feature-request/small-improvment-of-the-performance-of-working-with-jqgrid-methods#p27881 Feature Request http://www.trirand.com/blog/?page_id=393/feature-request/small-improvment-of-the-performance-of-working-with-jqgrid-methods#p27881 Hello Tony,

the code of jqGrid uses internally $.jgrid.no_legacy_api=true style of wotking with methods. It menas that one find everywhere in the code the calls like

$(ts).jqGrid("addSubGridCell",...)

see the line of code for example. Even if such calles will be done inside of loops the current style of accessing to jqGrid methods not allows to save one the functions addSubGridCell in a variable and call it directly inside of the loop's body.

I suggest to introduce very simple method getMethod

$.extend($.jgrid, {
    getMethod: function (name) {
        return $.jgrid.getAccessor($.fn.jqGrid, name);
    }
}); 

which get the reference to the method of jqGrid by string parameter (if needed one can include additional test whether the type of parameter is really string). If the method is not exist the method return undefined value.

Using the method one can rewrite above call of addSubGridCell in two parts. In the fisrt one at the beginning on the function (at the beginning of addJSONData function in the example) or directly before the loop one can use

var $self = $(ts),
    addSubGridCell = $.jgrid.getMethod("addSubGridCell");

to save the needed expressions in variables and then use

addSubGridCell.call($self,...)

instead of

$(ts).jqGrid("addSubGridCell",...)

The code will stay readable, but the performance will be improved.

Additionally one can add explicit test for undefined parameter (call without parameters) in the part of code:

$.fn.jqGrid = function( pin ) {
    if (typeof pin === 'string') {
        //var fn = $.fn.jqGrid[pin];
        var fn = $.jgrid.getMethod(pin);
        if (!fn) {
            throw ("jqGrid - No such method: " + pin);
        }
        var args = $.makeArray(arguments).slice(1);
        return fn.apply(this,args);
    }
    if (pin === undefined) {
        // allow the usage of jqGrid methods in the form $("#list").jqGrid().method(params)
        // for example
        //    $("#list").jqGrid().getGridParam("colModel")
        // So one can save $("#list").jqGrid() in a variable
        //    var $self = $("#list").jqGrid();
        // and one can use simplified form of method calls,
        // exactly like in case of no_legacy_api=false:
        //    var localRowData = $self.getLocalRow(rowid);
        return $.extend({}, this, $.fn.jqGrid);
    }
    return this.each( function() {
        ... 

Best regards

Oleg

]]> Sun, 09 Dec 2012 20:58:29 +0200