Как правильно составить регулярное выражение?

Question

[Михаил]

Всем привет!
Подскажите пожалуйста как правильно оформить паттерн для для функции preg_match ()?
Документацию по регулярным выражениям смотрел но что то так и не понял видимо, суть следующая:
1. Имеем строку следующего формата "nic0000-0000-0000", где (0000-0000-0000 - mac адрес ПК)
2. Строка является ключем массива, ключи все разные с разными параметрами ПК, поэтому и решил для поиска использовать preg_match (), поскольку карт может быть несколько и обратиться к конкретному полю не получится.
3. исходя из документации сделал паттерн следующего вида "$patternNic='/nic*/'" , что бы выбирал поля начинающиеся с nic и mac адрес в виде (*) любой последовательности.
4. При применении preg_match($patternNic, $string, $matches); в массив $matches пишутся только "nic", то есть массив имеет такой формат array ([0]=>'nic', [1]=>'nic' ). Функция почему то режет МАКи, я так понимаю проблема в паттерне, подскажите как его правильно написать для данной строки?

Answer 1

[GTRxShock]

как минимум "$patternNic='/nic.*/'"

либо:

preg_match('/nic([\d-]*)/', 'nic0000-0000-0000', $matches); 
var_dump($matches[1]); // string(14) "0000-0000-0000"

p.s. и лучше это назвать не паттерном, а регулярным выражением, так быстрее люди поймут, что вам надо

Comments

[Sledge5]

А если в адресе буквы?
Тогда уж так:

preg_match('/nic([\da-f\-]*)/i', 'nicabcd-1234-5678', $matches);

Answer 2

[Илья Мясин]

* влияет на предыдущий символ. Ваша регулярка ищет символы: "n" (1 шт), "i" (1 шт), "c" (0 и более шт).

Зачем вам тут вообще регулярка - не понимаю, по ощущениям скорее всего надо делать как-то иначе =) Если таки хочется получить регуляркой часть после nic (если я правильно понял ход мысли), то как-то так:

<?php
preg_match("~^nic(.+)$~", "nic0000-0000-0000", $matches);

В $matches будет

array (
  0 => 'nic0000-0000-0000',
  1 => '0000-0000-0000',
)

А, ну вот GTRxShock воспроизвел примерно то же самое, пока я писал =) Его вариант ловит цифры и дефис после nic в любом месте строки. Мой - все, что после nic, если строка начинается с nic. Какой лучше - хз, надо смотреть на ваши строки =)

Comments

[Михаил]

Спасибо за ваш ответ!
Регулярка тут нужна для поиска конкретного ключа массива.
Поскольку ключи не однородны, и некоторых сущностей, таких как планки ОЗУ, наклейки лицензий, и сетевые адаптеры, может быть несколько, и массив полученный из Request имеет следующий формат.


А здесь весь код цикла, с ОЗУ и лицензиями отрабатывает нормально, а вот с регулярным выражением для маков возникли проблемы.

foreach ($data as $key => $value)
       { 
           if (preg_match($patternRam, $key, $matches))
                  {   
                    unset($data[$key]);
                    foreach ($matches as $key => $value)
                        {
                            if ($key==1)
                            {
                                $ram["$i"]=$value;
                                $i++;
                            }
                        }
                       unset($matches);
                  }
            elseif (preg_match($patternLicense, $key, $matches))
                  {   
                    unset($data[$key]);
                    foreach ($matches as $key => $value)
                        {
                            if ($key==1)
                            {
                                $license["$j"]=$value;
                                $j++;
                            }
                        }
                       unset($matches);
                  }
            elseif (preg_match($patternNic, $key, $matches))
                  {  dump($matches);
                    unset($data[$key]);
                    foreach ($matches as $key => $value)
                        {
                            if ($key==0)
                            {
                                $nic["$k"]=$value;
                                $k++;
                            }
                        }
                       unset($matches);
                  }

[Илья Мясин]

Михаил: ок, давайте посмотрим на ваш код =)

