请帮助并优化我的代码。
用户必须输入某些值(日期、时间等)。如果您输入的内容不正确,您将收到重复请求。
需要检查他输入的字符串是否包含指定的参数,例如 format 中的时间ЧЧ:ММ或 format 中的日期ДД.ММ.ГГ。
下面给出了检查时间的示例。
我写了一个输入检查,但是:
- 我可能没有考虑到某些事情;
- 当然,这项检查可以做得更好、更简单。
protected void checkSymbbol() {
String valuez = "1df6:sdf01";
String value = valuez.replaceAll("[^\\d:]", "");
if (value.length() == 5) {
if (value.contains(":")){
System.out.println("верно");
} else {
System.out.println("ошибка");
}
} else
System.out.println("ошибка");
}
在此示例中,检查用户是否输入了16:01.
首先,上面的方法代码
checkSymbbol实际上是没有用的,因为它检查一些内部字符串,而不是输入参数,并且也不返回检查结果,而是打印一些东西。其次,上面的代码使用的不仅仅是奇怪的逻辑 - 首先它从输入字符串中删除除数字和冒号之外的所有字符
':',然后另外检查行中冒号的长度和是否存在,无论格式如何。也就是说,输入的字符串不仅可以包含各种垃圾,而且即使其中没有一个数字,结果也可以是“正确的”,例如,对于这样的字符串,String valuez = "abcd:<hello>[{}]::qwertyu:!~#$%&:";呈现的代码也会打印"верно"- 变量中将剩下 5 个冒号value。因此,如果任务是提供一对由冒号分隔的两个字符的输入,则最初不应有机会输入任何垃圾以查找其中有用的片段。
设置输入格式并使用标准方法更容易、更正确
String::matches(String regex)。然后,如果有必要,可以将用户预先清除的数据提交到这样的方法的输入:
同样,在检查格式时,您可以引入额外的限制,例如,如果不仅检查任何两位数字,还检查以小时/分钟为单位的时间:
注意:如果您有大量类似的检查使用,
String::matches您可能还需要一点优化,以便每次调用都不会创建正则表达式的实例Pattern,因为该调用input.matches(regex)相当于链式调用Pattern.compile(regex).matcher(input).matches()。由于最初的问题仍然是关于检查字符,值得指出的是,您可以使用更快的方法单独检查是否存在禁止字符
Matcher::find,然后检查格式,然后就不需要清除输入字符串:如果原始任务归结为处理已知的数据类型(例如日期/时间),则以相应方法的形式使用日期/时间 API 中的现有功能会更容易且更
DateTimeFormatter正确LocalDate::parse。LocalDateTime::parse