Thread: help whats wrong

Been doing this one and totally miffed with this even a tutor looked and could not see error the problem line indicates ")" expected and highlights the line i have made bold, perhaps another set of eyes can find the problem.

//this program decides amount of calories to a daily activity and weight

import java.util.Scanner;
 
public class calories1
{
    public static void main (String [] args)
 
{
 
        Scanner input = new Scanner (System.in);
 
        // delcaring variables
        double weight;
        String activityFactor;
 
        // program is now going to request input from user
 
        System.out.println("please enter your weight in Kilograms: ");
        weight  = input.nextDouble();
 
        System.out.println("Please enter if your lifestyle is active or inactive (a/i)");
        activityFactor = input.next();
 
        // program will calculate dependant on activity using if and else and or ||
 
            if (activityFactor.equals ("a") || activityFactor.equals ("A"))
           [B] System.out.println("your recommended calorie intake is " + (weight x 30)); [/B]
 
            else if (activityFactor.equals ("i") || activityFactor.equals ("I"))
            System.out.println("your recommended calorie intake is " + (weight x 20));
 
            else
            System.out.println("invalid input");
            System.out.println("GOODBYE");
 
        }
    }

import java.util.Scanner;
 
public class calories1
{
public static void main (String [] args)
 
{
 
Scanner input = new Scanner (System.in);
 
// delcaring variables
double weight;
String activityFactor;
 
// program is now going to request input from user
 
System.out.println("please enter your weight in Kilograms: ");
weight = input.nextDouble();
 
System.out.println("Please enter if your lifestyle is active or inactive (a/i)");
activityFactor = input.next();
 
// program will calculate dependant on activity using if and else and or ||
 
if (activityFactor.equals ("a") || activityFactor.equals ("A")) {
 
    System.out.println("your recommended calorie intake is " + (weight * 30));
}
else if (activityFactor.equals ("i") || activityFactor.equals ("I")) {
 
    System.out.println("your recommended calorie intake is " + (weight * 20));
}
 
else {
    System.out.println("invalid input");
    System.out.println("GOODBYE");
}
 
}
}

are you trying to multiply the weight? using "x" as an operator?
if thats the case then i edited your post... in programming the operator in multiplication is not 'x'
its an askterisk '*'

/ - division
* - multiplication

and another thing ill give you a simple advice..

whenever your are making a statement for an if-else always put it inside an opening and closing bracket { }
even if the statement is single.. single or compound always put it in a braces

because if you dont you will have some compiling error. ahm or some logic error..


anyway is this what you want?

ironic thing is that the tutor is teaching this and did not spot any of that that is bad and worrying because i on a degree course.

Thanks for you help

hmm in some sense thats not good for you. but in someway its very efficient because you will learn how to correct your own mistakes....
anyway im glad i statisfied you regarding with your post