Как правильно удалить часть строки?

Question

[Сергей Беловенцев]

Есть строка

odborka_vecher_01.jpg,%,podborka_vecher_37.jpg

причем ,%, может и не стоять перед переменой, к примеру если строка имеет такой вид podborka_vecher_37.jpg.
Есть переменая

var string = podborka_vecher_37.jpg

Надо в случае если перед podborka_vecher_37.jpg стоит ,%, удалить вместе с ,%,, а если нет то удалить просто строку. Подскажите как это лучше сделать?

Answer 1

[Алексей Немиро]

Раз подстроки разделены запятыми, то проще всего получить массив с помощью функции split, перебрать элементы циклом, собрать новый массив без ненужных данных и потом склеить с помощью join:

// входящие данные
var str = 'odborka_vecher_01.jpg,%,podborka_vecher_37.jpg';
// разбиваем на массив по запятой
var arr = str.split(',');
// фильтруем
var resultArr = [];
for (var i = 0; i < arr.length; i++)
{
  if (arr[i] != '%')
  {
    // добавляем в результат
    resultArr.push(arr[i]);
  }
}
// склеиваем новый массив в строку
var result = resultArr.join(',');
// выводим полученный результат в консоль
console.log(result);

Более короткий вариант и без join:

str.split(',').forEach(function(item) {
  if (item != '%') { 
    if (result != '') { result += ','; }
    result += item; 
  }
});

console.log(result);

Еще короче, с использованием filter:

var arr = str.split(',').filter(function(item) {
  return item != '%';
});

var result = arr.join(',');

В качестве альтернативного решения, обычный поиск и замена:

var result = str.replace(/,\%,/g, '');
// с учетом того, что % может быть первым или последним
// str.replace(/(\,*)\%(\,*)/g, '')

Comments

[Сергей Беловенцев]

не много не так я получаю строку `odborka_vecher_01.jpg,%,podborka_vecher_37.jpg`
но потом, условно говоря при нажатии, "удалить" мне надо удалить крайний элемент. Но элемент может оказаться первым и единственным тогда строка выглядит так `podborka_vecher_37.jpg`, а если он не первый и единственный то выглядит вот так `odborka_vecher_01.jpg,%,podborka_vecher_37.jpg` как мне удалить podborka_vecher_37.jpg и плюс разделитель вот этот `,%,`, если он есть.
Причем удаляемая строка хранится в переменой
`var string = "podborka_vecher_37.jpg"`

[Алексей Немиро]

Сергей Беловенцев: если делать разбивку на массив и в строке будет только один элемент, то в результате останется тоже, что и было на входе.

Если нужно брать только первый элемент, то просто делать разбивку на массив и выбирать его (индексация начинается с нуля):

var str = 'odborka_vecher_01.jpg,%,podborka_vecher_37.jpg';
var result = str.split(',')[0];
alert(result);

// это также будет работать, если в строке будет один элемент
str = 'odborka_vecher_01.jpg';
result = str.split(',')[0];
alert(result);

// или строка будет пустой
str = '';
result = str.split(',')[0];
alert(result);

https://jsfiddle.net/alekseynemiro/adm4v91c/

Если нужно будет обрабатывать ситуацию, когда в строке только один элемент и его требуется удалить, то можно просто проверить размер массива, если размер равен единице (1), то считать, что элементов больше нет и возвращать пустую строку:

// входящая строка
var str = 'odborka_vecher_01.jpg';
// результат по умолчанию - пустая строка
var result = '';
// если элементов больше одного
if (str.split(',').length > 1)
{
  // берем первый элемент
  result = str.split(',')[0];
}
// показываем результат
alert(result);

Если нужно реализовать удаление элементов с конца и на 100% известно, что все элементы в строке разделяются подстрокой ",%," (когда их больше одного), то проще использовать функции lastIndeOf и substr:

[Алексей Немиро]

Похоже я кривой HTML в комментарии сделал и последняя часть обрезалась. Не буду повторно писать.

Вот пример удаления последнего элемента:
https://jsfiddle.net/alekseynemiro/adm4v91c/1/