[英]Are there any ways to make this simple program more efficient?
該程序非常簡單:它接受一個字符串並用“_”替換元音,然后打印反轉的字符串。 我只是在尋找使我的代碼更專業的方法,並希望獲得一些提示。
import java.util.Scanner;
public class reverse_string {
public static void main (String[] args){
Scanner scan = new Scanner(System.in);
String input, reverseInput;
String legolas = new String();
System.out.println("Enter any input");
input = scan.nextLine();
StringBuilder newString = new StringBuilder(input);
System.out.println("The data you entered is: "+input);
for (int i =0; i <input.length();i++){
if (testIfVowel(input.charAt(i))){
newString.setCharAt(i,'_');
}
else{
newString.setCharAt(i, input.charAt(i));
}
}
for(int i = input.length()-1;i>=0;i--){
legolas = legolas +input.charAt(i);
}
reverseInput=reverseOrder(newString);
System.out.println("Your old data was: "+input+"\nYour new data is: "+newString +"\nYour old data in reverse is: "+legolas+"\nYour new data in reverse is: "+ reverseInput);
}
public static boolean testIfVowel(char x){
if(x =='a'||x=='A'||x=='e'||x=='E'||x=='i'||x=='I'||x=='o'||x=='O'||x=='u'||x=='U'){
return true;
}
else{
return false;
}
}
public static String reverseOrder(StringBuilder x){
String string= new String();
for(int i = x.length()-1;i>=0;i--){
string = string + x.charAt(i);
}
return string;
}
}
既然您只是用_
替換元音,為什么不使用 java 為您提供的replaceAll
方法和一些與元音匹配的正則表達式。
str.replaceAll("[aeiou]", "_");
並使用反轉它
reverseInput = new StringBuilder(str).reverse().toString()
所以這就是你的最終代碼在改進后的樣子
public static void main(String[] args) {
Scanner scan = new Scanner(System.in);
String input, reverseInput;
System.out.println("Enter any input");
input = scan.nextLine();
reverseInput = new StringBuilder(input).reverse().toString();
String withoutVowels = input.replaceAll("[aeiou]", "_");
String withoutVowelsReversed = new StringBuilder(withoutVowels).reverse().toString();
System.out.println("The data you entered is: " + input);
System.out.println("Your old data was: " + input + "\nYour new data is: " + withoutVowels + "\nYour old data in reverse is: " + reverseInput + "\nYour new data in reverse is: " + withoutVowelsReversed);
}
我建議您的邏輯不必要地復雜。 例如,你真的需要一個只使用一次的testIfVowel
方法嗎? 我對此表示懷疑。
我會做這樣的事情(偽代碼):
resultString = '';
for (i = 0; i < length(s); i++) {
if ("AEIOUaeiou".indexOf(s[i]) != 0) {
resultString = '_' + resultString;
} else {
resultString = s[i] + resultString;
}
}
(您可以使用任何語言執行此操作。)
請注意元音測試現在如何簡單明了,顯然適用於大寫和小寫版本。 並且,如何創建“反向字符串”,一次一個字符,只需在初始為空的 resultString前面插入新字符即可。
也許最重要的是(對我來說),這個版本“讀起來很容易”。 我不需要查看太多代碼來檢查它是否可以工作。 如果你願意,它“直截了當”。
干凈多了,我覺得……
這似乎不需要比以下更難:
String result =
new StringBuilder(input.replaceAll("[AEIOUaeiou]", "_"))
.reverse()
.toString();
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.