also need help this code

[bob michelle]

import java.util.*;
public class Animal{
 char name;                //DECLARE VARIABLES
 int byear;
 char species;
 double bal;
 char oname;

public Animal(char n,int by,char s,double b,char o){ //CONSTRUCTOR
 name=n;
 byear=by;
 species=s;
 bal=b;
 oname=o;
}

public char getPetName(){    //PRINTS PET NAME
 return name;
}

public int getBirthYear(){   //PRINTS BIRTH YEAR
 return byear;
}

public char getSpecies(){   //PRINTS SPECIES
 return species;
}

public double getBalance(){ //PRINTS BALANCE
 return bal;
}

public char getOwnersName(){  //PRINTS OWNERS NAME
 return oname;
}

public char setName(char name){
}
//java.io.File file = new java.io.File("cps2231Lab5.data");

//Scanner scan = new Scanner(file);
//int x = scan.nextInt();
//Animal [] pets = new Animal[x];

public void inputAnimals(){   //INPUTS DATA INTO ARRAY
// for(int i=0; i<pets.length; i++){
// Animal pet = new Animal(scan.next(),scan.nextInt(),scan.next(),scan.nextDouble(),scan.next());
// pet.Animal(scan.next(),scan.nextInt(),scan.next(),scan.nextDouble(),scan.next());

// pets[i] = pet;
//}      //END FOR STATEMENT
}      //END METHOD



//public Animal getOutPut(){  //PRINTS THE ARRAY AS IS
// for(int i=0;i<pets.length;i++){
//System.out.print(pets[i].getPetName());
 // pets[i].getBirthYear();
 // pets[i].getSpecies();

 // pets[i].getBalance();
 // pets[i].getOwnersName();
// }   //ENDS FOR STATEMENT
//}  //END METHOD

public static arraySort(){
 double temp;
 double high = 0.0;
// Animal temparray;
 for (int i=0;i<this.length;i++){
   temp = this[i].getBalance();
  // temparray=this[i];
   if(temp>this[i+1].getBalance()){
    this[i]=this[i+1].getBalance();
    this[i+1]=temp;
   }
   //if(temparray.getBalance() == high)
//System.out.print(temparray.getOutPut());
}   //ENDS FOR STATEMENT
}  //ENDS METHOD


public static printAnimals(){
 for (int i =0; i<this.length; i++){
  this[i].getPetName();
  this[i].getBirthYear();
  this[i].getSpecies();
  this[i].getBalance();
  this[i].getOwnersName();
 }
}
}  //ENDS CLASS

main class

import java.util.*;

public class AnimalTest{
public static void main(String [] args){

java.io.File file = new java.io.File(cps2231Lab5.data);
Scanner scan = new Scanner(file);
int x = nextInt();
Animal[] pets = new Animal[x];


 for(int i=0; i<pets.length; i++){
 Animal pet = new Animal;
 pet.Animal(scan.next(),scan.nextInt(),scan.next(),scan.nextDouble(),scan.next());

 pets[i] = pet;
}

pets.arraySort();
pets.printAnimals();
}
}

[Matthew Brown]

Hi Bob. Welcome to the Ranch! You need to actually ask a question if you want us to help you effectively, though. What specifically do you need help with?

[fred rosenberger]

at the very least, it looks like your inputAnimals method isn't closed...I think someone got a little trigger happy when commenting out stuff. The brace on line 35 needs to be uncommented (I think).

[Campbell Ritchie]

I think you are dealing with too much code all at once. You should write, maybe, five lines only before compiling and running whatever code you have. That way you find errors much more quickly.
You should also, I think, write write some of your } backwards. As you will see from the link, “backwards” means you write lines in a different order from what you usually do.

[fred rosenberger]

Campbell Ritchie wrote:You should write, maybe, five lines only before compiling and running whatever code you have.

FIVE? IMHO, that is still way too many. I never go more than 2-3.

[Campbell Ritchie]

public class Foo
{
   public Foo()
   {
   }
}

That’s six lines, and still does nothing much.

You have four lines each containing only { or }, so you ain’t going to get very far with three lines

But Fred and I agree that you must write little bits of code and make sure they work before going on to the next bit.

[fred rosenberger]

hmmm....maybe I don't really count "lines" of code. i would consider this to be one:

public class Foo
{
}

since technically, it can be written on one line:

public class Foo {}

and yes, so could your original six lines:

public class Foo { public foo() {} }

but that's just silly.

I guess I don't count braces as 'lines of code' since they can be written in so many different ways.

[Campbell Ritchie]

I think your line arithmetic and mine will result in our both writing the same amount of code at a time