Wow. I don't want to seem too critical - I know you are just learing this stuff - but that code is
much harder to follow than it need be. Aside from it being inside a JSP (please head Scott's advice) the choice of variable names is
terrible. And there are no comments. And no encasuplation.
So, what is your code doing:
1. You get a connection (using the JDBC-ODBC bridge - why?).
2. You create a Statement (which you then don't use for 20 lines)
3. You define two Strings that represent SQL (which you then don't use for 20 lines)
4. You get
another connection to the same database.
5. You create another Statement
6. You define a bunch of variables, including three more bits of what looks like SQL
7. You get a ResultSet using the first connection from the SQL "Select wt from Nifty"
8. You get a ResultSet using the second connection from the SQL "Select SUM(mcap) as t from Nifty" (why are you using more than one connection? why did you choose to do the sum in SQL?)
9. You define a couple of empty Strings
10. You step through the first ResultSet assigning the value wt to the variable weight.
11. if the weight is greater than some value you do some maths and create what looks like it might be more SQL
12. if the weight is less than some value you do some more maths and create some more of what looks like it might be SQL
13. otherwise you break out of the loop (so if weight == 0.20 the loop will end).
14. Now you start another loop through the second ResultsSet
15. You update a table many times overwriting each value for wt (Why do all the wt values have to be the same? And if they do, why not just issue one SQL statement?)
16. You execute two more bits of SQL
Now, the answer to your question is somewhere in the above. I could just have cut to the chase and told you outright, but I hade to understand all of the above first to work out what might be going wrong. This is a big failing in the code you've posted: I suggest you have a read through your course notes for the term "
encapsulation and see if you can improve your code. You will find it far quicker and easier to work if you do.
