Логика проверки наличия нескольких наборов символов в переменной номера телефона

[sudya12345]

Приветствую, подскажите по логике запроса в twig

 

Есть переменные с номерами телефонов, надо проверить код оператора из списка и уже на основании определенного кода назначить класс со своим лого оператора

мой временный костыль работает:

{% for contact_telephone in contact_telephones %}
		{% if contact_telephone starts with '+38(050)' %}
				<li><a href="tel:{{ contact_telephone|replace({" " : "", "-" : "", "(" : "", ")" : ""}) }}" class="phoneclick"><i class="us-vf"></i> {{ contact_telephone }}</a></li>
		{% elseif contact_telephone starts with '+38(067)' %}
				<li><a href="tel:{{ contact_telephone|replace({" " : "", "-" : "", "(" : "", ")" : ""}) }}" class="phoneclick"><i class="us-ks"></i> {{ contact_telephone }}</a></li>
		{% elseif contact_telephone starts with '+38(063)' %}
				<li><a href="tel:{{ contact_telephone|replace({" " : "", "-" : "", "(" : "", ")" : ""}) }}" class="phoneclick"><i class="us-lc"></i> {{ contact_telephone }}</a></li>
		{% else %}
				<li><a href="tel:{{ contact_telephone|replace({" " : "", "-" : "", "(" : "", ")" : ""}) }}" class="phoneclick"><i class="fas fa-phone-volume"></i> {{ contact_telephone }}</a></li>
		{% endif %}
{% endfor %}

как мне упростить код перебирая коды операторов телефонов   (050) or (066) or (095) or (099) ???

[ocdev_pro]

1 час назад, sudya12345 сказал:

Приветствую, подскажите по логике запроса в twig

 

Есть переменные с номерами телефонов, надо проверить код оператора из списка и уже на основании определенного кода назначить класс со своим лого оператора

мой временный костыль работает:

{% for contact_telephone in contact_telephones %}
		{% if contact_telephone starts with '+38(050)' %}
				<li><a href="tel:{{ contact_telephone|replace({" " : "", "-" : "", "(" : "", ")" : ""}) }}" class="phoneclick"><i class="us-vf"></i> {{ contact_telephone }}</a></li>
		{% elseif contact_telephone starts with '+38(067)' %}
				<li><a href="tel:{{ contact_telephone|replace({" " : "", "-" : "", "(" : "", ")" : ""}) }}" class="phoneclick"><i class="us-ks"></i> {{ contact_telephone }}</a></li>
		{% elseif contact_telephone starts with '+38(063)' %}
				<li><a href="tel:{{ contact_telephone|replace({" " : "", "-" : "", "(" : "", ")" : ""}) }}" class="phoneclick"><i class="us-lc"></i> {{ contact_telephone }}</a></li>
		{% else %}
				<li><a href="tel:{{ contact_telephone|replace({" " : "", "-" : "", "(" : "", ")" : ""}) }}" class="phoneclick"><i class="fas fa-phone-volume"></i> {{ contact_telephone }}</a></li>
		{% endif %}
{% endfor %}

как мне упростить код перебирая коды операторов телефонов   (050) or (066) or (095) or (099) ???

 

ну как минимум перестать использовать логику в шаблонах. по рукам надо бить за такое.
Пишите проверку через контроллер, strpos и switch case
учите php

[chukcha]

097
073 etc
коды городов?
Не? не надо?

 

Вам надо отформатировать номер перед выводом
проверить количество цифр (удалив все не цифры
Проверить длину
Проверить наличие тройки восьмерки нуля в начала


апд

Ой.. невнимательно прочитал первый пост

[chukcha]

Вы можете создать массив
 

{% set array_code = [
'+3097: 'код иконки',

] %}

{% code = contact_tlefone|slice(0,7)  %}


{% if array_code.code is defined %}

{% else %}


{% endif %}

 

[sudya12345]

Писал в попых и уставший в поисках решения)))

Некоторое уточнение

В Украине 3 мобильных оператора и у каждого несколько кодов

Водафон (050, 095, 066, 099)

Киевстар (067, 097, 096, 068, 098)

Лайфселл (063, 073, 093)

 

и вот клиент вводит несколько номеров например Водафона и у всех этих номеров с разным кодом должно быть лого Водафона, ну и если другого оператора соответственно лого другого оператора. К каждому номеру свой лого.

 

Как по мне, самое идеальное было бы использовать операторы или (or ||)

 

13 часов назад, ocdev_pro сказал:

 

