IF statement issue

**AndreaRenee** · 03-10-2012, 11:17 PM

This is probably just a stupid error on my part but I can't figure out where I've gone wrong. When I run this program to test it, it refuses to function beyond the IF statement & and I'm thinking there's something wrong there but I'm not sure.

Although the pictures I use for this are the same size (480x360) and two pictures of the same size is what is required for this to work properly and my code is the exact same as when I did just a morph instead of a movie morph, it will not run properly. Help would be greatly appreciated.

Code was made in DrJava IDE. Please ignore the comments (they are part of the requirement for my assignment)

Java Code:

public class MorphingMovie
{
  public static void main(String[] args)
  {
    //Use Swan.jpg from mediasources
    String pic1 = FileChooser.pickAFile();
    Picture startPicture = new Picture();
    int startWidth = startPicture.getWidth();
    int startHeight = startPicture.getHeight();
    
    //Use twoSwans.jpg from media sources
    String pic2 = FileChooser.pickAFile();
    Picture endPicture = new Picture(pic2);
    int endWidth = endPicture.getWidth();
    int endHeight = endPicture.getHeight();
    
    if((startWidth != endWidth) || (startHeight != endHeight))
    {
      SimpleOutput.showInformation("Sorry. The files are not the same size.");
    }
    else
    {
       int rate = SimpleInput.getIntNumber("Enter Integer Number: ");
    
    //Directory name
      String directoryName = ("C:/Movie1/");
    
    Picture[] pictureArray;
    pictureArray = new Picture[rate + 2];
    pictureArray[0] = startPicture;
    pictureArray[pictureArray.length - 1] = endPicture;
    
    //Morphing pictures
      for (int i = 1; i < pictureArray.length; i++)
    {
      Picture intermediate = new Picture(startWidth, startHeight);
      intermediate.morphStage(startPicture, endPicture, rate, i);
      pictureArray[i] = intermediate;
      }
    
    //Make a Frame Sequence
    FrameSequencer frameSequencer = new FrameSequencer(directoryName);
    for (int i = 0; i < pictureArray.length; i++)
        {
      frameSequencer.addFrame(pictureArray[i]);
        }
    
    //Make Movie 1 (movie of 20 frames)
        MoviePlayer movie1 = new MoviePlayer(directoryName);
    movie1.playMovie(rate);
 
    }
  }
}

**Norm** · 03-10-2012, 11:41 PM

it refuses to function beyond the IF statement

Can you explain what the program does?
Try adding some println statements to the program to show where it is executing and where it is refusing to go.
Print out the values of the variables that control where the code executes.

**ojn** · 03-11-2012, 12:00 AM

PHP Code:

if((startWidth == endWidth) && (startHeight == endHeight))
    {
      SimpleOutput.showInformation("Sorry. The files are not the same size.");
    }

The "error" you provide says the files aren't of the same size, but isn't that exactly what you test for in the if statement?

**AndreaRenee** · 03-11-2012, 12:24 AM

Originally Posted by ojn

PHP Code:

if((startWidth == endWidth) && (startHeight == endHeight))
    {
      SimpleOutput.showInformation("Sorry. The files are not the same size.");
    }

The "error" you provide says the files aren't of the same size, but isn't that exactly what you test for in the if statement?

yes it is but when I run the program WITH two files of the same size, it still comes up saying the files aren't the same size.

the original code was

Java Code:

 if(startWidth != endWidth || startHeight != endHeight)
{
SimpleOutput.showInformation("Sorry. The files are not the same size.");
}

**AndreaRenee** · 03-11-2012, 12:28 AM

The program is supposed to take one picture and morph it into another picture and display that morph as a movie. Original format of the code provided.

Java Code:

 if (startWidth != endWidth || startHeight != endHeight)
{
//Pictures are same size so is not supposed to go to this
SimpleOutput.showInformation("Sorry. These files are not the same size.")
}
//Supposed to go here when the pictures are the same size & continue to do the morph
else....

**Norm** · 03-11-2012, 12:51 AM

it still comes up saying the files aren't the same size.

Have you tried printing the values that are being compared to see what they are?

**AndreaRenee** · 03-11-2012, 01:32 AM

Originally Posted by Norm

Have you tried printing the values that are being compared to see what they are?

Yes I have and I just double checked them to make sure my eyes didn't play any tricks on me. They are both 480(width) by 360(height)

**Norm** · 03-11-2012, 01:48 AM

If the values are the same where does the execution flow go?
Did you add printlns to show where he code is executing?
Can you post the debug output and the code that is generating the output.

Java Code:

if((startWidth == endWidth) && (startHeight == endHeight))

If they are both equal then above statement would be true.
And the following would be false

Java Code:

if (startWidth != endWidth || startHeight != endHeight)

**AndreaRenee** · 03-11-2012, 01:56 AM

When I compile the program, it gives no errors. It compiles perfectly fine.

the If statement is to stop the program from running if the pictures are not the same size. so when it says..

Java Code:

 if (startWidth != endWidth || startHeight != endHeight)

then the program is supposed to print "Sorry. The files are not the same size" with this code

Java Code:

 SimpleOutput.showInformation("Sorry. The files are not the same size.")

When the files ARE the same size, it is supposed to go to the else and continue on initiating the code I originally posted.

The problem is that when it gets to the if statement, it gives me the "Sorry. The files are not the same size." even though the files are the same size.

**Norm** · 03-11-2012, 02:08 AM

it gives me the "Sorry. The files are not the same size." even though the files are the same size.

Can you post the debug println output and the code that is generating the output.

**ozzyman** · 03-11-2012, 02:11 AM

You forgot to set startPicture to pic1?

Java Code:

    String pic1 = FileChooser.pickAFile();
    Picture startPicture = new Picture();

The way you did for the other picture:

Java Code:

    String pic2 = FileChooser.pickAFile();
    Picture endPicture = new Picture(pic2);

Perhaps that was your problem?

**Norm** · 03-11-2012, 02:13 AM

Then how are the two values for the start and end images the same?

**AndreaRenee** · 03-11-2012, 02:14 AM

wow.... just wow... that was the problem. Thank you very much!! That was such a dumb mistake!

also, thanks you as well Norm for your help as well. I'm sorry I am so code-blind!

**ozzyman** · 03-11-2012, 02:14 AM

Haha... don't worry we've all had those days.

**Norm** · 03-11-2012, 02:17 AM

They are both

This is very confusing: How could the values be the same when you printed them out????
Obviously you didn't print them out or you would have seen that they were different.