jdbc foreign keys problem

[axenos]

Hello,

this is my code for an array in a database with 4 other arrays. C2J includes 3 fields: its id (primary key), idJava, idComponent...idJava and idComponent is from another 2 arrays from the DB.

public C2J(Statement statement) throws SQLException{

int up = statement.executeUpdate("INSERT INTO C2J (idJava,idComponent)"
+ "VALUES((SELECT idJava FROM JavaFiles),(SELECT idComponent FROM Components))");
}

I get an error that says:
Subquery returns more than 1 row

I tried using LIMIT 1 next to the ids (btw idJava and idComponent are foreign keys) but it doesn't really work right.

Any help?

[masijade]

What exactly are you trying to do? Are you trying to make a 1 to 1 connection between the two tables (one idJava to exactly one idComponent) or are you trying to create a 1 to many connection (ie one row for every idComponent per idJava or vce-versa)?

[Tolls]

And that's an unusual constructor it has to be said.

[axenos]

I know it's a strange constructor, but C2J has no other functionality...

my connection is one to many: many JavaFiles in one Constructor

I think I'll just have to add "Select idComponent FROM Components WHERE componentName=........"

ps. thanks for your relpy

[Tolls]

But that's my point.
Why have SQL executing in a constructor?
It's really (really) not the place for it.

[masijade]

INSERT INTO C2J (idJava,idComponent) (SELECT a.idJava, b.idComponent FROM JavaFiles a, Components b)

[axenos]

ok...i made a function for this and the constructor calls that function.

masijade i tried my way and your way and the result is the same. I keep getting this error:
Subquery returns more than 1 row

[masijade]

Don't use the values keyword (if that's what you are doing) and you may need to actually remove the parens around the select subquery. You want an insert-select not an insert-values.

That you are doing this stuff in a constructor (whather as a method called from a constructor, or directly in the constructor) definately more than hints at a design problem, though.