Rimuovi Last Comma da una stringa

Utilizzando JavaScript, come posso rimuovere l'ultima virgola, ma solo se la virgola è l'ultimo carattere o se c'è solo uno spazio bianco dopo la virgola? Questo è il mio codice Ho un violino funzionante . Ma ha un bug.

var str = 'This, is a test.'; 
alert( removeLastComma(str) ); // should remain unchanged

var str = 'This, is a test,'; 
alert( removeLastComma(str) ); // should remove the last comma

var str = 'This is a test,          '; 
alert( removeLastComma(str) ); // should remove the last comma

function removeLastComma(strng){        
    var n=strng.lastIndexOf(",");
    var a=strng.substring(0,n) 
    return a;
}

Questo rimuoverà l'ultima virgola e tutti gli spazi bianchi dopo di essa:

str = str.replace(/,\s*$/, "");

Usa un'espressione regolare:

• Il /segno l'inizio e la fine dell'espressione regolare

• L' ,corrisponde la virgola

• I \scaratteri spazi bianchi (spazio, tab, ecc.) E i *mezzi 0 o più

• Il $alla fine indica la fine della stringa

puoi rimuovere l'ultima virgola da una stringa usando il metodo slice (), trova l' esempio seguente :

var strVal = $.trim($('.txtValue').val());
var lastChar = strVal.slice(-1);
if (lastChar == ',') {
    strVal = strVal.slice(0, -1);
}

Ecco un esempio

function myFunction() {
	var strVal = $.trim($('.txtValue').text());
	var lastChar = strVal.slice(-1);
	if (lastChar == ',') { // check last character is string
		strVal = strVal.slice(0, -1); // trim last character
		$("#demo").text(strVal);
	}
}

<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>


<p class="txtValue">Striing with Commma,</p>

<button onclick="myFunction()">Try it</button>

<p id="demo"></p>

function removeLastComma(str) {
   return str.replace(/,(\s+)?$/, '');   
}

colpo lungo qui

var sentence="I got,. commas, here,";
var pattern=/,/g;
var currentIndex;
while (pattern.test(sentence)==true)  {    
  currentIndex=pattern.lastIndex;
 }
if(currentIndex==sentence.trim().length)
alert(sentence.substring(0,currentIndex-1));
else
 alert(sentence);

La risposta molto votata rimuove non solo la virgola finale, ma anche tutti gli spazi che seguono. Ma rimuovere quei seguenti spazi non era ciò che faceva parte del problema originale. Così:

let str = 'abc,def,ghi, ';
let str2 = str.replace(/,(?=\s*$)/, '');
alert("'" + str2 + "'");
'abc,def,ghi '

https://jsfiddle.net/dc8moa3k/

Il problema è che rimuovi l'ultima virgola nella stringa, non la virgola se è l'ultima cosa nella stringa. Quindi dovresti inserire un if per verificare se l'ultimo carattere è ',' e cambiarlo se lo è.

EDIT: è davvero così confuso?

"Questa è una stringa casuale"

Il tuo codice trova l'ultima virgola dalla stringa e memorizza solo 'This', perché l'ultima virgola è dopo 'This' non alla fine della stringa.

puoi rimuovere l'ultima virgola:

var sentence = "I got,. commas, here,";
sentence = sentence.replace(/(.+),$/, '$1');
console.log(sentence);

Rimuovi l'ultima virgola. Esempio funzionante

function truncateText() {
  var str= document.getElementById('input').value;
  str = str.replace(/,\s*$/, "");
  console.log(str);
}

<input id="input" value="address line one,"/>
<button onclick="truncateText()">Truncate</button>

Innanzitutto, si dovrebbe verificare se l'ultimo carattere è una virgola. Se esiste, rimuovilo.

if (str.indexOf(',', this.length - ','.length) !== -1) {
    str = str.substring(0, str.length - 1);
}

NOTA str.indexOf (',', this.length - ','. Length) può essere semplificato in str.indexOf (',', this.length - 1)