Program to output user entered info and calculate dosage based on species.

[Rebecca Berry]

Hello again all!  I am getting an error message and my code will not compile. I can not seem to figure out why. So any pointers in the right direction would be appreciated. The object of this program is to request user input, then return the input along with medication dosages based on the user's input. As I have mentioned before I am very new to Java. I am having some issues with the Java class I am in since there is really no instructor guidance. I  was told to purchase a book and given assignments that are not corresponding to the information in the book as we go. So I apologize for the funky code and thanks in advance for the pointers.
The error message that is showing up when I attempt to run my code is:

Exception in thread "main" java.lang.UnsupportedOperationException: Not supported yet.
at pet.Pet.<init>(Pet.java:15)
at pet.PetDemo.main(PetDemo.java:155)
C:\Users\Violettwilight\AppData\Local\NetBeans\Cache\8.1\executor-snippets\run.xml:53: Java returned: 1
BUILD FAILED (total time: 0 seconds)

And my code is as follows ( I should mention a large chunk of this code has been copied from source code from the course book-per professors instructions. I just had to make a few tweeks):

package pet;
import java.util.Scanner;

public class PetDemo
{
    private String name;
    private int age;
    private double weight;
    private boolean speciesDog;
    
    public String ToString( )
    {
        return ("Name: " + name + "Type" + speciesDog + " Age: " + age + " years"
                       + "\nWeight: " + weight + " pounds");
    }

    public PetDemo(String initialName, boolean speciesDog, int initialAge, 
                                          double initialWeight)
    {
        name = initialName;
        if ((initialAge < 0) || (initialWeight < 0))
        {
            System.out.println("Error: Negative age or weight.");
            System.exit(0);
        }
        else
        {
            age = initialAge;
            weight = initialWeight;
            speciesDog = true;
        }
    }

    public void set(String newName, int newAge, double newWeight, boolean type)
    {
        name = newName;
        if ((newAge < 0) || (newWeight < 0))
        {
            System.out.println("Error: Negative age or weight.");
            System.exit(0);
        }
        else
        {
            age = newAge;
            weight = newWeight;
            speciesDog = type;
        }
    }

    public PetDemo(String initialName)
    {
        name = initialName;
        age = 0;
        weight = 0;
    } 

    public void setName(String newName)
    {
        name = newName; 
    }

    public PetDemo(int initialAge)
    {
        name = "No name yet.";
        weight = 0;
        if (initialAge < 0)
        {
            System.out.println("Error: Negative age.");
            System.exit(0);
        }
        else
            age = initialAge;
    }

    public void setAge(int newAge)
    {
        if (newAge < 0)
        {
            System.out.println("Error: Negative age.");
            System.exit(0);
        }
        else
            age = newAge;
    }

    public PetDemo(double initialWeight)
    {
        name = "No name yet";
        age = 0;
        if (initialWeight < 0)
        {
            System.out.println("Error: Negative weight.");
            System.exit(0);
        }
        else
            weight = initialWeight;
    }

    public void setWeight(double newWeight)
    {
        if (newWeight < 0)
        {
            System.out.println("Error: Negative weight.");
            System.exit(0);
        }
        else
            weight = newWeight;
    }

    public PetDemo( )
    {
        name = "No name yet.";
        age = 0;
        weight = 0; 
    }

    public String getName( )
    {
        return name;
    }

    public int getAge( )
    {
        return age;
    }

    public double getWeight( )
    {
        return weight;
    }
    public double acepromazine()
    {
        if(speciesDog)
            return(weight/2.2) * (0.03/10);
        else
            return (weight/2.2) * (0.002/10);
    }
    public double carprofen()
    {
        if(speciesDog)
            return (weight /2.2) * (0.5/12);
        else
            return (weight/2.2) * (0.25/12);
    }



    public static void main(String[] args)
    {
        Pet usersPet = new Pet("Jane Doe");
        
        System.out.println("My records on your pet are incomplete.");
        System.out.println("Here is what they currently say:");
        System.out.println("");

        Scanner input = new Scanner(System.in);
        System.out.println("Please enter the pet's name:");
        String name = input.nextLine( );
        System.out.println("Is your pet a dog? (Y/N);");
        boolean speciesDog;
        String choice = input.nextLine();
        if(choice.equalsIgnoreCase("y"))
           speciesDog = true;
        else
           speciesDog = false;
       
        System.out.println("Please enter the pet's age:");
        int age = input.nextInt( );
        System.out.println("Please enter the pet's weight:");
        double weight = input.nextDouble( );
        
        usersPet.set(name, speciesDog, age, weight);
  
        System.out.println("My records now say:");
        System.out.println(usersPet);
        System.out.println("Dose of Carprofen: " + usersPet.carprofen());
        System.out.println("Dose of Acepromazine: " + usersPet.acepromazine());
       
       
   
    }
}

