2016-03-23 5 views
-2

私はアプリケーションをコンパイルして実行すると、その後は何も入力せずに整数のプロンプトを表示します。私は正しく、整数、メソッド、multiplynumbers()送信していない:?メソッドへの整数の送信

import java.util.Scanner; 

public class MultiplyNVRM2 
{ 
    private static int one=0; 

    private static int two=0; 

    private static int product =one * two; 



    public static void main(String [] args) 
    { 
    Scanner input = new Scanner(System.in); 

    System.out.printf("%nPlease enter an integer: "); 

     one = input.nextInt(); 

    System.out.printf("%nPlease enter another integer: "); 

     two = input.nextInt(); 

     input.nextLine(); 

     multiplyNumbers(); 


    }//end main() 

    public static int mulitplyNumbers() 
    { 

     product = one * two; 

    System.out.printf("%n%d * %d = %d", int one, int two, product); 


    }//end multiplyingNumbers 
} 
+3

この 'System.out.printf("%n%d *%d =%d "、int one、int two、product);は法的コードではありません。 –

+0

また、関数に整数を「送信」していません。変数1と2の値を使用する関数を呼び出すだけです。 –

+1

私はa)コードをコンパイルし、b)コードフォーマッタでコードを整理し、c)静的フィールドではなく引数として渡すことをお勧めします。 –

答えて

0

この操作を行います。

public static void mulitplyNumbers(int one, int two) { 
    int product = one * two; 
    System.out.printf("%n%d * %d = %d", one, two, product); 
} 

をと覚えてください。

  • をあなたのコードは何も返さない場合は、代わりにintのキーワードvoidを使用しています。結果を印刷してもう使用しないことに注意してください。
  • System.out.printfの引数と他の方法では、そのメソッドを呼び出すときに型を初期化する必要はありません。
  • パラメータとして整数を使用します。あなたが持つmain 1でメソッドを呼び出すことができます。

    multipleNumbers(one, two); 
    
+0

ありがとう、これは本当に私のためにそれをクリアしました。 – saophen

+0

助けてくれてうれしいです。あなたに一番の助けとなったここで最高の答えを受け入れるように、コンシェルにご相談ください。 –

0

この行は無効です:

System.out.printf("%n%d * %d = %d", int one, int two, product); 

あなたがいるので、あなたが方法printfのパラメータでワードint型を使用することはできません方法を宣言していません....あなたはそれを呼び出しています...

System.out.printf("%n%d * %d = %d", one, two, product); 
0
System.out.printf("%n%d * %d = %d",one,two, product); 
関連する問題