ну как минимум перестать использовать логику в шаблонах. по рукам надо бить за такое.
Пишите проверку через контроллер, strpos и switch case
учите php

 

учим)

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

 

Ну и если использовать case как мне перебрать несколько кодов одного оператора или под каждый код свой case? 

[chukcha]

8 минут назад, sudya12345 сказал:

В Украине 3 мобильных оператора

Да вы шо?
PeopleNet, Интертелеком

Я вам показал приблизительное решение

Ваш случай - частный, потому можно и накостылять.


 

[ocdev_pro]

14 минут назад, chukcha сказал:

Ваш случай - частный, потому можно и накостылять

Так если случай частый, может уже пора научиться нормально писать, а не костыли? Советчик 99 лвл

[chukcha]

1 минуту назад, ocdev_pro сказал:

Так если случай частый, может уже пора научиться нормально писать, а не костыли? Советчик 99 лвл

ой-ой-ой
Вывести три иконки - жестко привязанных к щаблону..ШАБЛОНУ.. А не к логике сайта..

Где вы увидели здесь логику?




 

[iglin]

7 минут назад, chukcha сказал:

Где вы увидели здесь логику?

Ну вообще меня учили что логика это как раз if, else, or и т.д. )) Может у меня уроки логики какие то неправильные были)

 

Ну а так тоже считаю что это лучше в контроллер всё засунуть. Создать массивы кодов для каждого оператора, и искать в них. Выводя в шаблон уже только картинку привязанную к каждому из массивов с кодами оператора.

[sudya12345]

27 минут назад, chukcha сказал:

Да вы шо?
PeopleNet, Интертелеком

этими операторами не воспользоваться обычным мобильным телефоном.

С таким успехом можно сказать и про все коды областей укртелекома

[chukcha]

6 минут назад, iglin сказал:

Ну вообще меня учили что логика это как раз if, else, or и т.д. )) Может у меня уроки логики какие то неправильные были)

 

Ну а так тоже считаю что это лучше в контроллер всё засунуть. Создать массивы кодов для каждого оператора, и искать в них. Выводя в шаблон уже только картинку привязанную к каждому из массивов с кодами оператора.

т.е. ни в одном шаблоне нет if else
Т.е. нет ни какакой логики?

Здесь логика выбора  иконки ..
 

 

7 минут назад, sudya12345 сказал:

не воспользоваться обычным мобильным телефоном.

хм. что из этого следуете?


Если вам нужен универсальный метод, с управлением кодами, иконками - есть смысл делать в контроллере
В частном случае(ваш) - костыль - решение
 

[ocdev_pro]

1 час назад, sudya12345 сказал:

этими операторами не воспользоваться обычным мобильным телефоном.

С таким успехом можно сказать и про все коды областей укртелекома

Еще как воспользуются, но не все. Много есть у кого код 089 Интертелеком или 091 LycaMobile, которая на базе ТриМоб (Укртелеком).

Если я правильно понял, Вы пытаетесь сделать универсальное решение, что бы контактные номера выводились сразу с иконкой оператора где-то на странице.
Тогда данные в контроллере которые вы получаете из настроек допустим

Вот рабочий пример реализации:

В контроллере создаете метод для проверки номера и определения ему иконки
 

private function checkPhone($phone) {
     $result = [];

    $phone = preg_replace("/[^0-9]/", '', $phone);

    $part_0 = substr($phone, 0, 2);

    switch ($part_0) {
        case '38':
            $phone = substr_replace($phone, '', 0, 2);
            break;
        case '80':
            $phone = substr_replace($phone, '0', 0, 2);
            break;
    }

    $phone_code = substr($phone, 0, 3);

    switch ($phone_code) {
        case '050':
        case '066':
        case '095':
        case '099':
            $result['icon'] = DIR_IMAGE . 'icon/vodafon.png'; //Путь к иконке
            break;
        case '063':
        case '073':
        case '093':
            $result['icon'] = DIR_IMAGE . 'icon/lifecell.png';
            break;
        case '067':
        case '068':
        case '096':
        case '097':
        case '098':
            $result['icon'] = DIR_IMAGE . 'icon/kievstar.png';
            break;
        default:
            $result['icon'] = DIR_IMAGE . 'icon/other_code.png';
    }
    
    $result['tel'] = '38' . $phone;

    return $result;
}

Допустим Вы получаете номера из настроек в виде строки 
 

$config_phones = '+380501234567,0934538411,38097155354';

//Превращаем строку с разделителем ',' в массив
$phones = explode(',', $config_phones);

//Обьявляем пустой массив для обработанных данных 
$data['contact_telephone'] = [];

