java basic array question

Question

List[K] = new Station(Location, Temp, Name);

In one method I enter in the values for Location and Name. In a later method I would like to loop through each Location and assign a unique Temp.

       public void Post() 
{
     double Temp = 0;
     int K;
     for(K = 0; K < Count ; K++)
     System.out.print(" " + List[K].GetLocation() + ": ");
     System.out.println("Enter Temperature");
     Temp = Input.nextDouble();

}

This simply dumps out all the locations and after that it prompts for a temperature and accepts it. But the temperature is not even assigned to the last value in the array.

It hurts to look at the syntax and naming convention. oracle.com/technetwork/java/codeconv-138413.html — Yuriy Zubarev
– Yuriy Zubarev, Commented Feb 28, 2011 at 5:31

ide · Accepted Answer · 2011-02-28 05:29:18Z

1

Put curly braces around your for-loop body.

public void Post() 
{
     double Temp = 0;
     int K;
     for(K = 0; K < Count ; K++) {
         System.out.print(" " + List[K].GetLocation() + ": ");
         System.out.println("Enter Temperature");
         Temp = Input.nextDouble();
     }
}

BTW, you should read the standard Java conventions.

answered Feb 28, 2011 at 5:29

ide

20.9k6 gold badges72 silver badges113 bronze badges

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

1 Comment

chief Over a year ago

You are correct the brace fixed it. I also had to add this to assign the new value into the array. List[K].SetTemp(Temp);

It Grunt · Accepted Answer · 2011-02-28 05:29:44Z

1

Is this what you would like to do? Print out each location in the list and request the temperate from the user?

   public void Post() 
{
 double Temp = 0;
 int K;
 for(K = 0; K < Count ; K++)
 {  
  System.out.print(" " + List[K].GetLocation() + ": ");
  System.out.println("Enter Temperature");
  Temp = Input.nextDouble();
 }
}

answered Feb 28, 2011 at 5:29

It Grunt

3,3983 gold badges23 silver badges35 bronze badges

Comments

user unknown · Accepted Answer · 2011-02-28 05:47:42Z

Why is Temp initialized in the first line? Why is there an assignement in the last line? It has no effect.

Why do you declare K outside of the loop? The Codeconventions asks for lowercase letters for attributes, parameters, variables and methods.

  list[k] = new Station (location, temp, name);
// 

public void post () 
{
     for (int k = 0; k < count; k++)
         System.out.print (" " + list[k].getLocation () + ": ");
     System.out.println ("Enter Temperature");
     double Temp = Input.nextDouble ();
}

In one method I enter in the values for Location and Name. In a later method I would like to loop through each Location and assign a unique Temp.

Well - you loop through stations, not locations, do you?

But the temperature is not even assigned to the last value in the array.

Which Array? The Array called List? You never assign temp to something, do you?

Do you have a setter in Station? Is it a writable attribute?

list[count-1].setTemp (temp); // or 
list[count-1].temp = temp;

might help.

Collectives™ on Stack Overflow

java basic array question

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related