Passing value from object string to string

Question

 try {


    BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
    System.out.println("Enter the continent;");
    String CN = in.readLine();
    String MaxDate="1";
    for(Earthquakerecd e : eqList)
    {
        if( e.getContinent().equals("CN"))
        {
          MaxDate=e.getDate();
        }

        {
            System.out.println( e.toString()); 
        }


    }

     System.out.println( MaxDate); 
    }

    catch (IOException e)
    {
        System.out.println("IOException has been caught");
    }

This is a simple problem i think. In this problem Maxdate is declared as 1. CN is a string for continent . If the user input matches the continent , then the date should be passed from e.getDate() to Maxdate. In anycase , we should never get the output as 1 , it should be some date from object e. I am always getting 1 for Maxdate. Any possible solutions? is my syntax right?

have you tried MaxDate=e.getDate().toString()? I'm not savvy with Java, but worths a shot — user497849
– user497849, Commented Feb 7, 2012 at 23:05
Why is MaxDate a String? Make it same class e.getDate() returns... Also btw, in Java convention is to start variables with lowercase, and classes with uppercase. — m0skit0
– m0skit0, Commented Feb 7, 2012 at 23:06
Are you sure you're getting inside the if? Put a breakpoint there. — m0skit0
– m0skit0, Commented Feb 7, 2012 at 23:08
@da_scav_from_windy_city You should accept an answer here. Both answers are correct and dbyrne's is very good. — cajunluke
– cajunluke, Commented Feb 25, 2012 at 1:23

cajunluke · Accepted Answer · 2012-02-25 01:22:21Z

7

Seems like you want:

if(e.getContinent().equals(CN))

Right now you are comparing against the String literal "CN". You aren't using the variable CN for anything.

I'm assuming you meant to insert an else after your if statement?

Also, in Java it's common to not start your variable names with uppercase letters (name your string cn instead of CN).

edited Feb 25, 2012 at 1:22

cajunluke

3,11130 silver badges28 bronze badges

answered Feb 7, 2012 at 23:05

dbyrne

61.5k13 gold badges91 silver badges103 bronze badges

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

Comments

user1084509 · Accepted Answer · 2012-02-07 23:09:36Z

Try doing it this way:

try {


    BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
    System.out.println("Enter the continent;");
    String CN = in.readLine();
    String MaxDate="1";
    for(Earthquakerecd e : eqList)
    {
        if( e.getContinent().equals(CN)) //When you put it with "" the continent value was compared to CN and not from the user input
        {
          MaxDate=e.getDate();
        }

        {
            System.out.println( e.toString()); 
        }


    }

     System.out.println( MaxDate); 
    }

    catch (IOException e)
    {
        System.out.println("IOException has been caught");
    }

if that doesnt work, then try by doing the for on this way:

for (int i=0; i<eqList.size(); i++) {
     Earthquakerecd e = eqList.get(i);
     //... code ... //
}

Collectives™ on Stack Overflow

Passing value from object string to string

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related