//Церез цикл отправляем номера на обработку
foreach ($phones as $number) {
    $data['contact_telephone'][] = [
      'number' => $number,
      'icon' => $this->checkPhone($number);
    ];
}

И затем в twig шаблоне выводите

{% for contact_telephone in contact_telephones %}
<li><a href="tel:{{ contact_telephone.tel }}" class="phoneclick"><img src="{{ contact_telephone.icon }}" /> {{ contact_telephone.tel }}</a></li>
{% endfor %}

@chukcha Дальше можно в отпуске сидеть и не засорять бесполезными сообщениями форум

[chukcha]

19 минут назад, ocdev_pro сказал:

Еще как воспользуются, но не все. Много есть у кого код 089 Интертелеком или 091 LycaMobile, которая на базе ТриМоб (Укртелеком).

Если я правильно понял, Вы пытаетесь сделать универсальное решение, что бы контактные номера выводились сразу с иконкой оператора где-то на странице.
Тогда данные в контроллере которые вы получаете из настроек допустим

Вот рабочий пример реализации:

В контроллере создаете метод для проверки номера и определения ему иконки
 

private function checkPhone($phone) {
     $result = [];

    $phone = preg_replace("/[^0-9]/", '', $phone);

    $part_0 = substr($phone, 0, 2);

    switch ($part_0) {
        case '38':
            $phone = substr_replace($phone, '', 0, 2);
            break;
        case '80':
            $phone = substr_replace($phone, '0', 0, 2);
            break;
    }

    $phone_code = substr($phone, 0, 3);

    switch ($phone_code) {
        case '050':
        case '066':
        case '095':
        case '099':
            $result['icon'] = DIR_IMAGE . 'icon/vodafon.png'; //Путь к иконке
            break;
        case '063':
        case '073':
        case '093':
            $result['icon'] = DIR_IMAGE . 'icon/lifecell.png';
            break;
        case '067':
        case '068':
        case '096':
        case '097':
        case '098':
            $result['icon'] = DIR_IMAGE . 'icon/kievstar.png';
            break;
        default:
            $result['icon'] = DIR_IMAGE . 'icon/other_code.png';
    }
    
    $result['tel'] = '38' . $phone;

    return $result;
}

Допустим Вы получаете номера из настроек в виде строки 
 

$config_phones = '+380501234567,0934538411,38097155354';

//Превращаем строку с разделителем ',' в массив
$phones = explode(',', $config_phones);

//Обьявляем пустой массив для обработанных данных 
$data['contact_telephone'] = [];

//Церез цикл отправляем номера на обработку
foreach ($phones as $number) {
    $data['contact_telephone'][] = [
      'number' => $number,
      'icon' => $this->checkPhone($number);
    ];
}

И затем в twig шаблоне выводите

{% for contact_telephone in contact_telephones %}
<li><a href="tel:{{ contact_telephone.tel }}" class="phoneclick"><img src="{{ contact_telephone.icon }}" /> {{ contact_telephone.tel }}</a></li>
{% endfor %}

@chukcha Дальше можно в отпуске сидеть и не засорять бесполезными сообщениями форум

Я так понимаю, что твоя полезность выше крыше


А теперь внимательно смотрим мой код  (да, показан в синтаксисе твига) ой.. бида!!!

и мой пост

 

7 часов назад, chukcha сказал:

Вам надо отформатировать номер перед выводом
проверить количество цифр (удалив все не цифры
Проверить длину
Проверить наличие тройки восьмерки нуля в начала

Что показано у тебя в коде..

    $phone = preg_replace("/[^0-9]/", '', $phone);

    $part_0 = substr($phone, 0, 2);

    

йошкин код...
Где проверка на количество?


Есть еще желчь?

 

[ocdev_pro]

  

14 минут назад, chukcha сказал:

Я так понимаю, что твоя полезность выше крыше

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

 

14 минут назад, chukcha сказал:

йошкин код...
Где проверка на количество?

Не проверка на количество, а выдергивание подстроки для проверки в каком формате номер, т.к в примере я показал, что номер может быть указан +380б 380, 050 итд.. Чукча иди проспись и почитай php и как им пользоваться. В шаблоне такое не пишут, если следовать ООП, то для всего есть своя область ответственности и описанное решение тобой это откровенный говнокод.

Никакой желчи, только факты и здравый смысл. Если ты так относишься к своему коду, к клиентским проектам.. то могу им и программистам после тебя только посочувствовать. Т.к дед не понимает, что такое чистота кода и ООП