Java code is not allowing my variable to be used

Question

When I try to run this code, it won't let me use "bmi". I'm trying to make a simple Body Mass Index calculator, but I don't understand why it won't work. If you could rewrite the code properly, it'd help me learn better.

import java.util.Scanner;

public class Bmi {
    int weight;
    int height;
    int bmi = weight /(height*height) * 703;

    public static void main(String[] args) {
        Scanner input = new Scanner(System.in);
        System.out.print("Enter weight: ");
        int weight = input.nextInt();
        System.out.print("Enter height: ");
        int height = input.nextInt();
        System.out.println(bmi);
   }

}

Also, bmi is currently evaluated once, with the default values of the height and weight (which is zero). You have to calculate it after the user has entered the values. — Andy Turner
– Andy Turner, Commented Feb 2, 2018 at 19:02
Yup, use static or use new on the whole BMI class to get an instance of the variable. — markspace
– markspace, Commented Feb 2, 2018 at 19:03
@Janar actually, it would be better to define bmi as a local variable, and remove the other two. — Andy Turner
– Andy Turner, Commented Feb 2, 2018 at 19:03
and the same variable names are used in the method which have local scope, which hides the class variables. And using int in the formula will lose the decimal places. — Andrew S
– Andrew S, Commented Feb 2, 2018 at 19:06

Suresh Atta · Accepted Answer · 2018-02-02 19:14:10Z

4

You are trying to use non static members inside static context.

You don't really need any instance/static variables here. Just go with local variables.

import java.util.Scanner;

public class Bmi {
public static void main(String[] args) {
     Scanner input = new Scanner(System.in);
     System.out.print("Enter weight: ");
     int weight = input.nextInt();
     System.out.print("Enter height: ");
     int height = input.nextInt();
     int bmi = weight /(height*height) * 703;
     System.out.println(bmi);
}
}

Although it is not happen in real world, when your height is greater than weight you end up in zero as integer division is happening. Better you change them to doubles. Give a read Why is the result of 1/3 == 0?

public static void main(String[] args) {   
        Scanner input = new Scanner(System.in);
        System.out.print("Enter weight: ");
        double weight = input.nextInt();
        System.out.print("Enter height: ");
        double height = input.nextInt();
        double bmi = weight / (height * height) * 703;
        System.out.println(bmi);
    }

Finally

int bmi = weight /(height*height) * 703;

That statement won't keep a track on values of weight and height. You need to reevaluate each time when they change.

edited Feb 2, 2018 at 19:14

answered Feb 2, 2018 at 19:06

Suresh Atta

122k38 gold badges206 silver badges315 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Michael K. Over a year ago

So is double generally better to use than int?

Suresh Atta Over a year ago

@MichaelK. No. Updated my answer and added a link. Give read.

Sahil Chhabra · Accepted Answer · 2018-02-02 19:40:38Z

1

You should never combine the worker and user code in a same class.

Better to create a separate class for calculator say BmiCalculator and define a static method to calculate bmi say calculateBmi. Static because its just dependent on the input it needs and nothing else. Then call and use this static method directly in your CallerClass

Please refer the below code:

import java.util.Scanner;

class BmiCalculator {

 public static double calculateBmi(double weight, double height){
    return weight /(height*height) * 703;
 }

}

public class CallerClass{
  public static void main(String[] args) {
     Scanner input = new Scanner(System.in);
     System.out.print("Enter weight: ");
     double weight = input.nextInt();
     System.out.print("Enter height: ");
     double height = input.nextInt();
     System.out.println(BmiCalculator.calculateBmi(weight,height));
  }
}

edited Feb 2, 2018 at 19:40

answered Feb 2, 2018 at 19:22

Sahil Chhabra

11.9k5 gold badges69 silver badges64 bronze badges

2 Comments

Suresh Atta Over a year ago

Good. Works great. Better you add few words about your answer.

Sahil Chhabra Over a year ago

@ꜱᴜʀᴇꜱʜᴀᴛᴛᴀ thanks for the suggestion. Updated the answer.

fxrbfg · Accepted Answer · 2018-02-02 19:40:45Z

It's because you trying to access instance context from static context. Simpliest way to fix this is add static modificator to your fields and remove bmi calculation from bmi field assignment because all static is initialized when class loaded. More concise way to write this code is that:

    import java.util.Scanner;

    public class Main {

        public static void main(String[] args) {
            Scanner input = new Scanner(System.in);
            System.out.print("Enter weight: ");
            int weight = input.nextInt();
            System.out.print("Enter height: ");
            int height = input.nextInt();
            Bmi bmi = new Bmi(weight, height); //create new bmi instance
            System.out.println(bmi.value());   // get it's value 
            System.out.println(bmi);            // calls toString() method (object text representation) 
//that we already override in Bmi class
        }
    }

We decouple Bmi calculation from user input with way to create separate class, now Bmi class knows nothing about user input and it's good because in this way we increased cohesion and reduce coupling in Main and Bmi classes. It helps to code reuse in different projects and ease mantain efforts. See https://en.wikipedia.org/wiki/GRASP_(object-oriented_design) for more information.

public class Bmi { //place to separate file with name Bmi.java
    private int weight;  
    private int height;

    public Bmi(int weight, int height) {
        this.weight = weight;
        this.height = height;
    }

    /** 
        returns a bmi value
    */
    public int value() {
        return weight / (height  * height) * 703;
    }

    public String toString() {
        return value();
    }

}

Collectives™ on Stack Overflow

Java code is not allowing my variable to be used

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related