...

Signature
Andrew Thompson
http://www.athompson.info/andrew/
sahm wrote:
>> I can not get data from Table cell
> ...
>> I try to get String from cell in jTable
> Unless you mean JTable, I do not know the class
> you are referring to, and if you *are* referring to
> JTable, please take care to use the correct
> capitalisation.
sahm wrote:
>> every time I try to get data from cell I only get null
>> and this is my code
> * <http://www.physci.org/codes/sscce.html>
sahm wrote:
>> Class.forName("com.mysql.jdbc.Driver");
You only need to load the class once, not every time you make a connection.
>> Connection con;
>> con = DriverManager.getConnection("jdbc:mysql://localhost/uni","root","java");
Why not
Connection con =
DriverManager.getConnection( "jdbc:mysql://localhost/uni", "root", "java" );
?
>> Statement stat = con.createStatement();
>> ResultSet result;
>> result = stat.executeQuery("select max(no) from student_info_table");
Why not
ResultSet result =
stat.executeQuery( "select max(no) from student_info_table" );
?
>> result.next();
Ingo R. Homann wrote:
> Are you sure, result.next() returns true?
>> int mx, c;
>> mx = result.getInt(1);
>> mx = mx +1;
You set mx to a value, then increment it separately instead of in one
statement, and you don't ever set a value for c.
>> String inserting_Data = "insert into fee_part_table (feeNO, feeParts,
>> fee_1st_part) values(?, ?, ?)";
>> PreparedStatement ps = null;
>> ps = con.prepareStatement(inserting_Data);
Why initialize ps then immediately discard the initial value?
>> ps.setInt(1, mx);
>> ps.setInt(2, c);
c was never set.
>> String st;
>> st = String.valueOf(jTable2.getValueAt(0, 0));
Why not
String st = String.valueOf( jTable2.getValueAt(0, 0) );
?
>> JOptionPane.showMessageDialog(this, st);
Intermingling GUI and DB code like this is not good structure. You should
also be sure that GUI code runs on the Event Dispatch Thread (EDT) and that
the database code does not.
>> ps.setString(3, String.valueOf(jTable2.getValueAt(0, 0)));
>> ps.executeUpdate();
Roedy Green said:
> ... you posted this twice independently. This splits the discussion.
> Don't do this. See
<http://mindprod.com/jgloss/multiposting.html>
> These are code snippets. They do not even compile here,
> let alone display the problem. Perhaps if you prepared an
[quoted text clipped - 3 lines]
>
> * <http://www.physci.org/codes/sscce.html>

Signature
Lew
apm35@student.open.ac.uk - 25 Sep 2007 08:24 GMT
> Intermingling GUI and DB code like this is not good structure. You should
> also be sure that GUI code runs on the Event Dispatch Thread (EDT) and that
> the database code does not.
I'd put it a bit stronger than that. Intermingling GUI and DB code
like this means that the GUI may not work properly since GUI work must
be done in the EDT in order for swing to function properly. Also, when
in the EDT, non-GUI work must be shunted off to a different thread so
that the EDT is only doing GUI work.
-Andrew Marlow