Program output wrong

Question

Hi,
class MyTokenizer has one constructor, MyTokenizer(String s), and one method, tokenAt(int i) that returns the i-th token of the string passed to the constructor. It follows the usual Java convention of counting from zero, and uses the default token delimiters as defined in the StringTokenizer class. The method returns null if the argument of tokenAt is less than zero or greater than the number of tokens in the string. For example, if one constructs an object
MyTokenizer hamlet = new MyTokenizer(�To be or not to be�)
then
hamlet.tokenAt(0) returns �To�,
hamlet.tokenAt(3) returns �not�,
hamlet.tokenAt(17) returns null, etc.
There is another class, MyTokenizerTest, that uses JOptionPane to get a string from the user. Then, in a loop, I use another JOptionPane to get integers from the user and pass them as arguments to tokenAt, displaying the return value in the DOS window. I Exit from the loop when the user presses �Cancel� on the JOptionPane
Just try running this program and you will see the wrong output.
Please tell help me with this.
<---------------------------------------------------------------------->
import java.util.StringTokenizer;
import java.lang.Character;
public class MyTokenizer
     private String str;
     Constructs an object of MyTokenizer
     @param s The string
     public MyTokenizer(String s)
          str = s;
     Finds a token of the string at any position
     @param i the ith token of the string
     @retrun token the token at the ith position
     public String tokenAt(int i)
          String str1;
          str1 = "";
          int words;
          words = 0;
          int k;
          k = 0;
          for (int j = 0; j < str.length(); j++)
               if (Character.isWhitespace(str.charAt(j)))
                    words++;
               if (words == i + 1)
                    k = j;
                    break;
          for (int c = k + 1; c < str.length(); c++)
               str1 = str1 + str.charAt(c);
               if (Character.isWhitespace(str.charAt(c)))
                    break;
          return str1;
<---------------------------------------------------------------------->
import javax.swing.JOptionPane;
import java.util.StringTokenizer;
import java.lang.String;
This class tests the MyTokenizer class
public class MyTokenizerTest
     Tests the methods of MyTokenizer class
     @param args not used
     public static void main(String[] args)
          String s = JOptionPane.showInputDialog("Enter a String");
          System.out.println(s);
          MyTokenizer myToken = new MyTokenizer(s);
          boolean done = false;
          while (!done)
               String strInt = JOptionPane.showInputDialog("Enter the position of a word");
               if (strInt == null)
                    done = true;
               else
               int i = Integer.parseInt(strInt);
               System.out.println(myToken.tokenAt(i));

Answer 1

Firstly since your Tokenizer is 0-based, why should you add one at the following points? Try removing the + 1
if (words == i + 1)
for (int c = k + 1; c < str.length(); c++)
Secondly you check for whitespace before you check whether the word has been reached. This results in a miss for the 1st word. Switch the two if blocks in "for (int j = 0; j < str.length(); j++)" and try again.
HTH.