优化代码问题，多个如果其他语句PHP

Question

我是一个非常糟糕的编码器，但我正努力变得更好。我最近编写了一些有用的东西，万岁！但我觉得必须有一个更清洁的方式。对象是：

如果存在GET变量，则将其用作cookie值。如果变量不存在，请使用默认值。如果Cookie已经存在，则使用该值而不是新值。

if (!isset($_COOKIE['id'])) { 
    if (!isset($_GET[id])) { 
    $cookid="115"; 
    } else { 
    $cookid= $_GET["id"]; 
    } 
    setcookie("id", $cookid , time() + 31536000); 
} else { 
    $cookid= $_COOKIE['id']; 
} 

Answer 1

以下是我会写它：

所有的

if(isset($_COOKIE['id'])) { 
    $cookid = $_COOKIE['id']; 
} else { 
    $cookid = isset($_GET["id"]) ? $_GET["id"] : "115"; 

    setcookie("id", $cookid, time() + 31536000); 
} 

首先，按照一些约定，你应该测试肯定的情况下（isset(...)），而不是负（!isset()）。我认为它使更多的可读代码，所以我已经切换它。其次，我使用ternary operator（condition ? expr1 : expr2）消除了一个if/else块，这是可以接受的，因为您只是使用if/else块来决定将哪两个值分配给该变量。但是，应该谨慎使用三元运算符，因为如果过度使用它可能会使代码不易读。

第三，即使是单行if/else块，我也倾向于使用花括号，但至少我认为如果将它们用于if，则应该使用大括号作为else。

最后，只是一个可读性说明：尽量与你的空白保持一致。在第四行中，=周围没有任何空格，但在第6行和第10行中后面有空格，但之前没有空格。为了便于阅读，在操作员的两边都留有空白几乎总是可取的。

哦，$_GET["id"]是正确的; $_GET[id]不是。如果您的错误报告级别足够高，它（正确）会发出警告（并且您通常应该使用error_reporting(E_ALL);进行开发，以便看到它们）。

Answer 2

if (isset($_GET['id'])) { 
    $cookid = (int) $_GET['id']; 
    setcookie('id', $cookid, time() + 31536000); 
} else if (isset($_COOKIE['id'])) { 
    $cookid = (int) $_COOKIE['id']; 
} else { 
    $cookid = 115; 
    setcookie('id', $cookid, time() + 31536000); 
} 

Answer 3

我会封装成可以很容易地重新使用的功能：

<?php 
/** 
* If GET variable exists use it as the cookie value. 
* If the Variable doesn't exist use a default value. 
* If the Cookie already exists use that value instead of a new one. 
* 
* @param string $name name of cookie/get variable 
* @param string $default default value 
* @return string the value. 
*/ 
function cookie_get_value($name, $default) { 
    if (isset($_COOKIE[$name])) { 
     $value = $_COOKIE[$name]; 
    } else { 
     $value = isset($_GET[$name]) ? $_GET[$name] : $default; 
     setcookie($name, $value , time() + 31536000); 
    } 
    return $value; 
} 

echo cookie_get_value('id', '115'); 

最后，你需要做的，如果的你需要做的。要使自己的代码更好，并不总是那么容易，但是如果将它放入自己的函数中，则更容易修改。

Answer 4

我相信你可以使用$ _REQUEST，因为它已经有$ _ POST，$ _ GET和$ _COOKIE它

if (!isset($_REQUEST['id'])) 
{ 
    $cookid = '115'; 
    setcookie("id", $cookid , time() + 31536000); 
} 
else 
    $cookid = $_REQUEST['id'];