1

我遇到了愚蠢的问题，但仍然存在重大问题。我正在为类似“注册”的页面创建验证。我需要检查输入的值是否无害并且符合我的需要。感谢 Fuel 框架，这很容易。下一步...我还需要检查用户名或电子邮件是否未被占用。我已经编写了可以完成这项工作的方法。但是当我需要编写所有“if 语句”时，我的问题就来了。

这正是我现在所拥有的。我不确定这是正确的方法。我可以用“如果”和很多级别（我猜是嵌套）来写它。我可以在那里使用“if/else..if”。你会用什么？

if ( $validation->run() === true ) {

    if ( Diesel::usernameExists( $username ) === false ) {

        $error = 'This username is already taken! Try again...';

    }

    if ( Diesel::emailExists( $email ) === false ) {

        $error = 'This e-mail is already taken! Try again...';

    }


    if ( !isSet( $error ) ) {

        Diesel::signUp( $username, sha1( $password ), $email );


        Session::set_flash( 'notification', 'You have been successfully signed-up! Thanks...' );
        Session::set_flash( 'location', 'sign-in' );

        $this->response->redirect( 'notification' );

    }

} else {

    Session::set_flash( 'error', 'There was something wrong with validation! Try again...' );

    $this->response->redirect( 'sign-up' );

}

换句话说，我的目标是：

1) 检查输入的内容是否有害且符合我的需要 ( $validation->run())，

2) 一步一步……用户名不是已经被占用了吗（usernameExists()）？

3) 电子邮件不是已经被占用了吗（emailExists()）？

4) 可能还有更多...

例如，如果步骤#1 为假，应用程序不需要进一步执行步骤#2。之后我需要包含错误消息的字符串，但是，你知道，每个步骤的消息都是不同的。然后我可以传递消息来显示它（如果有消息）！当然，我可以编写有效的代码，但我正在寻找最佳实践……我必须遵循 DRY（不要重复自己）和 KISS（保持简单，愚蠢！）原则来保持我的代码可读且易于理解和维护。

4

2 回答 2

3

我会使用 if then else if，这是我认为最有意义的。这样，如果第一个 if 为真，那就是它会做的一切，否则它将进入下一个。我认为这是你想要的。

于 2011-05-15T14:34:49.040 回答
1

你为什么不从这些中制作验证方法呢？

http://fuelphp.com/docs/classes/validation.html在“扩展验证类”下检查，这样它们将在运行验证时运行。您可以按照“错误消息”中的说明设置错误消息 （顺便说一句，您也可以添加任何类、当前控制器或模型，使用$validation->add_callable('classname')whereclassname可以是类名或对象实例）

于 2011-05-19T15:10:07.550 回答