0

Написал скрипт для облегчения скачивания файлов с файлообменников(ФО). Сейчас устроено так: Пользователь через форму отправляет ссылку скрипту, скрипт проверяет через "preg_match" название ФО, и для каждого ФО свой метод получения прямой ссылки на файл. Весь скрипт построено на условиях if:

if(preg_match("/site1/", $url)){
Получение url
}
if(preg_match("/site2/", $url)){
}
if(preg_match("/site3/", $url)){
}
if(preg_match("/site4/", $url)){
}

Как сделать это все через функции - читал про них, но так и не пойму до сих пор как их использовать.

8
  • А что именно вы хотите сделать через функции? 31 авг 2015 в 19:03
  • Если вам нужно проверить что сайт - одно из значений, preg_match - явный перебор. Можно просто искать домен в строке. 31 авг 2015 в 19:06
  • @DmitriySimushev Получение прямой ссылки, я предполагаю что функции более облегчают и оптимизируют код. 31 авг 2015 в 19:16
  • @ВладимирМартьянов а можно подробнее? 31 авг 2015 в 19:17
  • Функцию предпочтительно использовать, если код выполняется множество раз. В данной задаче такая надобность отсутствует. Для оптимизации лучше заменить preg_match на strpos. Использование функций в любом языке программирования аналогично. Раньше не программировал? Читай учебники, справочники... В принципе, ничего сложного: оборачиваешь тот же код в функцию так: function funkciya() {Код} и вызываешь funkciya(); 31 авг 2015 в 19:26

1 ответ 1

1

верхние коментарии хороши в вашем случае функцию сделел бы такую.

    function file ($url){
      switch ($url){
         case(preg_match("/site1/", $url)):
           return Получение url;
        break;
        case(preg_match("/site2/", $url)):
           return Получение url;
        break;
        case(preg_match("/site3/", $url)):
           return Получение url;
        break;
        case(preg_match("/site4/", $url)):
           return Получение url;
        break;
     }
}
$urlThe=file(http://...)

где "http://..." это адрес вашего обменика, он передаеться в функцию file в виде парамерта $url обрабатывается функцией и потом через return передается переменой $urlThe. И если много раз используете if то мне кажеться целесообразнее использовать switch http://php.net/manual/ru/control-structures.switch.php

3
  • Возможно мой вопрос прозвучит глупо, но как в case добавлять большие участки кода? 1 сен 2015 в 10:27
  • @edvardpotter Честно ни когда не обрашал внимание на болльшие участки кода в swith но вроде выполняется все до break в карайнем случае запихните в теже переменые весь код и веред или через функции.
    – Sergalas
    2 сен 2015 в 18:08
  • @edvardpotter: нельзя ли выделять эти большие участки кода в отдельные методы/функции?
    – Nick Volynkin
    3 сен 2015 в 9:29

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.