trapping two appearance of a character

[joseph okon]

class Contain {
 
    String word;
    String ocurrChar = "@";
    int mark = 2;
   public Contain(String word) {
 
if(countChar(word).equals(mark)) {
  System.out.println("Contain two @ ");
        }
  }
 
  public boolean countChar(String word) {
   
   
   for (int k = 0; k< word.length(); k++){
     return (word.indexOf(ocurrChar) >= 3);
}
  return countChar(word);
    
  }
 
   public static void main(String args[]) {
    Contain con=new Contain(args[0]);
   }
}

pls don't laught ! can't think of any stuff to make it work

[fbr - added code tags]
[ December 11, 2007: Message edited by: Fred Rosenberger ]

[Joanne Neal]

Originally posted by joseph okon:

class Contain {
 
    String word;
    String ocurrChar = "@";
    int mark = 2;
   public Contain(String word) {
 
if(countChar(word).equals(mark)) {
  System.out.println("Contain two @ ");
        }
  }
 
  public boolean countChar(String word) {
   
   
   for (int k = 0; k< word.length(); k++){
     return (word.indexOf(ocurrChar) >= 3);
}
  return countChar(word);
    
  }
 
   public static void main(String args[]) {
    Contain con=new Contain(args[0]);
   }
}

pls don't laught ! can't think of any stuff to make it work

[fbr - added code tags]

[ December 11, 2007: Message edited by: Fred Rosenberger ]

One error I can see is that you are trying to use equals to compare two integers. Primitive data types (such as int) don't have methods. You need to use == to compare them.

Fix that and if it still doesn't compile, post the error messages you are getting (exactly as they appear). Once it's compiling we can then tackle the logic.

[Ulf Dittmer]

I've made a few modifications -marked in bold- to help you get going. What's missing is the spot labelled "TODO". What needs to happen there to make the code work?

class Contain {
 
    String word;
    [B]char[/B] ocurrChar = '@';
    int mark = 2;
 
   public Contain(String word) { 
      if ([B]countChar(word) >= mark[/B]) {
  System.out.println("Contain two @ ");
      }
   }
 
   public [B]int[/B] countChar(String word) {
     [B]int count = 0;[/B]
   
      for (int k = 0; k< word.length(); k++) {
         // TODO
      }
 
      return [B]count[/B]; 
   }
 
   public static void main(String args[]) {
      Contain con=new Contain(args[0]);
   }
}

[Shane W]

Hi
As pointed out by folks above, there are some erroes in your code.Find below a simple piece of code that achieves what your code would have(more than 2 @).I have done away with the countChar method.
Incase you want to count how many times "@" appears,you need to modify countChar method

class Contain {
    
    String ocurrChar = "@";
    
   public Contain(String word) {
          if(word.indexOf(ocurrChar)!=-1
             &&word.indexOf(ocurrChar)!=word.lastIndexOf(ocurrChar))
                 System.out.println("Contains more than two @ ");
          }
 
    public static void main(String args[]) {
    Contain con=new Contain(args[0]);
   }
}

Shane

[joseph okon]

thanks guys, i have gotten the right answer to my code