我有3个输入字段:用户名,密码和类型。
如果3个中的任何一个不正确,我只会得到1个错误消息,表明" usern&传球不正确"。
如何分解错误消息,以便向我显示密码,用户名或类型的值是否输入错误。
这些是我使用的2个文件:Login Model和LoginController。顺便说一句,下面的代码没有错误。
完美无缺。我只是想扩展它以分解错误信息。
LoginController文件:
DB string “BMW”
Query string “A BMW is a fancy car”
LoginModel文件:
public class LoginController implements Initializable {
/**
* Initializes the controller class.
*/
public LoginModel loginModel = new LoginModel();
@FXML private Label isConnected;
@FXML private JFXTextField txtUsername;
@FXML private JFXPasswordField txtPassword;
@FXML private ComboBox<String> comboType;
ObservableList<String> list = FXCollections.observableArrayList("admin", "manager", "clerk");
@Override
public void initialize(URL url, ResourceBundle rb) {
comboType.setItems(list);
if(loginModel.isDbConnected()) {
isConnected.setText("Connected");
}
else {
isConnected.setText("Not Connected");
}
}
public void Login (ActionEvent event) {
try {
if(loginModel.isLogin(comboType.getValue(), txtUsername.getText(), txtPassword.getText())) {
isConnected.setText("usern & pass is correct");
//closes login fxml file
((Node)event.getSource()).getScene().getWindow().hide();
//loads main interface fxml file
Stage primaryStage = new Stage();
FXMLLoader loader = new FXMLLoader();
Pane root = loader.load(getClass().getResource("/gui/uicomponents/Main.fxml").openStream());
MainController mainController = (MainController)loader.getController();
mainController.getUser("Hi " + txtUsername.getText());
Scene scene = new Scene(root);
scene.getStylesheets().add(getClass().getResource("/resources/css/Consolidated.css").toExternalForm());
primaryStage.setMaximized(true);
primaryStage.setTitle("Main Interface");
primaryStage.setScene(scene);
primaryStage.show();
}
else {
isConnected.setText("usern & pass is not correct");
}
} catch (SQLException ex) {
isConnected.setText("usern & pass is not correct");
Logger.getLogger(LoginController.class.getName()).log(Level.SEVERE, null, ex);
} catch (IOException ex) {
Logger.getLogger(LoginController.class.getName()).log(Level.SEVERE, null, ex);
}
}
}
答案 0 :(得分:2)
从安全角度来看,这不是一个好习惯,以揭示细节的哪一部分是不正确的,即,它会使您的应用程序更容易受到黑客攻击,因为您提供了更多提示userid
是否不正确或password
不正确或type
是否错误。
但是,如果您真的想根据自己的项目要求显示非常具体的错误消息,可以通过检查Type
和UserName
是否存在然后定义&amp;来实现。抛出自定义异常,例如TypeNotFoundException
或UserNameNotFoundException
例外情况,如下所示:
登录()方法:
public void Login (ActionEvent event) {
try {
if(loginModel.isLogin(comboType.getValue(),
txtUsername.getText(), txtPassword.getText())) {
//add your code
}
else {
isConnected.setText(" pass is not correct");
}
} catch (TypeNotFoundException ex) {
isConnected.setText("Type is not correct");
//add your logger
} catch (UserNameNotFoundException ex) {
isConnected.setText("Username is not correct");
//add your logger
} catch (SQLException ex) {
isConnected.setText("technical problem,please try after some time");
//add your logger
} catch (IOException ex) {
//add your logger
}
}
<强> isLogin():强>
public boolean isLogin(String type, String user, String pass) throws SQLException {
PreparedStatement preparedStatement = null;
ResultSet resultSet = null;
String query = "select * from users where type = ?
and username = ? and password = ?";
try {
//other code
//if returns result from db
if(resultSet.next()) {
return true;
}
else {
//String typeQuery = "select * from users where type = ?";
//Execute the typeQuery and throw TypeNotFoundException
//String userNameQuery = "select * from users where username = ?";
//Execute the userNameQuery and throw UserNameNotFoundException
return false;
}
} //other code as is
}
答案 1 :(得分:1)
简单地说,您的方法 isLogin()需要对传入数据进行更明确的分析。
例如,您可以先检查用户名是否已知。
但是你看到了;从安全角度来看,您可能不希望向用户提供此类详细信息。假设攻击者猜测用户名。当你给他不同的消息时,用户身份不明,密码无效;然后你帮他攻击你。您可以将此类信息放入日志文件中,但不必将所有详细信息都用于使用!谈论安全性:似乎您打算将密码作为纯文本存储在数据库中。那绝对不行!
但鉴于您的客户直接与数据库通信,安全性似乎并不是您的首要任务。