2016-06-28 4 views
-2
import java.util.Scanner; 
public class hey { 



public static void main(String[] args) 
{ 
    Scanner in = new Scanner(System.in); 
    System.out.print("Please enter an integer between 6 and 12, inclusive: "); 
    int num = in.nextInt(); 
    System.out.print(num); 
    System.out.println(); 
    boolean result = shouldProcess(num); 
    processInput(result, num); // passing the result and num 

} 

public static boolean shouldProcess(int n) 
{ 
    if (n>=6 && n<12) 
    { 
     return true; 
    } 
    else 
    { 
     return false; 
    } 
} 

public static void processInput(boolean result2, int num) // added int num argument 
{ 
    if (result2 == true) 
    { 

      int sum = 0; 
      for (int i = 1; i <=num; i++) 
      { 
        sum +=i; 
      } 
      System.out.println("The sum from 1 to " + num+ " is: " +sum); 

    } 
    else 
    { 
     System.out.println("Number is outside of acceptable range"); 
    } 
} 
} 

このコードは、与えられた数が6から12までの数である場合、shouldProcessでブール値を返します。それから、私はメソッドprocessInputを使ってshouldProcessを使って、計算して和を出力できるかどうかを判断します(メソッドshouldProcessがtrueを返す場合にのみ、findSumというメソッドを呼び出します)。メソッドを使用した基本的なJavaプログラミング

public static void processInput(boolean result2, int num) 
{ 
    if (result2 == true) 
    { 

      int sum = 0; 
      for (int i = 1; i <=num; i++) 
      { 
        sum +=i; 
      } 
      System.out.println("The sum from 1 to " + num+ " is: " +sum); 

    } 
    else 
    { 
     System.out.println("Number is outside of acceptable range"); 
    } 
+0

しかし、あなたはすでに合計を構築していますか? – Fildor

+0

どのように私はその合計を取って、processInputに続くfindSumという独自のメソッドにします – basicjava

+1

shouldProcessは包括的ではありません。あなたは<= 12の代わりに12未満を言っています – Beto

答えて

0
if (result2 == true) 
{ 

     int sum = 0; 
     for (int i = 1; i <=num; i++) 
     { 
       sum +=i; 
     } 
     System.out.println("The sum from 1 to " + num+ " is: " +sum); 

} 

次のようになります:

if (result2 == true) 
{ 

     System.out.println("The sum from 1 to " + num+ " is: " +findSum(num)); 

} 

public static int findSum(num){ 
    int sum = 0; 
    for(int i = 1; i <= num; i++){ sum+= i; } // usually I'll use int i = 0 and add an offset in the loop. 
    return sum; 
} 
私のコードでは、このfindSum方法は、ブロックと作りますところで

:あなたのShouldProcessメソッドを短縮することができます。

public static boolean shouldProcess(int n) 
{ 
    return (n>=6 && n<=12); // both inclusive! 
} 
+0

どこにpublic static int findSum(num)を編集しますか? – basicjava

+0

okay nevermind私はそれを非常に感謝した – basicjava

関連する問題