Remover Última Vírgula de uma sequência

Usando JavaScript, como posso remover a última vírgula, mas apenas se a vírgula for o último caractere ou se houver apenas espaço em branco após a vírgula? Este é o meu código. Eu tenho um violino de trabalho . Mas tem um 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;
}

Isso removerá a última vírgula e qualquer espaço em branco depois dela:

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

Ele usa uma expressão regular:

• A /marca do início e do fim da expressão regular

• A ,corresponde a vírgula

• Os \scaracteres de espaço em branco médio (espaço, tabulação etc.) e o *meio 0 ou mais

• O $no final significa o fim da string

você pode remover a última vírgula de uma string usando o método slice (), encontre o exemplo abaixo :

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

Aqui está um exemplo

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+)?$/, '');   
}

tiro longo aqui

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);

A resposta muito votada remove não apenas a vírgula final, mas também os espaços a seguir. Mas remover os seguintes espaços não era o que fazia parte do problema original. Assim:

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

https://jsfiddle.net/dc8moa3k/

O problema é que você remove a última vírgula da sequência, não a vírgula se for a última coisa na sequência. Portanto, você deve colocar um if para verificar se o último caractere é ',' e alterá-lo, se for.

EDIT: É realmente tão confuso?

'Isso é uma string aleatória'

Seu código localiza a última vírgula da string e armazena apenas 'This', porque a última vírgula é depois de 'This' e não no final da string.

você pode remover a última vírgula:

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

Remova a última vírgula. Exemplo de trabalho

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>

Primeiro, deve-se verificar se o último caractere é uma vírgula. Se existir, remova-o.

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

NOTA str.indexOf (',', this.length - ','. Length) pode ser simplificado para str.indexOf (',', this.length - 1)