REPOST code formatting error::String Search Algorithms

Question

Hi im trying to get the program below to work, the method "naive" is redundant but works, its just there to show you what im trying to do. isSpam should be passed a message, and it then needs to use the patterns located in spamPatterns (just a text file read in and separated into separate lines to an array of strings) and try and find them in the message.
What im trying to do is implement a method that looks for the last character of the pattern, then if found skips back to the first character and then continues left to right to progress through the word. If at any point it finds a letter NOT in the pattern, it skips forward the length of the pattern, from the point it found the end letter. I hope this makes sense?
I cant get it to work, but I can't figure out why, I have some idea it might be to do with misuse of -1's here and there. Please help!
p.s. please dont refer me to other algorithms boyermoore etc, i want mine to work!
public class SpamFilter {
     public String[] spamPatterns;
    // construct a spam filter with the given spam patterns
    public SpamFilter(String[] patterns) {
         //Creates spamPatterns array
         spamPatterns = new String[patterns.length];
         //Fills spamPatterns array with patternfile content
         for (int i=0; i < patterns.length; i++) {
              spamPatterns[i] = patterns;
// using java default "indexof" method
public boolean naive(String message) {
     int q;
     boolean spam = false;
     /* Progressively goes through each array entry in spamPatterns
     * and uses message.indexOf(this.spamPatterns[i]) to check
     * for occurences of the spamPatterns. If this results in a
     * not null output (e.g.-1) spam is set to true */
     for (int i=0; i < this.spamPatterns.length; i++) {
          q = message.indexOf(this.spamPatterns[i]);
          if (q!=-1) {
               spam=true;
return spam;
//trying to be clever (the idea is it skips characters it knows it doesnt need to look at)
public boolean isSpam(String message) {
     boolean spam=false;
     int l = this.spamPatterns.length;
     int skip=0;
     int messagecount=0;
     int patterncount=0;
     int reset=0;
     int flag=0;
     int check=0;
     int endcheck=0;
     char endchar, messagechar, patternchar ;
     for(int i=0; i<l; i++) {
          endchar = spamPatterns[i].charAt(spamPatterns[i].length()-1);
          skip = spamPatterns[i].length();
          messagechar = message.charAt(skip-1);
          patterncount=spamPatterns[i].length()-1;
          patternchar=endchar;
          flag=0;
          //System.out.println("PATTERNCOUNT: "+patterncount);
          //System.out.println("PATTERNCHAR: "+patternchar);
          for(messagecount=skip-1; messagecount<message.length(); messagecount++) {
               //if(flag>1){System.out.println("FLAG: "+flag+" PATTERNCOUNT: "+patterncount+" PATTERNCHAR: "+patternchar+" MESSAGECOUNT:"+messagecount+" MESSAGECHAR:"+messagechar);}
               if (patternchar==messagechar) {
                    flag+=1;
                    if (flag==1) {
                         messagecount-=patterncount;
                         messagechar=message.charAt(messagecount);
                         patterncount-=(skip-2);
                         patternchar=spamPatterns[i].charAt(patterncount);
                    else {
                         if (flag<spamPatterns[i].length()) { 
                              messagechar=message.charAt(messagecount);
                              patterncount+=1;
                              patternchar=spamPatterns[i].charAt(patterncount);
                         else {
                              spam=true;
                              flag=0;
                              patternchar=endchar;
               else {
                    if (flag==0) {
                         reset=messagecount+skip;
                    else if (flag>0) {
                         reset=messagecount+(skip-(flag-1))+skip;
                    messagecount=reset;
                    patternchar=endchar;
                    patterncount=(skip-1);
                    if (messagecount<message.length()) {
                         messagechar=message.charAt(messagecount);
                         flag=0;
                    else {
                         messagecount=message.length();
     System.out.println(spam);
     return spam;

Answer 1

Jornica,
This is a known issue and we are working on a new revamped formatter.
Thanks
Barry