Will someone please tell me what I'm doing wrong with this?

[Tolls]

Should it be

 public boolean equals(t1)

?

Your original equals() method signature was correct, before you edited your original post.
It was just the code inside the method that was wrong.

[Toll]

Actually, let me throw you a method I tend to use in my classes:

public void debug(String s)
{
  System.out.println(s);
}

It's a simple enough method; it's mostly as a way to show how method declarations should look. You need to specify the type of variable that is sent to the method; in this case, it will accept a String.

[psx2514]

Have a look at the Java Method tutorial.

nevermind. you're not really helping me. Like I said. I tried everything I could, and came here as a last resort. That would be looking at resources outside of this forum. So, directing me towards resources outside of this forum is a moot point since I already did so. So, please, help me directly with my problem, or don't help me at all.

[psx2514]

Also, I'm not supposed to edit the TemperatureTest class. Only the Temperature class. Please read my OP if you haven't done so.

[psx2514]

Will someone just please tell me how to fix this? I don't have time to figure this out anymore. I give up. I've spent too much time on this.

[Toll]

As Tolls said, your original equals was correct in its definition, it just had the wrong code in it. Also, you should be able to make use of some of your methods in the Temperature class to avoid having to convert the temperatures in your equals method.

[psx2514]

As Tolls said, your original equals was correct in its definition, it just had the wrong code in it. Also, you should be able to make use of some of your methods in the Temperature class to avoid having to convert the temperatures in your equals method.

Please be more specific. WHAT PART of the code is wrong? Which lines?

[Toll]

Let's go over it line by line. Please answer each question.

  public boolean equals() {
    if (type == 'F') { // I assume you're planning to add another case for Celsius once this one works?
      double TempC = (degrees - 32) * 5/9; // Okay, so you have the degree in Celsius in TempC.
      double result = Math.round(TempC*10)/10.0; // Why do you have the division in there?
      return (this.degrees == result); // In which cases would the degrees and result actually be the same?
    }
    else {
      return false; // This means that anytime this Temperature object is in Celsius, it returns false. I'm assuming this will change once Fahrenheit works.
    }
  }

Also... Where does it compare two different Temperature objects in there?

[psx2514]

Let's go over it line by line. Please answer each question.

  public boolean equals() {
    if (type == 'F') { // I assume you're planning to add another case for Celsius once this one works?
      double TempC = (degrees - 32) * 5/9; // Okay, so you have the degree in Celsius in TempC.
      double result = Math.round(TempC*10)/10.0; // Why do you have the division in there?
      return (this.degrees == result); // In which cases would the degrees and result actually be the same?
    }
    else {
      return false; // This means that anytime this Temperature object is in Celsius, it returns false. I'm assuming this will change once Fahrenheit works.
    }
  }

Also... Where does it compare two different Temperature objects in there?

I need to prove that 212 degrees F is equal to 100 degrees C.

Look, this isn't going anywhere. Just tell me the answer. I don't care anymore. I'll learn from this for the next time I need to do this. This is just making me more frustrated. So, I don't care if I can't figure it out myself. Please just give me the solution, and I'll figure out what I did wrong based on that. This is due very soon, and I won't be able to figure it out by then. Thanks.

[Toll]

That doesn't answer a single question I asked though...

[psx2514]

That doesn't answer a single question I asked though...

I don't have time to play 20 questions. This is due very shortly.

I need one comparison method named equals():

example: returns true if temperature1 is equivalent to temperature2


Note:

This means 32F should be equal to 0C.You should convert all temperatures to either Celsius or Fahrenheit (using the get methods) so you are comparing similar values. Method returns a boolean.

[Toll]

I'm not trying to play 20 questions with you. I'm trying to guide you towards solving this yourself. The one thing I won't do is to just hand you the code you need though; you'll need to code it yourself.

The note did say you should use the get methods though, yet I don't see those actually being used anywhere in the code. Try using those in your equals method.

[psx2514]

I'm not trying to play 20 questions with you. I'm trying to guide you towards solving this yourself. The one thing I won't do is to just hand you the code you need though; you'll need to code it yourself.

The note did say you should use the get methods though, yet I don't see those actually being used anywhere in the code. Try using those in your equals method.

See lines 57-64 and 67-76

[Tolls]

This means 32F should be equal to 0C.You should convert all temperatures to either Celsius or Fahrenheit (using the get methods) so you are comparing similar values. Method returns a boolean.

There's your answer right there.
Pick one.
Either compare the F value (using getF for both Temperature objects) or compare the C value (using the getC method for both).
No need to even check the 'type' variable.
It's a straight 'return x == y'.

Of course you need to revert your signature to the original.

And we do not provide fully coded answers here.

So try and write the above.
Here's the skeleton again:

public boolean equals(Temperature t1) {
 // your code goes in here
}

[psx2514]

Okay. I got it! I finished (except for test case 2, which is extra credit). Is there anything I can do to clean my code up a little, or is it fine the way it is.

Also, any tips on how to do the extra credit, which is to "Implement a readInput() method to prompt user for degrees and type and then reads the values. The Scanner class does not have a specific method for reading chars. No input checking is done for units. It is assumed that user inputs only a 'C' or 'F' for units Lower case 'c' and 'f' are acceptable and do not need to be changed to upper case

Hint:

Read the input as a string and pick out first element from the string (which is the character you want)."

[SJF]

Toll is trying to help you figure out your problem. I fail to see how that is wasting anyone's time but his/(her?) own.

Your equals is broken. equals should always be in the form Object.equals(Object obj) (Or in this case Temperature.equals(Temperature temp) ) Object (Java Platform SE 7 )

AND t1.equals(t1) must always be true //REFLEXIVE
AND if t1.equals(t2) then t2.equals(t1) must also be true. //SYMMETRIC
AND if t1.equals(t3) and t3.equals(t2) then t1.equals(t2) must be true //TRANSITIVE

If you don't have a Temperature.equals(Temperature t) defined, the Java Compiler will use Object.equals(Object obj) for the comparison. This will end poorly as they are not the same Object.

[Toll]

A bit late on that reply, I think :P And I'm horribly rusty with Scanner, so all I can give there is Scanner (Java Platform SE 6)

[Junky]

Anyone else think OP is acting like a butthole?

[DrummondAW]

Anyone else think OP is acting like a butthole?

First time I've ever seen someone declare that someone trying to help them was wasting their time.