What am I doing wrong?

[javajen]

What am I doing wrong? the txt file is attached........

Write a main method that will read the file secret.txt, separate it into word
tokens. You should process the tokens by taking the first letter of every fifth word, starting with the first word in the file. These letters should converted to capitals, then be appended to a StringBuilder object to form a word which will be printed to the console to display the secret message.



import java.util.StringTokenizer; //to use the string tokenizer class
import java.io.*;
import java.util.Scanner;

public class SecretJava {

public static void main (String[] args) throws IOException
{
File file = new File("C:\\java\\bin\\secret.txt");
Scanner secretFile = new Scanner(file);

String line;

line = secretFile.nextLine();

StringBuilder code = new StringBuilder();

String[] tokens = line.split(" ");

char[] codeArray = new char[tokens.length];

for(int i = 0; i < tokens.length; i = i + 4)
{
codeArray[i] = tokens[i].charAt(0);
code.append(Character.toUpperCase(codeArray[i]));
}

System.out.println(code);
}

}

[Junky]

What am I doing wrong?

since we don't read minds you need to tell us. Do you get compiler errors? Incorrect output/behaviour. Provide details.

[Dark]

The more details you provide about whats going wrong with your code the more help we can give. If you get an error post it here, if it compiles and run then show us what you get and what you're supposed to get. That way we can analyze what might be going wrong.

The more information we have the easier it is to narrow down the snippet of code thats causing a problem.

[Iron Lion]

for(int i = 0; i < tokens.length; i = i + 4)
{
codeArray[i] = tokens[i].charAt(0);
code.append(Character.toUpperCase(codeArray[i]));
}

This will look at the elements of codeArray in the order 0, 4, 8, 12 [...]. You want to be looking at 0, 5, 10, 15 [...]. I can't guarantee that that's the only thing wrong, but it's the only thing I noticed straight away - and, as the others are saying, it helps if you're more specific with your problems.