Почему метод не действует «правильно?»

Question

Я правильно отсканировал число, но методы не работают правильно. Первый ничего не делает, а второй - в бесконечный цикл.

Вызывается метод неправильно. Я не уверен что делать.

import java.util.Scanner; 
public class testSequence { 

public static void main(String[] args) { 
    Scanner scan = new Scanner(System.in); 
    System.out.println("Enter a number: "); 
    int enterNumber = scan.nextInt(); 
    System.out.println("1 for Iteration, 2 for Recursion: "); 
    int type = scan.nextInt(); 

    if (type == 1){ 
     computeIteration(enterNumber); 
    } else if (type == 2){ 
     computeRecursion(enterNumber); 
    } 
} 


public static int computeIteration(int enterNumber) { 
    int answer; 
    int multiplier = 1; 
    int count = 0; 
    int addend = 0; 
    if (enterNumber == 0) { 
     count++; 
     return enterNumber; 
    } else if (enterNumber == 1) { 
     count++; 
     return enterNumber; 
    } else { 

     for (int i = 0; i <= enterNumber; i++) {//need to adjust "i" for counter correction 

      enterNumber = (multiplier * 2) + addend; 
      addend = multiplier; 
      multiplier = enterNumber; 
      count += 1; 
     }//end for loop 
     answer = enterNumber; 
    }//end else 
    return answer; 
}//end computeIteration 

public static int computeRecursion(int n) { 
    int count = 0; 
    if (n == 0) { 
     count++; 
     return 0; 
    } else if (n == 1) { 
     count++; 
     return 1; 
    } else { 
     count++; 
     return computeRecursion(2 * (n - 1)) + computeRecursion(n - 2); 
    } 

}//end computerRecursion() 

}//end Sequence() 

Answer 1

Вы никогда не печатаете ответ.

if (type == 1){ 
     computeIteration(enterNumber); 
    } else if (type == 2){ 
     computeRecursion(enterNumber); 
    } 

Обратите внимание, как вы вызываете функции, но вы ничего не делаете с результатом.

вероятно Вы имели в виду:

if (type == 1){ 
    System.out.println(computeIteration(enterNumber)); 
    } else if (type == 2){ 
    System.out.println(computeRecursion(enterNumber)); 
    } 

Или, если вы хотите получить фантазии:

UnaryOperator<Integer> f = 
    type == 1 ? 
     computeIteration 
     : computeRecursion; 

System.out.println(f.apply(enterNumber)) ; 

просто прибавление, так как вы просили. Я использую тернарный оператор, потому что мне нужно выбирать между двумя вещами. В случае, подобном этому, он более аккуратный, чем полный оператор if.

UnaryOperator - это функциональный интерфейс. В основном, используя их, вы можете сохранить функцию внутри переменной. Это полезно, когда в таких случаях вы хотите выбрать между двумя функциями, чьи подписи одинаковы (обе ваши функции принимают int и возвращают int), и используют результат.

Я сохраняю одну из ваших функций в f, а затем назову ее путем записи f.apply(9) (apply «применяет» аргументы функции, называя ее).

Обратите внимание: вы не должны использовать функциональные интерфейсы только для ударов, так как они могут сделать код менее понятным. При правильном использовании, они могут сделать код много проще; особенно в сочетании с анонимными функциями.