Использование next() в параметре if/else statement

Question

Так что я уверен, что использую следующий и 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{ 
      ... 
     } 
    } 

} 

Plus, не использовать == для сравнения строк. Оператор == проверяет, является ли он одним и тем же экземпляром объекта, но не является ли контент равным. Вместо этого используйте метод equals.

Answer 3

j.next() на итераторе по существу переходит к следующему циклу, поэтому вы делаете 4 цикла в каждом цикле. Ответ

Answer 4

K-MERA является правильным, но считают, с помощью переключателя, так как это бессимптомно более эффективным:

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; 
     } 
    } 
} 

Кроме того, я бы предпочел использовать простой (полукокс) System.in.read(), если вам нужно прочитать char по char нет необходимости в загрязнении вашей кучи избыточными экземплярами String, что является именно тем, что Scanner.next() делает , если только вы не вносите входной сигнал обычным способом.