Hjælp til en lille Servlet opgave

Det var godt nok et udvidet check mod spammere.

Som jeg laeser den skal du skrive en metode der kaldes med et konfig array og en URL,
metoden checker saa i config arrayets lige elementer (index 0, 2 etc.) og hvis
der er match returneres den tilsvarende ulige element (index 1, 2 etc.). Hvis ingen
match returneres en fast streng.

En simpel for loekke og lidt indeksering (husk kravet om at man skal bruge
det laengste match !) boer kunne loese den.

Hej Arne jeg har lavet følgende:

public class HandlerFactory {
   
    public String getHandler(String[] config, String requestUri) {
        String handle = "SXHoef9"; 
        for (int i = 0; i < config.length; i++) {
            if((i % 2) == 0) {
                if(config[i].trim().equals(requestUri.trim())) {
                return "??";
                }
            }
        }
        return handle;
      }

    public static void main(String[] args) {
        String[] config = { "/", "MainServlet", "/nav", "NavigationServlet" };
        String requestUri = "/nav/test";
        System.out.println(new HandlerFactory().getHandler(config, requestUri));
       
    }

}


Men hvis der er en match skal jeg returnere ??

config[i+1]

Ok hvad er det det lige du mener med det længste match ?

/nav/test matcher baade / og /nav, men fordi /nav er laengst er det den som gaelder

hvordan kan /nav/test matche /. Hvis det er tilfældet skal jeg så ikke lave en split på min streng /nav/test og splitte den op i ?? kan du ikke kigge lidt på min code. For nu er jeg lidt forvirret
Da jeg prøvede koden fik jeg følgende fejl: (Fra hjemmesiden )

Testcase 2 failed:
  config: { "/", "TestServlet", "/test", "TestServlet2" }
  request: "/servlet/TestServlet"
ERROR: A testcase failed, please revise your code

/nav/test starter med baade / og /nav

Også kom koden så til at se ud

public class HandlerFactory {
    private final static String REGEX = "[/\\\\]+";
    public String getHandler(String[] config, String requestUri) {
        String tmp = "";
        int index = -1;
        String[]split = new String[0];
        int counter = 0;
        String handle = "SXHoef9";

        for (int i = 0; i < requestUri.length(); i++) {
            String sub = requestUri.substring(i, i+1);
            if(sub.equals("/")) {
                counter++;
            }
        }

        split = requestUri.split(REGEX);
        for (int i = 1; i <= counter; i++) {
            if(!requestUri.endsWith("/")) {
                tmp = split[i];
                tmp = "/"+tmp;
            }
           
            if(requestUri.endsWith("/") && counter > 1) {
                tmp = split[i-1];
                tmp = "/"+tmp;
            }
            for (int j = 0; j < config.length; j++) {
                if((j % 2) == 0) {
                    if(requestUri.startsWith("/")) {
                        if(config[j].trim().equals("/")){
                            if(j > index) {
                                index = j;
                            }
                        }
                    }

                    if(config[j].trim().equals(requestUri.trim())) {
                        if(j > index) {
                            index = j;
                        }
                    }
                    if(config[j].trim().equals(tmp.trim())) {
                        if(j > index) {
                            index = j;
                        }
                    }
                }
            }   
        }
        if(index > -1 ) {
            return config[index+1];
        }
        else
            return handle;
    }
}

Arne smider du et svar :-)

Jeg tror at koden kan laes kortere.

Men hvad pokker.

Det er ihvertfald en lidt heftig test for at undgå spamere.

CATPCHA kan godt gå hjem og lægge sig !

Og svar.

Og samtidig over på Java... :-)

http://www.eksperten.dk/spm/800819#rid6898935