hi
ok, im not clear about what you are trying to achieve and where you have troubles with your code (several pieces are missing: some classes like FileOutput and so on). but i have some tips for you:
- if your Fileinput/outpout classes have some try/catch, make sure you print the exception at least to the console (Exception.printStacktrace()). better would be to pass them on and react on them.
- don't compare Strings using '=='! use the equals() method. in short: '==' compares the reference while equals() compares the state
- when you write to streams (e.g.) a file then make sure you close them in a finally clause. i dont see you closing any stream but perhaps thats encapsulatet in the FileInput/output classes ?
- try to stick with
java coding guidelines: the member "AccountName" should start lowercase.
- in your createAccount method you take have an "accountName" parameter but in the end you write the "AccountName" member/field to the file. if this is a null reference, and you "swallow" exceptions in your Fileoutput class, then this could be your problem: NullPointerException.
- your methods return Boolean (wrapper object) according to the signature but you return boolean (primitive). this is no problem because java 1.5 autoboxes/unboxes them.... but i was not sure if you understand the differences ?
- your createAccountPass method does always return "true"... therefore there is no sense in returning anything at all. just make the method "void"
have fun
pascal
[ May 08, 2006: Message edited by: pascal betz ]