Re: Help me regarding this topic
smart wrote:
Hi, to all i am new to core java and question on Swings. See this code
and help me.
We can only help if you tell is what you think is going wrong.
- is there a compiler error message?
- is there a run-time error message?
- does the output differ from what you expect? How exactly?
I want to calculate student marks and his average. When i click button
total marks will have calculate and i am posting code below
I think you really should pay attention to what Andrew and others have
said about formatting your source code - I find it hideously unreadable.
if(ae.getSource()==btnCal)
Looks like this is part of some event handler like an ActionListener.
It's a shame you don't provide a complete (but small) example program.
{
btnNew.setEnabled(true);
btnSave.setEnabled(false);
btnDel.setEnabled(true);
Connection conSno;
Statement stSno;
Presumably those are java.sql.Connection and java.sql.Statement - I
recommend you read about Model-View-Controller (MVC) patterns and try to
keep database code in separate classes from your GUI code.
int Total,Average;
That level of indentation is ridiculous, please convert tabs to two
(maybe four) spaces before posting. Most editors and IDEs I've used
allow you to configure them so that tabs are always converted to spaces.
Cariable names should start with lower case ("english" instead of
"English") otherwise you confuse readers by making variable names look
like class names.
int English = Integer.praseInt(tfEng.getText());
int Telugu = Integer.praseInt(tfTel.getText());
int Mathmatics = Integer.praseInt(tfMaths.getText());
int Science = Integer.praseInt(tfSci.getText());
int Social = Integer.praseInt(tfSoc.getText());
int Hindi = Integer.praseInt(tfHin.getText());
I'd have an array or HashMap of JTextFields to eliminate this repetition.
Class.forName("sun.Jdbc.Odbc.JdbcOdbcDriver");
conSno =
DriverManager.getConnection("jdbc:odbc:DRIVER{Microsoft Access
Driver(*.mdb)};DBQ=D:\\ProgressReport.mdb");
See above. Anyway, your JDBC code seems to be scattered around
incoherently, I'd gather it all up in one place.
Total = English+Telugu+Mathmatics+Science+Social+Hindi;
Average = Total / 6;
tfTot.setText(String.ValueOf(Total));
tfAvg.setText(String.ValueOf(Average));
stSno=conSno.createStatement();
ResultSet rsSno=stSno.executeQuery("select Tot,Avg from
ProgressReport where Sno = '"+Sno+"'");
if(rsSno.next())
I usually use while(xxx.next()) unless I have specifically checked that
only one record will be returned.
{
tfTot.setText(rsSno.getString(12));
tfAvg.setText(rsSno.getString(13));
getString(12) will not return anything useful since your SQL statement
only selects two fields from the table.
You are overwriting the text in tfTot, which you just put your
calculated total into. It is no surprise you don't ever see the
calculated total - I guess that is the actual problem (which you failed
to say).
} // end of the if condition
} // end of the btnCal
Unless you pay some attention to the advice about tabs and
capitalisation, I will find it very hard to summon the will to respond
any further.
--
RGB