Как можно сократить мой код?

Question

[Maks]

Задача кода - брать строку и заменять в ней последний параметр

Строка: matrix(1,0,0,1,0,0)
Собственно, нужно изменить последний 0 на 80
Все цифры в строке могут меняться, т.е. простым индексом выполнить задачу нельзя, но количество параметров постоянно (т.е. 6).

Я сделал следующим образом:

str = "matrix(1,0,0,1,0,0)";
obj = str.split("(")[1];
obj = obj.split(")")[0];
obj = obj.split(",");
obj[5] = "80";

objs = [];
objs.push(obj[0], obj[1], obj[2], obj[3], obj[4], obj[5]);
newStr = "matrix(" + objs.join() + ")";

Конечно, работает, но уверен, что все можно сделать гораздо проще. Подскажите как.
jsFiddle для удобства: https://jsfiddle.net/727g0Lm1/

Answer 1

[Алексей Сергей]

можно регуляркой менять:

str = "matrix(1,0,0,1,0,0)";
newStr = str.replace(/-?\d+\)/, '80)');

Comments

[Maks]

а если последний параметр отрицателен, как включить минус? Скажем matrix(1,0,0,1,0,-90)

[Алексей Сергей]

поправил, теперь и минус распознает

[Maks]

Алексей Сергей: Спасибо за решение! Идеальное сокращение

Answer 2

[Сергей Протько]

"matrix(1,0,0,1,0,0)".replace(/,\d+\)$/, ',80)');

либо если хотите менять любой индекс:

"matrix(1,0,0,1,0,0)".replace(/\(([\d\,]+?)\)$/, function (m, params) {
    params = params.split(',');
    params[5] = 80; // можно менять любой индекс

    return '(' + params.join(',') +  ')';
});

Comments

[Maks]

Спасибо за подобное решение! Чтобы учитывать минус я дополнил регулярку минусом после запятой \(([\d\,-]+?)\)$ это правильно?

[Сергей Протько]

Maks: да, это просто перечисление допустимых символов.

Answer 3

[Ivanq]

https://jsfiddle.net/Ivanq/727g0Lm1/1/

Answer 4

[Алексей Уколов]

https://jsfiddle.net/koceg/8na8fzda/1/
https://jsfiddle.net/koceg/8na8fzda/2/

Answer 5

[Виталий Инчин ☢]

var
      str = "matrix(1,0,0,1,0,0)",
      r = str.split(",")
;
 
r.splice(r.length - 1, 1, "80)");
//Или-или, как больше нравится
r[r.length - 1] ="80)";

alert(r.join(","));

Comments

[Ivanq]

arr.splice(index, 1, value) === arr[index] = value, не?

[Виталий Инчин ☢]

Ivanq: да, но так изящнее при работе с массивами.

[Ivanq]

Виталий Инчин ☢: ну, когда изменяется один параметр, мой вариант читабельнее. А то splice, length...

[Виталий Инчин ☢]

Ivanq: я просто не поклонник каши между [ и ]...

[Ivanq]

Виталий Инчин ☢: каши??? Чем [] хуже функций с несколькими параметрами?

[Виталий Инчин ☢]

Ivanq:

array[objects.perA-perB*oliluya.perC-per/88] = what.that;

или

array.f(objects.perA-perB*oliluya.perC-per/88, what.that)

Мне гораздо симпатичнее второй вариант, но это не предмет спора, а личное дело каждого.

[Ivanq]

Виталий Инчин ☢: Что не спора - это точно. Давайте галантно разойдемся.