主页 / 问题 / 980167
Accepted
orel-22
Asked:2020-05-11 12:35:25 +0000 UTC2020-05-11 12:35:25 +0000 UTC

如何简洁地编写代码？

  • 772

有这么小的功能。有没有办法让它更宽敞、更高效？也许用另一种结构（不是 if else）或以不同的方式来实现比较？PS我刚开始学习PHP

function game_parameters_check($gameMode, $level, $betValue)
{
    if($gameMode != 'real' && $gameMode != 'demo')
    {
        exit();
    }
    if($level != 'easy' && $level != 'medium' &&  $level != 'hard')
    {
        exit();
    }
    if($betValue < 15 || $betValue > 500)
    {
        exit();
    }
}
php

3 个回答

  1. Best Answer

    PS我刚开始学习PHP

    我的答案不会是关于如何用三行代码编写代码。

    1 函数名称

    通常动词放在名称的开头：get、set、update、check、validate等。函数/方法的名称应该反映它/他所做的事情。因此check_game_parameters：检查游戏参数。

    2 功能目的

    一个功能如此丰富exit且不显示错误消息并不严重。再多几个这样的功能，然后你会被使用var_dump.

    如果它是一个检查/验证函数，那么至少它应该返回trueor false。因此：如果传递的游戏参数满足某些预定条件，则返回true。在所有其他情况下 - false。

    3 函数参数

    在输入中，我们有三个参数，每个参数都通过两个或三个检查。目前总计 - 7 次检查。

    问题几乎总是由太长的方法引起，通常包含大量隐藏在它们通常包含的复杂逻辑下的信息。这里重构的主要类型是方法提取，它将一段代码变成一个单独的方法。——马丁·福勒。

    在我们的例子中，不是一个方法，而是一个函数。这三个参数是我们需要的三个单独的函数true，或者false。

    4 参数命名

    1. $gameMode- 我们检查游戏的参数，所以前缀game是多余的。
    2. $level伟大的。
    3. $betValue- 从上下文来看，很明显这就是这个意思。
    echo '<pre>';

function check_game_parameters($mode, $level, int $bet)
{
    return validate_mode($mode)
        && validate_level($level)
        && validate_bet($bet);
}

function validate_mode($mode) {
    return in_array($mode, [
            'real',
            'demo'
        ]);
}

function validate_level($level) {
    return in_array($level, [
            'easy',
            'medium',
            'hard'
        ]);
}

function validate_bet(int $bet) {
    return $bet >= 15 && $bet <= 500;
}

if (!check_game_parameters('real', 'hard', 15)) {
    echo 'Проверка не пройдена<br>';
    exit();
}

echo 'Проверка пройдена<br>';
    • 3

  2. 好吧，只有两种选择：

    function game_parameters_check($gameMode, $level, $betValue) {
    if($gameMode != 'real' && $gameMode != 'demo') exit();
    if($level != 'easy' && $level != 'medium' &&  $level != 'hard') exit();
    if($betValue < 15 || $betValue > 500) exit();
}

    任何一个

    function game_parameters_check($gameMode, $level, $betValue) {
    if(!in_array($gameMode, [ 'real', 'demo' ])) exit();
    if(!in_array($level, [ 'easy', 'medium', 'hard' ])) exit();
    if($betValue < 15 || $betValue > 500) exit();
}

    在第二个选项中，当然，最好将 gameMode 变量转换为小写。

    • 1 
  3. function check_game_parameters($gameMode, $level, $betValue)
{
    $allowedRule = [
        'level' => ['easy','medium','hard'],
        'game_mode' => ['real','demo'],

    ];

    if(! in_array($gameMode,$allowedRule['game_mode']) ||
        ! in_array($level,$allowedRule['level'])) exit();


    if($betValue <= 15 || $betValue > 400) exit();

}

    如果将来添加另一个关卡或游戏模式怎么办？添加新的如果？在这种情况下，将这个新模式\级别添加到数组中就足够了$allowedRule

    • 1

相关问题