エラーを解決したい。

Question

import java.util.Scanner;

 class Main {
  public static void main(String[] args) {
    Scanner scanner = new Scanner(System.in);
    System.out.print("名前:");
    String firstName = scanner.next();
    
    System.out.print("名字:");
    String lastName = scanner.next();
    
    System.out.print("年齢:");
    int age =  scanner.nextInt();
    
    System.out.print("身長(m):");
    double height =  scanner.nextDouble();
    
    System.out.print("体重(Kg):");
    double weight =  scanner.nextDouble();
    

    Person.printData(firstName, lastName,age, height, weight) ;
      
    } 
}

main.java

  class Person {
  
   public static void printData(String firstName, String lastName, int age, double height,  double weight) {
      
     
    System.out.println("名前は" +fullName(firstName, lastName) + "です");
    
    System.out.println("年齢は" + age + "歳です" );
    
    if (age >= 20) {
       System.out.println("成年者です");
     } else {
       System.out.println(" 未成年者です");
    }
    System.out.println("身長は" + height + "mです");
    System.out.println("体重は" + weight + "kgです");
    
    
    
   double bmi =  bmi(height, weight);
    System.out.println("BMIは" + Math.round(bmi) + "です");
   
    }  
   public static String fullName(String firstName, String lastName) {
     return firstName + " " + lastName;
   }
   
    public static double bmi(double weight, double height) {
    return weight /  height /  height ;
   }
   
  }
    
  

Person.java

![]

何度も答えを見ながらしましたが、出来ませんでした。
ご教示願います。

Comments

[m.ts10806]

何をしたくて組んだコードなのか分かりません。
要件を具体的に記載してください。
「実現したいこと」「起きていること」
コードもそうですがエラーもコピ＆ペーストが基本です。
単なるデバッグ依頼は非推奨です。

[m.ts10806]

下記のような結末になるのを懸念しています。
https://teratail.com/questions/78924#reply-124263
「自分で解決すべき」は全ての問題に言えることです。
他人の力を借りることを選択するのであれば、「自身がこれだけやってもダメだった」ことをきちんと記載すべきです。
「答えを見ながらしたけどできなかった」で伝わることはほとんどありませんよ。

Answer 1

メソッド定義と、呼び出し側で身長と体重が逆になってますね。

double bmi =  bmi(height, weight);

public static double bmi(double weight, double height)

Answer 2

bmiメソッドの引数は(weight, height)ですが、printDateメソッド内でbmiメソッドを呼び出す際に(height, weight)で呼び出しています。2もいかない数を50で割りますから、丸めたら0になります。

Comments

[goforward]

ありがとうございました。問題解決しました