Question

我已经使用条件运算符编写了一段代码：

$iPage=($this->request()->get('search') ? $this->request()->getInt('req4') : ($this->request()->getInt('req3') ? $this->request()->getInt('req3') : 1 ));

我将其转换为if-else格式，如下所示：

if($this->request()->get('search')) {
  $this->request()->getInt('req4')
} else {
  ($this->request()->getInt('req3')
}

请有人帮我说明我是否已正确进行代码转换？如果我在转换中犯了任何错误，请更正错误并让我知道错误。

Answer 1

这是原始代码（使用条件运算符）格式化为更易读（根本不需要外括号）：

$iPage = (
    $this->request()->get('search') ?
    $this->request()->getInt('req4') :
    (
        $this->request()->getInt('req3') ?
        $this->request()->getInt('req3') :
        1
    )
);

在第一步中，我们将$this->request()提取到一个变量中;它使代码更容易阅读，运行速度更快（避免四次执行相同的函数调用;每次都返回相同的值）：

$request = $this->request();
$iPage = (
    $request->get('search') ?
    $request->getInt('req4') :
    (
        $request->getInt('req3') ?
        $request->getInt('req3') :
        1
    )
);

现在，让我们用if/then/else替换条件：

$request = $this->request();
if ($request->get('search')) {
    $iPage = $request->getInt('req4');
} else {
    if ($request->getInt('req3')) {
        $iPage = $request->getInt('req3');
    } else {
        $iPage = 1;
    }
}

现在，我们注意到值1是默认值$iPage。如果满足有关请求的某些条件，则使用$iPage的值计算$request的值;否则$iPage变为1。让我们提取这些信息并简化代码：

// Default value 
$iPage = 1;
// Get the request into a variable for faster and shorter code
$request = $this->request();
// If 'search' is set, use 'req4' else use 'req3'
if ($request->get('search')) {
    $iPage = $request->getInt('req4');
} else {
    if ($request->getInt('req3')) {
        $iPage = $request->getInt('req3');
    }
}

现在我们可以将else与其包含的if合并到：

// Default value 
$iPage = 1;
// Get the request into a variable for faster and shorter code
$request = $this->request();
// If 'search' is set, use 'req4' else use 'req3'
if ($request->get('search')) {
    $iPage = $request->getInt('req4');
} elseif ($request->getInt('req3')) {
    $iPage = $request->getInt('req3');
}

如果我们将默认值放回if / else构造中，您可能会觉得更容易理解，如下所示：

// Get the request into a variable for faster and shorter code
$request = $this->request();
if ($request->get('search')) {
    // If 'search' is provided, use 'req4'
    $iPage = $request->getInt('req4');
} elseif ($request->getInt('req3')) {
    // else, use 'req3' if provided
    $iPage = $request->getInt('req3');
} else {
    // else default to 1
    $iPage = 1;
}

或者，如果您不介意使用条件运算符（并获得更紧凑的代码），则可以使用由if表单引入的功能将第二个?:打包回// Get the request into a variable for faster and shorter code $request = $this->request(); // If 'search' is provided, use 'req4' if ($request->get('search')) { $iPage = $request->getInt('req4'); } else { // Use 'req3' if provided else default to 1 $iPage = $request->getInt('req3') ?: 1; }表单。 PHP 5.3（"Since PHP 5.3, it is possible to leave out the middle part of the ternary operator"）：

found

使用上述任何代码转换;所有这些都提供了相同的结果我个人更喜欢最后一个;它结构紧凑，易于阅读。

Answer 2

不，转换应该是：

if ($this->request()->get('search'))
    $iPage = $this->request()->getInt('req4');
elseif ($this->request()->getInt('req3'))
    $iPage = $this->request()->getInt('req3');
else
    $iPage = 1;

Answer 3

正确的转换是：

if ($this->request()->get('search') {
    $iPage = $this->request()->getInt('req4');
} else {
    if $this->request()->getInt('req3') {
        $iPage = $this->request()->getInt('req3');
    } else {
        $iPage = 1;
    }
}

但是你可以合并第一个和第二个if，使其成为elseif

如何将PHP三元表达式转换为if-else？

3 个答案: