BigDecimal 四捨五入 使い方

Question

前提・実現したいこと

BigDecimalを使って小数第〇位まで四捨五入させたい。初心者な質問ですいません。

発生している問題・エラーメッセージ

ローカル変数 bd1 の値は使用されていませんと出てしまいます。
エラーメッセージ
ローカル変数bd1の値は使用されていません

該当のソースコード

import java.math.BigDecimal;
import java.math.RoundingMode;

public class HealthCheckLogic {
    public void execute(Health health) {

        // BMIの計算
        double height = health.getHeight();
        double weight = health.getWeight();
        double bmi = 10000 * (weight / height / height);
        health.setBmi(bmi);
        BigDecimal bd = new BigDecimal(bmi);
        BigDecimal bd1 = bd.setScale(2, RoundingMode.HALF_UP);

試したこと

BigDecimal bd = new BigDecimal.valueOf(bmi);や
doubleValue()も使いましたが、結果は同じでした。

補足情報（FW/ツールのバージョンなど）

eclipse
Tomcat8

Comments

[swordone]

｢結果は同じでした。｣といわれても、結果って何のことか分からないです。

[退会済みユーザー]

すみません。ローカル変数bd1の値は使用されていませんというエラーです。

[swordone]

最終的な目標がわかりません。四捨五入してどうしたいのか？

[退会済みユーザー]

bmiの計算結果を小数第三位で四捨五入して第二位まで表示させたいと思っています。今の自分の理解では、bdにbmiの計算結果が格納されており、bd1にbdの小数を四捨五入した値が入っていると思っていたのですが、そこが間違いということでしょうか？

[swordone]

なら、それを出力すればいいだけでは？
何をそんなにウジウジ言ってるのか分かりません。

Answer 1

ローカル変数bd1の値は使用されていません

これはエラーではなく警告文です。
メソッドのローカル変数が宣言されているにも関わらず、その変数をメソッド内のどこでも使っていないことによる警告です。メソッド内で使いたいからこそのローカル変数なのであって、それをどこでも使っていないのはおかしくないですか？とコンパイラが警告してくれているのです。
警告を消すだけなら、その変数を何かのメソッドに渡すか、変数のメソッドを起動するかすればとりあえず消えます。

// 変数を何かのメソッドに渡す
System.out.println(db1);

// 変数のメソッドを起動する
// 注：この結果を格納する変数を宣言するとまた同じ警告が出る
db1.doubleValue();

ただ、コードの流れから言ってこれは本質的にやりたいことではないですよね。
ここでいくらいじってもhealthという謎の変数に渡されたbmiの値に全く影響はありませんから。
何をしたいのかよく確認してください。

Answer 2

bd1というBigDecimal型の変数を新たに宣言する必要がないことに気づき、
bd = bd.setScale(2, RoundingMode.HALF_UP);
とすることで解決しました。
モデルやサーブレットのこともお伝えしておらず、自分のやりたいことがうまく伝えられなかったにも関わらずお答えくださりありがとうございます。
以後質問の質もあげていけるよう頑張りたいと思いますので、また機会がありましたらよろしくお願いします。

Answer 3

ローカル変数bd1の値は使用されていません

変数への代入文はあるけど、その変数を読み取る処理がないだけでは

Comments

[退会済みユーザー]

回答ありがとうございます。変数の読み取り処理というのは、System.outなどの値の出力文がないということでしょうか？

[gentaro]

最終行からひとつ上の行の
BigDecimal bd = new BigDecimal(bmi);
のbdは、最終行の
BigDecimal bd1 = bd.setScale(2, RoundingMode.HALF_UP);
をみればbdが使われてることがわかります。
では、同じようにbd1がどこかで使われてますか？

[momon-ga]

yes < 変数の読み取り処理というのは、System.outなどの値の出力文がないということでしょうか？