strReplace для нескольких значений

[demianimp]

Друзья, прошу посмотреть код и дать совет.

Появилась необходимость в строке заменить несколько символов.
Например: ОАО "Какая-то компания и сим'вол" , нужно ' и " заменить на `.

Первое, что приходит на ум это strReplace(strReplace()), но что делать если символов много? (больше одно уже много)

Я написал следующий метод в Global:

static str strReplaceContainer(str _str, container _fromStrCon, container _toStrCon)
{
    int     i;
    str     fromStr,
            toStr;
    ;
    if(conLen(_fromStrCon) != 0 && conLen(_toStrCon) != 0)
    {
        for(i = 1; i <= conLen(_fromStrCon); i++)
        {
            fromStr = conPeek(_fromStrCon, i);
            toStr   = conPeek(_toStrCon, i < conLen(_toStrCon) ? i : conLen(_toStrCon));

            _str = strReplace(_str, fromStr, toStr);
        }
    }

    return _str;
}

Благодаря этому методу можно заменить вот такой код

    strReplace(strReplace(strReplace(custTable.Name, '\"', '`'),'\'', '`'), '&', '_');

на

    strReplaceContainer(custTable.Name, ['\"', '\'', '&'], ['`', '`', '_']);

[Ruff]

Когда заменяющий символ один и тот же, можно через регулярку:

    TextBuffer tb = new TextBuffer();
    ;
    tb.setText("ООО 'Рога & копыта\"");
    tb.regularExpressions(true);

    tb.replace('["\'&]', '`');
    info(tb.getText());

[S.Kuskov]

Регулярные выражения можно использовать через Net. Если версия аксапты позволяет )

info(System.Text.RegularExpressions.Regex::Replace("ООО 'Рога & копыта\"", '["\'&]', '`'));

[demianimp]

Цитата:

Сообщение от Ruff

Когда заменяющий символ один и тот же, можно через регулярку:

    TextBuffer tb = new TextBuffer();
    ;
    tb.setText("ООО 'Рога & копыта\"");
    tb.regularExpressions(true);

    tb.replace('["\'&]', '`');
    info(tb.getText());

Цитата:

Сообщение от S.Kuskov

Регулярные выражения можно использовать через Net. Если версия аксапты позволяет )

info(System.Text.RegularExpressions.Regex::Replace("ООО 'Рога & копыта\"", '["\'&]', '`'));

Это очень сильно подходит, когда нужно заменить на что-то одно.
Мой алгоритм позволяет это сделать, а еще он позволяет заменить конкретные значения на другие значения. Например V на В, P на П и т.п.

На мой взгляд более универсальное решение, но могу ошибаться.

[Владимир Максимов]

Насколько я вижу, в постановке задачи речь идет не о регулярных выражениях, а о последовательной замене. Как "зарядка для ума" описанный метод подойдет. Но лично я бы просто написал 3 последовательных замены

str strName;

strName = custTable.Name;
strName = global::strReplace(strName, '\"', '`');
strName = global::strReplace(strName, '\'', '`');
strName = global::strReplace(strName, '&', '_');

Не вижу никакого практического смысла "вытягивать" команду замены в одну строку. И редактировать сложно и сама задача довольно специфическая. Я бы сказал, не очевидная.

Если так уж хочется написать посимвольную замену, то имеет смысл не использовать существующий метод global::strReplace(), а написать свой собственный, где в качестве параметра передаются две строки с наборами символов. Не замена текста на текст, а замена каждого символа из первого параметра на соответствующий символ из второго.

Чтобы было понятно о чем речь, есть функция ChrTran в FoxPro. Вот ее описание

https://msdn.microsoft.com/en-us/lib...(v=vs.71).aspx

Написать ее аналог в X++, думаю, будет не очень сложно

[demianimp]

Цитата:

Сообщение от Владимир Максимов

Насколько я вижу, в постановке задачи речь идет не о регулярных выражениях, а о последовательной замене. Как "зарядка для ума" описанный метод подойдет. Но лично я бы просто написал 3 последовательных замены

str strName;

strName = custTable.Name;
strName = global::strReplace(strTest, '\"', '`');
strName = global::strReplace(strTest, '\'', '`');
strName = global::strReplace(strTest, '&', '_');

Не вижу никакого практического смысла "вытягивать" команду замены в одну строку. И редактировать сложно и сама задача довольно специфическая. Я бы сказал, не очевидная.

Если так уж хочется написать посимвольную замену, то имеет смысл не использовать существующий метод global::strReplace(), а написать свой собственный, где в качестве параметра передаются две строки с наборами символов. Не замена текста на текст, а замена каждого символа из первого параметра на соответствующий символ из второго.

Чтобы было понятно о чем речь, есть функция ChrTran в FoxPro. Вот ее описание

https://msdn.microsoft.com/en-us/lib...(v=vs.71).aspx

Написать ее аналог в X++, думаю, будет не очень сложно

Моя вина, неудачный пример взял =) Все зацепились за один символ.

Можно и 100 раз использовать strReplace в коде, но зачем все тянуть в код, если можно заменить 100 strReplace на один strReplaceContainer? Если что-то одно поменять нужно то используем strReplace, если несколько значений, то strReplaceContainer.

Или я просто себе лишнею проблему придумал?

[Ruff]

Цитата:

Сообщение от demianimp

заменить 100 strReplace на один strReplaceContainer?
....
Или я просто себе лишнею проблему придумал?

Вот для 100 замен (да даже для 10+ замен) - точно лишняя проблема. Как минимум для тех, кому придется этот код поддерживать, модифицировать. В 100 интуитивно понятных строках кода найти нужное соответствие не составит труда, а вот считать элементы в длинных контейнерах, чтобы что-то поменять... мозг вскипит

Теоретически подход с контейнером наверно имеет преимущества, если он наполняется динамически, в зависимости от каких-то условий, действий пользователя и т.п. Но у меня сейчас не получается придумать практического примера такого применения.

[terraByteG]

Цитата:

Сообщение от demianimp

Можно и 100 раз использовать strReplace в коде, но зачем все тянуть в код, если можно заменить 100 strReplace на один strReplaceContainer? Если что-то одно поменять нужно то используем strReplace, если несколько значений, то strReplaceContainer.

Глаза не боитесь сломать, выискивая потом какой элемент на кокой меняется, хотя бы 10+ уже истерия может начаться...
Проще Copy/Paste strReplace сделать...