Supprimer la dernière virgule d'une chaîne

En utilisant JavaScript, comment puis-je supprimer la dernière virgule, mais seulement si la virgule est le dernier caractère ou s'il n'y a qu'un espace blanc après la virgule? Ceci est mon code. J'ai un violon qui fonctionne . Mais il y a 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;
}

Cela supprimera la dernière virgule et tout espace blanc après:

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

Il utilise une expression régulière:

• La /marque le début et la fin de l'expression régulière

• Les ,correspond à la virgule

• Les \scaractères d'espaces moyens (espace, tabulation, etc.) et les *moyens 0 ou plus

• Le $à la fin signifie la fin de la chaîne

vous pouvez supprimer la dernière virgule d'une chaîne en utilisant la méthode slice (), trouvez l' exemple ci-dessous :

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

Voici un exemple

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

long shot ici

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 réponse fortement votée supprime non seulement la virgule finale, mais également tous les espaces qui suivent. Mais la suppression des espaces suivants ne faisait pas partie du problème initial. Alors:

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

https://jsfiddle.net/dc8moa3k/

Le problème est que vous supprimez la dernière virgule de la chaîne, pas la virgule si c'est la dernière chose de la chaîne. Vous devriez donc mettre un if pour vérifier si le dernier caractère est «,» et le changer si c'est le cas.

EDIT: Est-ce vraiment déroutant?

'Ceci, est une chaîne aléatoire'

Votre code trouve la dernière virgule de la chaîne et stocke uniquement «Ceci», car la dernière virgule est après «Ceci» et non à la fin de la chaîne.

vous pouvez supprimer la dernière virgule:

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

Supprimez la dernière virgule. Exemple de travail

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>

Tout d'abord, il faut vérifier si le dernier caractère est une virgule. S'il existe, supprimez-le.

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

REMARQUE str.indexOf (',', this.length - ','. Length) peut être simplifié en str.indexOf (',', this.length - 1)