variable always stuck at 1

Question

I have this assignment where I need to populate a 2d array using info from a text file. I am trying to make every thing even using the primekey variable to replace my poscon that I am using temporarily. What is stopping me from doing that is that primekey wont increment right because of another

(MAIN PROBLEM) variable that is stuck, which is the emailcheck variable. I'm pretty sure it has to do with my for loop syntax but I can't quite figure it out.

       ...

try{
    Scanner check = new Scanner(file);
    Scanner checkNext = new Scanner(file);       

    System.out.println("Success File load");
    String data=check.next();
    System.out.println("data.next() works");

    int emailcheck=0;
    int primekey=0;

    while(check.hasNext()){
        posCon++;
        //check for @ symbol
        for(int i=0;i<data.length();i++){
            if(data.charAt(i)=='@'){
                emailcheck=emailcheck+1;
            }
        }

        //populates position array
        if(data.equalsIgnoreCase("staff")||
                data.equalsIgnoreCase("freshman")||
                data.equalsIgnoreCase("sohmore")||
                data.equalsIgnoreCase("junior")||
                data.equalsIgnoreCase("senior")||
                data.equalsIgnoreCase("adjunct")||
                data.equalsIgnoreCase("professor"))
        {
            db[0][posCon]=data;
            sort=1;
            data=check.next();

        }
        //id
        else if(sort==1){
            db[1][posCon]=data;
            sort=2;
            data=check.next();
        }
        //firstname
        else if(sort==2){
            db[2][posCon]=data;
            sort=3;
            data=check.next();
        }
        //lastname
        else if(sort==3){
            db[3][posCon]=data;
            sort=4;
            data=check.next();
        }
        //department
        else if(sort==4){
            db[4][posCon]=data;
            sort=5;
            data=check.next();
        }
        //email
        else if(sort==5 && emailcheck==1){
            db[5][posCon]=data;
            sort=6;
            emailcheck=0;
        }
        else if(sort==5 && emailcheck==0){
            db[5][posCon]="not here";
            sort=6;
        }
        //room
        else if(sort==6){
            db[6][posCon]=data;
            sort=0;
            data=check.next();
            emailcheck=0;
            primekey=primekey+1;
            System.out.println(primekey);
        }
        else{
            sort=0;
            data=check.next();
            emailcheck=0;
        }

    }
}catch(FileNotFoundException e) {
    e.printStackTrace();
}
}//End Constructor

here is the data from the text file

Staff 77778 Julie Chang Registrar
Adjunct 19778 Mike Thompson CS [email protected] GITC2400
Staff 30041 Anne Mathews Security
Junior 98444 Serene Murray Math [email protected]
Freshman 98772 Bob Mathew CS [email protected]
Professor 19010 Joan Berry Math [email protected] GITC2315C
Professor 19871 Aparna Khat CS [email protected] GITC1400
Adjunct 18821 Hari Mentor Physics [email protected] CK231
Staff 20112 Jim George Plant
Junior 68339 Tom Harry CS [email protected]
Senior 78883 Vince Charles IT [email protected]
Freshman 87777 Susan Han EE [email protected]
Senior 88888 Janki Khat IE [email protected]
Staff 5555 Aparna Sen Plant
Senior 66663 Jill Kaley it [email protected]
Staff 77777 Joe Batra plumbing
Staff 33333 Jim Natale Plumbing

I advise you to indent the code properly. The problem might get clearer then. — aioobe, Dec 04 '14 at 08:51
Your `primekey` is only incremented in case `sort == 6` but where do you set this var? — Eypros, Dec 04 '14 at 08:59
What should emailcheck contain? When should it be 0 and when 1? — Eran, Dec 04 '14 at 08:59
sorry I wasn't clear. my overall goal is to have the array populate while keeping it organized. The only overall issue I am having while ignoring the primekey issue is the emailcheck variable. Emailcheck is supposed to equal 1 or 0 depending if the @ character is located in the word or not. the problem is that it is always set to 1. — user2872194, Dec 04 '14 at 09:04

score 1 · Accepted Answer · answered Dec 04 '14 at 09:03

1

You have the data = check.next() call in the wrong place it should just after the start of the loop with the check.hasNext() test in it, and not elsewhere. You only need one Scanner instance the second one you declare is redundant.

answered Dec 04 '14 at 09:03

Matthew V Carey

196
1
10

variable always stuck at 1

1 Answers1