Javascript-Funktion zum Vergleich zwei tausend Trennzeichen Nummer funktioniert nicht

Question

Ich versuche, eine Komparatorfunktion in Javascript zu schreiben, die zwei Zahlen wie '100,000', '200,000' usw. vergleicht und ihre Differenz zurückgibt. Ich will das netz ag in der Sortier eingebauten Brauch verwenden, die von ‚comparator‘ -Funktionalität hier für die Spalte Datum versehen: https://www.ag-grid.com/javascript-grid-sorting/

Mein Code ist wie folgt:

function thousandsComparator(Num1, Num2) { 
    var Number1 = thousandsToComparableNumber(Num1); 
    var Number2 = thousandsToComparableNumber(Num2); 

    if (Number1===null && Number2===null) { 
     return 0; 
    } 
    if (Number1===null) { 
     return -1; 
    } 
    if (Number2===null) { 
     return 1; 
    } 

    return Number1 - Number2; 
} 

function thousandsToComparableNumber(num) { 
    var total = parseFloat(num.replace(/,/g, '')); 
    return total; 

} 

Aber es funktioniert nicht. Es sortiert die Spalte nicht! ->{headerName: "ORDER Qty", field: "ORDER_QTY", width: 150, comparator: thousandsComparator, unSortIcon: true, cellStyle:{"text-align":"right"}},

Die Spaltendaten ist wie dieser "100.000", "200000" usw.

Answer 1

an Ihrem Plunker Suchen Sie sind dateComparator, die, wie es steht Komma berücksichtigt nicht statt. Sie haben date1, date2) als Argumente zu dateComparator, aber verwenden Sie date1Number/date2Number innerhalb.

Sie müssen dateComparator ändern:

function dateComparator(date1Number, date2Number) { 

    if (date1Number === null && date2Number === null) { 
     return 0; 
    } 
    if (date1Number === null) { 
     return -1; 
    } 
    if (date2Number === null) { 
     return 1; 
    } 

    return thousandsToComparableNumber(date1Number) - thousandsToComparableNumber(date2Number); 
} 

Dies wird Ihr Problem zu beheben, aber eine bessere Lösung wäre es, die Daten im Rohzustand zu verlassen und einen cellrenderer verwenden Kommas als seperators zu haben. die folgenden für Ihre COLDEF

Mit anderen Worten, entfernen Sie die insgesamt dateComparator und haben:

{ 
    headerName: "ORDER Qty", 
    field: "ORDER_QTY", 
    width: 150, 
    cellStyle: { 
     "text-align": "right" 
    }, 
    cellRenderer: function (params) { 
     return Number(params.value).toLocaleString(); 
    } 
} 

Answer 2

Sie haben einen Tippfehler in Ihrem Code, und ich empfehle Ihnen nulls zur Unterstützung in thousandsToComparableNumber

function thousandsComparator(Num1, Num2) { 
    var Number1 = thousandsToComparableNumber(Num1); 
    var Number2 = thousandsToComparableNumber(Num2); 

    if (Number1===null && Number2===null) { 
     return 0; 
    } 
    if (Number1===null) { 
     return -1; 
    } 
    if (Number2===null) { 
     return 1; 
    } 

    return Number1 - Number2; // there was a typo --> return Number1 - 2Number2; 
} 

function thousandsToComparableNumber(num) { 

    if (num === null){ 
     return null; 
    } 

    var total = parseFloat(num.replace(/,/g, '')); 
    return total; 

} 

Answer 3

Vielleicht sollten Sie versuchen, zu ersetzen:

Rückgabe Nummer1 - 2Number2;

mit:

Rückkehr Number1 - Number2;

Answer 4

Sie müssen sich für null überprüfen, bevor Sie thousandsToComparable nennen, weil null keine .replace() Methode hat, so wird es einen Fehler, wenn Sie versuchen, null zu konvertieren.

function thousandsComparator(Num1, Num2) { 

    if (Num1===Num2) { 
     return 0; 
    } 
    if (Num1===null) { 
     return -1; 
    } 
    if (Num2===null) { 
     return 1; 
    } 

    var Number1 = thousandsToComparableNumber(Num1); 
    var Number2 = thousandsToComparableNumber(Num2); 
    return Number1 - Number2; 
}