Changes between Initial Version and Version 3 of Ticket #554


Ignore:
Timestamp:
Jul 4, 2020, 6:28:08 PM (4 years ago)
Author:
alx
Comment:

Я считаю, что когда что-то кому-то объясняешь, пример должен иллюстрировать объяснение, дополнять его, помогая лучшему пониманию, но не заменять его.

Когда я предлагал пример для описания регулярного выражения ^(?![3,4]0\d{2})\d*, я предложил дополнить описание этого выражения примером, а не оставить один только пример вместо описания. Александр в своем комментарии также предлагал перед примером номеров, вызовы которых не будут приниматься, описать смысл выражения. Однако в тексте РЭ редакции 42 в таблице все объяснение состоит только из примера двух номеров, вызовы которых не будут приниматься. Считаю это неправильным, так как по двум частным случаям трудно понять общий принцип.

Предлагаю дополнить пример в описании регулярного выражения ^(?![34]0\d{2}) общим описанием его смысла (или вообще исключить это выражение из таблицы, так как без общего описания не вижу смысла в примере).

Здесь же в описании регулярного выражения с заменой ^/^2(\d{6})/83422\1 говорится: "В данном примере 'выражение'- ^5(\d{3}), а 'замена' – 6\1". Это совершенно неверно, так как на самом деле здесь регулярное выражение (то, что расположено между двумя символами '/') - ^2(\d{6}), строка замены - 83422\1. Ну и далее неоднократно упоминается то же выражение и цифра '5', которой вообще нет ни в собственно регулярном выражении, ни в строке замены. Предлагаю исправить описание.

Обратите внимание, что когда я предлагал дополнить описание последнего выражения фразой "и из номера удаляются все символы кроме первых семи", я был неправ: строкой, состоящей из префикса 8342 и первых семи цифр вызываемого номера, будет заменена только та часть вызываемого номера, которая совпадает с регулярным выражением, то есть как раз первые семь цифр. Все, что в номере вызываемого есть после этих семи цифр, останется в неизменном виде. То есть никакие цифры из номера удаляться не будут! Описание тикета исправил.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #554

    • Property Status newreopened
  • Ticket #554 – Description

    initial v3  
    1111`^[3,4]0\d{2}$` В описании написано: "принимает вызовы только на номера вида 30хх, 40хx". Здесь в описании та же проблема, что и в предыдущем примере. Предлагаю переформулировать описание примерно так: принимает только вызовы номеров, начинающихся с одного из символов "три" ('3'), "запятая" (',') или "четыре" ('4'), второй символ которых "ноль" ('0'), за которым следует ровно две любые цифры. Например '3074', ',039', '4081'.
    1212
    13 `/^2(\d{6})/83422\1` Во-первых, стоит, наверное, во вводной части уточнить, что ряд конфигурационных параметров поддерживает специальную форму записи - регулярное выражение с заменой, иначе неискушенный читатель подумает, что такая форма - часть синтаксиса PCRE. На самом деле это не так, такая форма записи - наше собственное изобретение. Во-вторых, в описании к примеру написано: "номер из 7 цифр, начинающийся с цифры 2". Это не совсем точно. Более точным будет "из 7 цифр, за которыми могут следовать любые символы". Далее же текст "к номеру добавляется префикс '8342'" следует дополнить: "и из номера удаляются все символы кроме первых семи" (так как только цифры номера со 2-й по 7-ю добавляются к комбинации '83422' в правой части).
     13`/^2(\d{6})/83422\1` Во-первых, стоит, наверное, во вводной части уточнить, что ряд конфигурационных параметров поддерживает специальную форму записи - регулярное выражение с заменой, иначе неискушенный читатель подумает, что такая форма - часть синтаксиса PCRE. На самом деле это не так, такая форма записи - наше собственное изобретение. Во-вторых, в описании к примеру написано: "номер из 7 цифр, начинающийся с цифры 2". Это не совсем точно. Более точным будет "из 7 цифр, за которыми могут следовать любые символы".