在 if/else 语句的参数中使用 next()

Question

所以我很确定我错误地使用了 next 和 hasNext... 我正在尝试输入一串 ACGT 字符，然后计算各个结果字母。提前致谢。

import java.util.Scanner;

public class A1Novice {
    public static void main(String[] args){
        String input = "";
        Scanner s = new Scanner(System.in);
        Scanner j = new Scanner(input);
        System.out.println("Enter nucleobases (enter end when done)");
        while(true){
            input = s.next() + input;
            if(s.next().contains("end")){
                break;
            }
        }
        process(j, input);  
    }

    public static void process(Scanner j, String input){
        int a = 0, c = 0, g = 0, t = 0;
        while(j.hasNext()){
            if (j.next()=="A"){
                a++;
            }
            else if(j.next()=="C"){
                c++;
            }
            else if(j.next()=="G"){
                g++;
            }
            else if(j.next()=="T"){
                t++;
            }
            else{
                System.out.println("A count: " + a);
                System.out.println("C count: " + c);
                System.out.println("G count: " + g);
                System.out.println("T count: " + t);
                break;
                }
         }

    }
}

Answer 1

如前所述，您必须将下一个元素放入局部变量中。

但这还不够：请不要使用== 来比较字符串。使用String的方法equals，甚至equalsIgnoreCase。

你应该使用类似的东西：

String val = j.next();
if(val.equals("A")){ // you could use val.equalsIgnoreCase("A") to be case insensitive
    ...
} else if (val.equals("B")){
    ...

正如 aljipa 的回答所建议的那样，您可以使用开关。如果您希望您的测试不区分大小写，也可以编写如下内容：

String val = j.next();
switch (val.toUpperCase()) {
    case "A":
        a++;
        break;
    case "B":
        ...

Answer 2

把它放在一个局部变量中。

 String next = j.next();

并在你的 if 语句中使用它。否则，您将在每次检查中推进迭代器。

它应该是这样的：

public static void process(Scanner j, String input){
    int a = 0, c = 0, g = 0, t = 0;

    while(j.hasNext()){
        String n = j.next();
        if (n.equals("A")){
            a++;
        }
        else if(n.equals("C")){
            c++;
        }
        else if(n.equals("G")){
            g++;
        }
        else if(n.equals("T")){
            t++;
        }
        else{
            ...
        }
     }

}

另外，不要使用 == 来比较字符串。 == 运算符将检查它是否是同一个对象实例，但不检查内容是否相等。请改用equals 方法。

Answer 3

k-mera 的回答是正确的，但考虑使用 switch，因为它在无症状方面更有效：

public static void process( Scanner j, String input ) {
 int a = 0, c = 0, g = 0, t = 0;
 String ch;
 while( j.hasNext() ) {
     switch( ch = j.next() ) {
         case "A":
              a++;
              break;
     case "C":
          c++;
          break;
     case "G":
          g++;
          break;
     case "T":
          t++;
          break;
         default:
              System.out.println("A count: " + a);
              System.out.println("C count: " + c);
              System.out.println("G count: " + g);
              System.out.println("T count: " + t);
              return;
         }
     }
}

此外，如果您需要逐个字符地读取字符，我宁愿使用简单的 (char) System.in.read()，不需要用多余的 String 实例污染堆，这正是 Scanner.next() 所做的strong>除非您不以自定义方式标记输入。

Answer 4

迭代器上的 j.next() 基本上会进入下一个循环，因此您在每个循环上进行 4 个循环。