0) изменять массив внутри цикла по этому же массиву - верный способ поседеть раньше срока )))
1) вместо $license["$j"] можно $license[$j] - хотя бы для подсветки IDE
2) вместо $license[$j]= '...'; $j++; можно $license[]= '...'; - такая запись просто добавляет новый элемент в конец, счетчик не нужен.
3) обходить $matches циклом не нужно - если регулярка сработала и в ней есть скобки, все индексы будут заполнены, можно просто писать $matches[1].
4) если данные ваши (то есть вы управляете тем, как они приходят), лучше сделать, чтобы в них значения уже приходили сгруппированными. То есть добиться такого инпута:

$data = [
    'type' => 'Рабочая станция',
    // ...
    'nic' => [
        '0000-0000-0000-0000',
        '0000-0000-0000-0001'
    ],
    'license' => [
        '0',
        '1'
    ]
];

Если данные приходят из формы, используйте названия инпутов вида prop[] - они будут преобразовываться в массив. Добавление строчек можно сделать на клиенте, в простейшем виде вот так:

<?php
echo '<pre>'.print_r($_POST,1).'</pre>';
?>
<form action="" method="post">
    <input name="nic[]" value="0000-0000-0000-0001" />
    <input name="nic[]" value="0000-0000-0000-0002" />
    <span class="add_nic">добавить nic</span>
    <input type="submit" value="ok" />
</form>
<script type="text/javascript">
    document.querySelector('.add_nic').addEventListener(
        'click', 
        function(e) {
            var inp = document.createElement('input');
            inp.setAttribute('name', 'nic[]');
            this.parentNode.insertBefore(inp, this);
        }
    );
</script>

5) Если данные не ваши, и если такой фигни много и она может добавляться, я бы вместо длинной цепочки if-elseif сделал массив обработчиков, которые можно добавлять "снаружи".

Что-то типа такого:

<?php

class dataConverter {

    public function addHandler($closure, $rex = null)
    {
    
    }
    
    public function __construct() 
    {
        // добавляем дефолтные обработчики
    	$this->addHandler( 
          	function($k, $v, $matches) {
            	// [значение, ключ, сохранять_как_массив]
            	return [$matches[1], 'nic', true];
            },
            '~^nic(\.+)$~'
        );
    }
    
    public function convert($data)
    {
    	foreach ($data as $k => $v) {
            foreach ($this->handlers as $handler) {
               // проверяем
            }
        }
    }
}

Работать будет чуть медленнее, зато проще конфигурировать. Стоит ли заморачиваться - зависит от того, сколько таких обработчиков и с какой периодичностью они могут изменяться.

[Михаил]

Илья Мясин: спасибо большое за ваши ценные советы!
Я только учусь, поэтому обязательно приму к сведению и постараюсь оптимизировать сие творение)
А что касается данных, то да я их получаю из формы, и эти 3 параметра выбираются в виде списка checkboxов, которые я получаю из базы в виде полных списков возможных вариантов, имеющихся в конторе.
Из вашего примера я правильно понял, что запись в массив значений отдельных инпутов реализует именно javascript, и без него это будут отдельные массивы, а скорее даже ошибка и постоянная перезапись ввиду одинаковых имен инпутов?

[Илья Мясин]

Михаил: через JS у меня сделано добавление нового значения, если предполагается, что юзер может добавлять произвольные значения. Если он только выбирает из заданных, то все еще проще:

<form action="" method="post">
    <label>
        0001
        <input name="nic[]" value="0000-0000-0000-0001" type="checkbox" />
    </label>
    <label>
        0002
        <input name="nic[]" value="0000-0000-0000-0002" type="checkbox" />
    </label>
    <input type="submit" />
</form>

Если отметить оба чекбокса, в $_POST попадет такой массив:

Array
(
    [nic] => Array
        (
            [0] => 0000-0000-0000-0001
            [1] => 0000-0000-0000-0002
        )

)

[Михаил]

Илья Мясин: Понял, спасибо огромное!
Раньше не подозревал, что можно напрямую из инпутов массив формировать, приходилось брать то что пришло в $_POST и уже с помощью вот таких костылей приводить данные к нужному формату.

Answer 3

[Sledge5]

$mac = substr("nic0000-0000-0000", 3);

Не понимаю, зачем тут RegExp?