Перейти к публикации
Поиск в
  • Дополнительно...
Искать результаты, содержащие...
Искать результаты в...

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


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) ???

Ссылка на комментарий
Поделиться на других сайтах


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

Ссылка на комментарий
Поделиться на других сайтах

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

 

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


апд

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

Ссылка на комментарий
Поделиться на других сайтах

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

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

] %}

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


{% if array_code.code is defined %}

{% else %}


{% endif %}

 

Ссылка на комментарий
Поделиться на других сайтах

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

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

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

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

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

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

 

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

 

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

 

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

 

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

 

учим)

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

 

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

Ссылка на комментарий
Поделиться на других сайтах


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

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

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

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

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


 

Ссылка на комментарий
Поделиться на других сайтах

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

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

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

Ссылка на комментарий
Поделиться на других сайтах

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

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

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

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




 

Ссылка на комментарий
Поделиться на других сайтах

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

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

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

 

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

  • +1 1
Ссылка на комментарий
Поделиться на других сайтах

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

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

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

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

Ссылка на комментарий
Поделиться на других сайтах


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

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

 

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

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

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

 

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

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

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


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

Ссылка на комментарий
Поделиться на других сайтах

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 Дальше можно в отпуске сидеть и не засорять бесполезными сообщениями форум

  • +1 1
Ссылка на комментарий
Поделиться на других сайтах

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);

    


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


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

 

Ссылка на комментарий
Поделиться на других сайтах

  

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

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

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

 

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

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

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

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

Ссылка на комментарий
Поделиться на других сайтах

Создайте аккаунт или войдите в него для комментирования

Вы должны быть пользователем, чтобы оставить комментарий

Создать аккаунт

Зарегистрируйтесь для получения аккаунта. Это просто!

Зарегистрировать аккаунт

Войти

Уже зарегистрированы? Войдите здесь.

Войти сейчас
 Поделиться

  • Сейчас на странице   0 пользователей

    • Нет пользователей, просматривающих эту страницу.
×
×
  • Создать...

Важная информация

На нашем сайте используются файлы cookie и происходит обработка некоторых персональных данных пользователей, чтобы улучшить пользовательский интерфейс. Чтобы узнать для чего и какие персональные данные мы обрабатываем перейдите по ссылке. Если Вы нажмете «Я даю согласие», это означает, что Вы понимаете и принимаете все условия, указанные в этом Уведомлении о Конфиденциальности.