This week's book giveaway is in the OCMJEA forum. We're giving away four copies of OCM Java EE 6 Enterprise Architect Exam Guide and have Paul Allen & Joseph Bambara on-line! See this thread for details.
Here's a handy tip that you can apply to all your code. In a catch block, never, never, never, never, never do something like System.out.println("Error HRadd"); At the very least dump the exception so that you can see what went wrong:
If it turns out that the error is something to do with the syntax of that SQL, it's probably because this statementdidn't leave a blank space between the column list (EmpID) and the word "values". Was it really necessary to do that string concatenation instead of just using a single string literal? Makes it much easier to make mistakes like that.