Hi Thufir,
Would there a be a better approach to this class?
It's just supposed to output a result set. however, since there has to
be an update mechanism, I figured that might as well be incorporated as
well.
One thing I notice is that there's no real need to extend table, so I
think I'm taking maybe a bad approach.
Depends on your preferences. If you're in need of an ecs table
constructed from a resultset, then i don't see any real problems with
this approach. Maybe, i would prefer some kind of factory instead, but
that's my personal opinion.
Also, I'm getting a null pointer error further down, commented in CAPS.
At first the data is good, then a null value. Oh, maybe the start of a
nonexistent column? Not sure.
Depends on your database: If some columns allow null values, you can of
course get NULL in a resultset. When you know the type of a cols value
is int (col 1), you can directly get the int from the resultset by using
rs.getInt(j), with rs.wasNull() you can check if the last cols value
in database was null (you're in need of this check, because when ie
using getInt(), you don't know if the resulting int 0 is 0 because
that's the value in database or in database there was null.
---snip---
for (int j = 1; j <= colCount; j++) {
TD td = new TD();
td.setPrettyPrint(isPretty);
if (j == 1) {
int id = Integer.parseInt(element);
Form form = this.getDeleteButtonForm(resultSet.getInt(j));
td.addElement(form);
} else {
String element = resultSet.getString(j);
if ( resultSet.wasNull() ) {
td.addElement("-");
} else {
Form form = new ElementForm(element, 1,2);
//form.addElement(element);
td.addElement(form);
}
}
tr.addElement(td);
addElement(tr);
}
---snip---
grretings,
Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: ecs-user-unsubscr...@jakarta.apache.org
For additional commands, e-mail: ecs-user-h...@jakarta.apache.org
