is if elseif elseif possible vs if else?

Question

Я новичок в php, и я действительно запутался, мой код кажется правильным, и он работает, но не полностью. Я пытаюсь реализовать перевод, используя сеанс для пользователей, для реализации какой-либо другой функции при вызове условия, если elseif elseif, мне нужно использовать hain if-statements с if else else или это возможно, как я сделал вначале.

<?php 

if ($_GET['lang']=='fr') { 
include('fr.php'); 
session_start(); 
$_SESSION['lang']='fr'; 
} 


elseif ($_GET['lang']=='en') { 
include('gb.php'); 
session_start(); 
$_SESSION['lang']='en'; 
} 

elseif ($_GET['lang']=='it') { 
include('it.php'); 
session_start(); 
$_SESSION['lang']='it'; 
} 

elseif ($_SESSION['lang']=='fr') 
{ 
include('fr.php'); 
} 

elseif ($_SESSION['lang']=='en') 
{ 
include('gb.php'); 
} 

elseif ($_SESSION['lang']=='it') 
{ 
include('it.php'); 
} 

else 
{ 
include('fr.php'); 
} 
?> 

Answer 1

Вы можете использовать переключатель заявление для аккуратным кода вид:

 

switch($_SESSION['lang']) { 
    case 'it': 
     include(..); 
     break; 
    case 'en': 
     include(..); 
     break; 

    etc.. 
} 
 

см http://php.net/manual/en/control-structures.switch.php

Answer 2

При использовании сессий, вы всегда должны вызывать session_start(), будь вы инициализации сеанса или просто возобновить его, так что вытащите его из своих заявлений и поставьте поверх своей страницы. Кроме того, я не совсем понимаю, чего вы пытаетесь достичь или что-то не получается.

Answer 3

Первое, что я заметил здесь, - это много повторяющегося кода и то, что по существу делает поиск из нескольких мест. Если вы реструктурируете свой код, вы можете устранить большую часть сложности. Вот то же самое сделано гораздо более простым способом:

<?php 

define('DEFAULT_LANG', 'fr'); 
$lang_inc = array(
    'fr' => 'fr.php', 
    'en' => 'gb.php', 
    'it' => 'it.php', 
); 

session_start() 

if ($_GET['lang'] && array_key_exists('lang', $_GET)) { 
    $lang = $_GET['lang']; 
} elseif ($_SESSION['lang'] && array_key_exists('lang', $_SESSION)) { 
    $lang = $_SESSION['lang']; 
} else { 
    $lang = DEFAULT_LANG; 
} 

$lang_file = $lang_inc[$lang]; 
include($lang_file); 

$_SESION['lang'] = $lang; 

Answer 4

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

Вы также не должны перезагружать $ _SESSION ['lang'], если явно не задано $ _GET ['lang'].

Это будет более эффективным и менее половины кода, используемого с if/else или условными обозначениями переключателя на основе значения lang.

<?php 
session_start(); 

// Only set $_SESSION[$lang] if it is not set 
// or if $_GET['lang'] is explicitly defined 
if (isset($_GET['lang']) || !isset($_SESSION['lang'])) { 

    // Array of allowed languages 
    $allowed = array('fr', 'en', 'it'); 

    // Set fr to default if lang isn't defined or not allowed 
    $_SESSION['lang'] = isset($_GET['lang']) && in_array($_GET['lang'], $allowed) ? $_GET['lang'] : 'fr'; 
} 

// dynamically include file 
include($_SESSION['lang'] . '.php');