-----------------------------------

mariomania88

Tue Jan 13, 2004 10:27 am


String arrays in String return methods

-----------------------------------

I am using a string array in a return method, but I can't get it to work.
Assistance is needed.  Below is my code.    

It is for the purpose of checking if a postal code is typed in correctly.  I have a global string array called pcode 
private static String process (String checkPcode[] = new String [])
    {
        if (checkPcode [0].equalsIgnoreCase ("A") || checkPcode [0].equalsIgnoreCase ("B") || checkPcode [0].equalsIgnoreCase ("C") || checkPcode [0].equalsIgnoreCase ("D") || checkPcode [0].equalsIgnoreCase ("E") || checkPcode [0].equalsIgnoreCase ("F") || checkPcode [0].equalsIgnoreCase ("G") && checkPcode [0].equalsIgnoreCase ("H") || checkPcode [0].equalsIgnoreCase ("I") || checkPcode [0].equalsIgnoreCase ("J") || checkPcode [0].equalsIgnoreCase ("K") || checkPcode [0].equalsIgnoreCase ("L") || checkPcode [0].equalsIgnoreCase ("M") || checkPcode [0].equalsIgnoreCase ("N") || checkPcode [0].equalsIgnoreCase ("O") || checkPcode [0].equalsIgnoreCase ("P") || checkPcode [0].equalsIgnoreCase ("Q") || checkPcode [0].equalsIgnoreCase ("R") || checkPcode [0].equalsIgnoreCase ("S") || checkPcode [0].equalsIgnoreCase ("T") || checkPcode [0].equalsIgnoreCase ("U") || checkPcode [0].equalsIgnoreCase ("V") || checkPcode [0].equalsIgnoreCase ("W") || checkPcode [0].equalsIgnoreCase ("X") || checkPcode [0].equalsIgnoreCase ("Y") || checkPcode [0].equalsIgnoreCase ("Z") && (checkPcode [1].equals ("0") || checkPcode [1].equals ("1") || checkPcode [1].equals ("2") || checkPcode [1].equals ("3") || checkPcode [1].equals ("4") || checkPcode [1].equals ("5") || checkPcode [1].equals ("6") || checkPcode [1].equals ("7") || checkPcode [1].equals ("8") || checkPcode [1].equals ("9") && checkPcode [2].equalsIgnoreCase ("A") || checkPcode [2].equalsIgnoreCase ("B") || checkPcode [2].equalsIgnoreCase ("C") || checkPcode [2].equalsIgnoreCase ("D") || checkPcode [2].equalsIgnoreCase ("E") || checkPcode [2].equalsIgnoreCase ("F") || checkPcode [2].equalsIgnoreCase ("G") && checkPcode [2].equalsIgnoreCase ("H") || checkPcode [2].equalsIgnoreCase ("I") || checkPcode [2].equalsIgnoreCase ("J") || checkPcode [2].equalsIgnoreCase ("K") || checkPcode [2].equalsIgnoreCase ("L") || checkPcode [2].equalsIgnoreCase ("M") || checkPcode [2].equalsIgnoreCase ("N") || checkPcode [2].equalsIgnoreCase ("O") || checkPcode [2].equalsIgnoreCase ("P") || checkPcode [2].equalsIgnoreCase ("Q") || checkPcode [2].equalsIgnoreCase ("R") || checkPcode [2].equalsIgnoreCase ("S") || checkPcode [2].equalsIgnoreCase ("T") || checkPcode [2].equalsIgnoreCase ("U") || checkPcode [2].equalsIgnoreCase ("V") || checkPcode [2].equalsIgnoreCase ("W") || checkPcode [2].equalsIgnoreCase ("X") || checkPcode [2].equalsIgnoreCase ("Y") || checkPcode [2].equalsIgnoreCase ("Z") && checkPcode [3].equals ("0") || checkPcode [3].equals ("1") || checkPcode [3].equals ("2") || checkPcode [3].equals ("3") || checkPcode [3].equals ("4") || checkPcode [3].equals ("5") || checkPcode [3].equals ("6") || checkPcode [3].equals ("7") || checkPcode [3].equals ("8") || checkPcode [3].equals ("9") && checkPcode [4].equalsIgnoreCase ("A") || checkPcode [4].equalsIgnoreCase ("B") || checkPcode [4].equalsIgnoreCase ("C") || checkPcode [4].equalsIgnoreCase ("D") || checkPcode [4].equalsIgnoreCase ("E") || checkPcode [4].equalsIgnoreCase ("F") || checkPcode [4].equalsIgnoreCase ("G") || checkPcode [4].equalsIgnoreCase ("H") || checkPcode [4].equalsIgnoreCase ("I") || checkPcode [4].equalsIgnoreCase ("J") || checkPcode [4].equalsIgnoreCase ("K") || checkPcode [4].equalsIgnoreCase ("L") || checkPcode [4].equalsIgnoreCase ("M") || checkPcode [4].equalsIgnoreCase ("N") || checkPcode [4].equalsIgnoreCase ("O") || checkPcode [4].equalsIgnoreCase ("P") || checkPcode [4].equalsIgnoreCase ("Q") || checkPcode [4].equalsIgnoreCase ("R") || checkPcode [4].equalsIgnoreCase ("S") || checkPcode [4].equalsIgnoreCase ("T") || checkPcode [4].equalsIgnoreCase ("U") || checkPcode [4].equalsIgnoreCase ("V") || checkPcode [4].equalsIgnoreCase ("W") || checkPcode [4].equalsIgnoreCase ("X") || checkPcode [4].equalsIgnoreCase ("Y") || checkPcode [4].equalsIgnoreCase ("Z") && checkPcode [5].equals ("0") || checkPcode [5].equals ("1") || checkPcode [5].equals ("2") || checkPcode [5].equals ("3") || checkPcode [5].equals ("4") || checkPcode [5].equals ("5") || checkPcode [5].equals ("6") || checkPcode [5].equals ("7") || checkPcode [5].equals ("8") || checkPcode [5].equals ("9")))
        {
            return ("Your postal code is correct");
        }
        else
        {
            return ("Your postal code is incorrect");
        }


MOD EDIT: put ur code within the CODE tags please


-----------------------------------

rizzix

Tue Jan 13, 2004 7:25 pm



-----------------------------------

wow thats a lot of code u got there. id suggest u use a for loop instead of that huge if statement.


-----------------------------------

wtd

Mon Feb 09, 2004 11:43 am



-----------------------------------

Or at least put the validation into a separate function/method which in turns uses a loop.  This also looks like a good place to use a regular expression.

http://www.compsci.ca/v2/viewtopic.php?t=1795