javaの引数での質問です。メソッド内での引数の呼び出しがうまくいかないです。

Question

前提・実現したいこと

javaの引数での質問です。

二つ目のメソッドにてmultiplication(false,out);
と呼び出すとうまくいきません。改善方法があればご指摘お願いします。

前提として実現したいことは出来ています。
1つめのメソッド multiplication 最終的に実行させたいメソッド。
このメソッドで，【引数を4つ受け取って】掛け算を行う。
第一引数と第二引数を掛け算し，第三引数の値が true であるときにかぎって，掛け算の結果をさらに２乗し。また，計算結果の値を画面に表示処理。

2つ目のメソッド
このメソッドは【引数を2つ受け取って】１つめのメソッドを呼び出し。
1つめのメソッドを呼び出す際に足りない情報を【デフォルト値】で補う。

以下のコードでやりたいことは実現できているのですがmultiplication(false,out);を呼び出すのがうまくいきません
ご指摘お願いします。

package org.mypackage.sample;

import java.io.IOException;
import java.io.PrintWriter;

import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;


/**
 * Servlet implementation class Method3
 */
@WebServlet("/Method3")
public class Method3 extends HttpServlet {
	private static final long serialVersionUID = 1L;

    /**
     * @see HttpServlet#HttpServlet()
     */
    public Method3() {
        super();
        // TODO Auto-generated constructor stub
    }

	/**
	 * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse response)
	 */

   /* 一つ目のメソッド*/
    void multiplication(int num1,int num2,boolean bl,PrintWriter pw){

    	/*第一引数と第二引数の掛け算*/
       int num3=num1*num2;
       if(bl) {num3=num3*num3;}

       pw.print(num3);

    }

    /*二つ目のメソッド*/
    void multiplication(boolean bl,PrintWriter pw) {
    	/*int num1 num2のデフォルト値を設定*/
    	pw.print("<br>");
    	multiplication(1,1,true, pw);

    }

	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
		 response.setContentType("text/html;charset=UTF-8");
		    try (PrintWriter out = response.getWriter()) {

		        // multiplicationメソッドの呼び出し

		    	multiplication(2,2,true,out);

		    	multiplication(true,out);
		    }

	}


	/**
	 * @see HttpServlet#doPost(HttpServletRequest request, HttpServletResponse response)
	 */
	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
		doGet(request, response);
	}

}

Comments

[y_waiwai]

うまくいかない、とはどうなるんでしょうか

[shinami]

>前提として実現したいことは出来ています。
>以下のコードでやりたいことは実現できているのですが
>multiplication(false,out);を呼び出すのがうまくいきません
実現できてはいるけれども実現できていない？
これでも良いけれどもっと良い方法があるかという事でしょうか？

Answer 1

multiplication(2,2,true,out);
multiplication(true,out);
となっているので
num1=2、num2=2、blはtrueの固定となります。
falseの時はうまくいかない？とはどういう事でしょうか？
multiplication(false,out);に変えて確かめればできるのではないでしょうか？
ただし、メソッド２でメソッド１を呼び出すときにtrue固定になっているのでやっぱり同じで二乗します。
※補足：固定と言っていますけど渡した直後の話です。
finalがないので渡した後に代入はできましたね。
メソッド１の呼び出しでもfalseにするのか、それともblにするのか。
multiplication(false.out);とした時にはどのようになる事をお望みでしょうか？

void multiplication(int num1,int num2,boolean bl,PrintWriter pw){

  /*第一引数と第二引数の掛け算*/
  int num3=num1*num2; //

  if(bl) { //if(bl)はif(bl == true)と同じです。
    num3=num3*num3; //trueの時だけ計算されます。
  } 

  pw.print(num3);

}

    /*二つ目のメソッド*/
void multiplication(boolean bl,PrintWriter pw) {

  /*int num1 num2のデフォルト値を設定*/
  pw.print("<br>");
  multiplication(1,1,true, pw); //blがfalseでもここでtrueとしているのでやっぱり二乗

}

Answer 2

protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
         response.setContentType("text/html;charset=UTF-8");
  multiplication(2,2,true,out);

  multiplication(true,out);
}

これに対し、

/*二つ目のメソッド*/
void multiplication(boolean bl,PrintWriter pw) {
  /*int num1 num2のデフォルト値を設定*/
  pw.print("<br>");
  multiplication(1,1,true, pw); /////// ここの第３引数は bl では？
}

bl の値がどこにも使われてないので、falseを指定していても動かないのでしょう。

Answer 3

/二つ目のメソッド/

void multiplication(boolean bl,PrintWriter pw) {
    /*int num1 num2のデフォルト値を設定*/
    pw.print("<br>");
    multiplication(1,1,true, pw);
}

１つ目の引数は使用されてないですね