Thread: Array and counter in for loop

I need to be able to use the counter and the array in the same loop. The way it is now I get a java.util.NoSuchElementException (no line found). How can i change my code so that it is no longer trying to go past the end of the file.

import java.util.Scanner;
import java.io.FileNotFoundException;
import java.lang.String;
import java.io.File;
import java.io.FileInputStream;
 
public class Assignment1{
 
    public static void main(String [] args){    
        //Reads file containing fractions
        Scanner inputFile = null;
        try {
            inputFile = new Scanner(new FileInputStream("fractions.txt"));
        }
        catch (FileNotFoundException e) {
            System.out.println("File not found or not opened.");
            System.exit(0);
        }
 
        //variables
        String[]fractions = new String[100];  //will take in the fractions 
        String[]split = new String[2];        //used to split the fractions
        int [] numerator = new int [100];     // store numerators
        int [] denominator = new int [100];   //store denominators
        int count = 0;                        //number of lines in file
 
        while(inputFile.hasNextLine()){    
           for(int i=0; i<=count ; i++){
                fractions[i]=inputFile.nextLine();
                count++;
                System.out.println(fractions[i]);
           }
        }

Thank you very much. That fixed it. Don't suppose you can solve my other problem of why I am getting a java.lang.NullPointerException? I tried making my String[]split just a string, but it didn't like that.

I added on:

for(int i=0; i<=count; i++){
            split = fractions[i].split("/");
            numerator[i]=Integer.valueOf(split[0]);
            denominator[i]=Integer.valueOf(split[1]);  
}

The editor highlights

split = fractions[i].split("/");

I also tried

split[i] = fractions [i].split("/");

but when I add the [i], it tells me that it is an incompatible type because it is an array rather than a string.

Yes. The fractions array has 100 slots. It has that many because I won't know the size of the file that is used to test my code. Is there another way to do the array without setting it at [100]?

So I have managed to get most of my code done (i hope). I am only having a couple issues. The first is that I am unsure how to distinguish when a fraction has already been counted. For example with a file that looks like:
6/3
7/2
6/3

The output is:
6/3 has 2
7/2 has 1
6/3 has 2

Would there be anyway to make the loop(s) skip fractions that have already been counted? Or would I have to created a loop for the system.out.println statements that did that? The code bellow is just the loops to count.

 
        int num;
        int den;
        //compare denominator array
        for(int i = 0; i<=count; i++){
            den = denominator[i];
            num = numerator[i];
            for(int a = 1; a<count; a++){
                if(den == denominator[a]){ 
                   if(num == numerator[a]){
                        dencount++;
                   }     
                }
            }
            System.out.println(dencount);
            dencount=0; 
        }
    }