[Jeanne Boyarsky]

Rebecca,
The error is actually occurring in the Pet class. Can you show that class too?

[Ganish Patil]

Seems mistakenly wrote package name rather than PetDemo class name here, as Jeanne mentioned.

Pet usersPet = new Pet("Jane Doe");

Also need to check the order of arguments here on line no 176

usersPet.set(name, speciesDog, age, weight);

with respect to method declaration on line no 35

public void set(String newName, int newAge, double newWeight, boolean type)

[Campbell Ritchie]

Please always quote the details of the book, to avoid copyright problems, and also the page number, so we can find the source if we happen to have the book.
That Exception looks like what you get from NetBeans if you tell is to create a class with a number of methods. All methods have this structure (or similar):-

public void myMethod(int i)  {
    throw new UnupportedOperationException("not implemented yet");
}

That is useful for development; if you try a method which you have not yet completed, you suffer an Exception and you know about it. When you implement each method, you are supposed to delete the line about the Exception.

There is something not right about your drug methods. I think you should have a Drug class with name and mg per kilo fields instead of those methods. What is the 2.2 Is that some sort of conversion between lb and kg? Surely it is 2.205?

[Ganish Patil]

I think it would be easier to find the error, if you follow general format like first variable declaration, constructors, setter & getter methods then main method.

You forgot to call ToString method here on line no 179.

System.out.println(usersPet);

It is good to use curly braces while using if-then, although they contain only one statement each.  Omitting them can make the code more brittle. If a second statement is later added to the "then" clause, a common mistake would be forgetting to add the newly required braces. The compiler cannot catch this sort of error; you'll just get the wrong results. One example here

if(choice.equalsIgnoreCase("y"))
    speciesDog = true;
else
    speciesDog = false;

[Campbell Ritchie]

Agree about braces but not about the field speciesDog. You are going to create dreadful spaghetti code if you try all those fields. What about cats, and calculating the dose of morphine? You will have to think again about different dosages for different species.

[Campbell Ritchie]

Actually you should not write things like

if (something())
{
   somethingElse(true);
}
else
{
   somethingElse(false);
}

You simply write,
somethingElse(something());
or, in this case,
speciesDog = input.equalsIgnoreCase("y");

[Ganish Patil]

Campbell Ritchie wrote:Agree about braces but not about the field speciesDog. You are going to create dreadful spaghetti code if you try all those fields. What about cats, and calculating the dose of morphine? You will have to think again about different dosages for different species.

Yes I think OP didn't consider other animals. It would be easy if we create a separate class named Pet which will have these fields String petName, int petAge, float petWeight and String petType.

[Campbell Ritchie]

Ganish Patil wrote:. . . It would be easy if we create a separate class named Pet which will have these fields String petName, int petAge, float petWeight and String petType.

Maybe. But surely you are better off having an abstract Pet class and subclasses Dog, Cat, etc etc.

[Ganish Patil]

Campbell Ritchie wrote:Maybe. But surely you are better off having an abstract Pet class and subclasses Dog, Cat, etc etc.

Ohh I see, you mean declare common fields of Pets in Pet abstract class. Common behaviours of Pets comes in Pet abstract class like (Just assumed) sleeping(), drinkingWater() as defined methods of Pet abstract class whose implementation we know and methods (behaviour) which are type specific like makeNoise() where Dog barks and Cat meows so their implementation will be in respective classes i.e. type specific. Means when we know partial implementation then go for abstract class and when have no idea of implementation then go for interface, hope I'm correct?

[Campbell Ritchie]

We have an FAQ about the differences between abstract classes and interfaces. I hope that will help.

[Ganish Patil]

Campbell Ritchie wrote:I hope that will help

Yes helped a lot, Wiki is really a great source of information, I better peruse them. Thank you  

[Campbell Ritchie]

I was thinking you can overload the methods to calculate the dosages by type of Pet, but I now think I was mistaken and that may simply make the code really complicated.