Как написать это, если заявление (я) более коротким способом [дубликата]

Question

Как написать это, если заявление (я) более коротким способом [дубликата]

0

Можно ли написать это короче? Я не могу вносить изменения в метод accept_lang и всегда возвращает логическое значение.

if ($ci->agent->accept_lang('en-US') OR $ci->agent->accept_lang('en-GB') OR $ci->agent->accept_lang('en-au'))

Badr Hari 08 янв. 2015, в 16:19

Источник

0

нет, вы не можете, без изменения метода
Steve 08 янв. 2015, в 15:26
0

Вы пытаетесь избежать вызова метода 3 раза, или вы просто хотите меньше символов в этой строке?
Flosculus 08 янв. 2015, в 15:29
0

Меньше символов, @Flosculus
Badr Hari 08 янв. 2015, в 15:31
0

@singhakash Я знаю, как проверить, находится ли что-то в массиве, мне просто любопытно, можно ли написать его короче, если мне нужно использовать встроенный метод этого фреймворка.
Badr Hari 08 янв. 2015, в 15:33
0

Готовы ли вы иметь другую функцию, чтобы сделать это короче?
Flosculus 08 янв. 2015, в 15:37
0

Может быть, попробуйте что-то вроде этого: $arr = array("en", "de"); array_walk($arr, array($ci, "agent"));
Rizier123 08 янв. 2015, в 15:41

Показать ещё 4 комментария

Теги:

php

coding-style

3 ответа

0

В основном ваши ищут истинный в массиве результатов, используя in_array Icw array_map можно построить следующий Однострочник;

if (in_array(true, array_map(array($ci, "accept_lang"), array("en-US", "en-GB", "en-au"))))

Сломался немного, чтобы более четко показать, что происходит;

$langAccept = array_map(array($ci, "accept_lang"), array("en-US", "en-GB", "en-au"));
if (in_array(true, $langAccept)) {
    // ...
}

Gio 08 янв. 2015, в 15:34

0

Как насчет использования? Он копирует из

Определение языка браузера в PHP

почему бы вам не сделать это простым и чистым

<?php
$lang = substr($_SERVER['HTTP_ACCEPT_LANGUAGE'], 0, 2);
switch ($lang){
    case "fr":
        //echo "PAGE FR";
        include("index_fr.php");//include check session FR
        break;
    case "it":
        //echo "PAGE IT";
        include("index_it.php");
        break;
    case "en":
        //echo "PAGE EN";
        include("index_en.php");
        break;        
    default:
        //echo "PAGE EN - Setting Default";
        include("index_en.php");//include EN in all other cases of different lang detection
        break;
}
?>

cpplover - Slw Essencial 08 янв. 2015, в 14:01

Ещё вопросы

нет, вы не можете, без изменения метода
Вы пытаетесь избежать вызова метода 3 раза, или вы просто хотите меньше символов в этой строке?
@singhakash Я знаю, как проверить, находится ли что-то в массиве, мне просто любопытно, можно ли написать его короче, если мне нужно использовать встроенный метод этого фреймворка.
Готовы ли вы иметь другую функцию, чтобы сделать это короче?
Может быть, попробуйте что-то вроде этого: $arr = array("en", "de"); array_walk($arr, array($ci, "agent"));

Dave Goten · Accepted Answer · 2015-01-08T13-21-00.000Z

Может быть, не короче, но простой цикл может помочь. Сначала добавьте наиболее распространенные области, чтобы уменьшить количество проверок и перерыв, чтобы остановить проверку, если вы найдете его.

<?php
foreach(array('en-US','en-GB','en-au') as $lang) {if($ci->agent->accept_lang($lang)){stuff();break;}}
?>

или разразился:

<?php
foreach(array('en-US','en-GB','en-au') as $lang) {
  if($ci->agent->accept_lang($lang)){
    stuff(); // or whatever else you need, or set a $flag = true
    break;
  }
}
?>

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