OleDbException was Unhandled - Crashes after a mins

Asked Feb 03 '17 at 13:19

Active Jul 05 '20 at 21:12

Viewed 50 times

After approximately 40 seconds of running this code and inserting ~400 records into the table I get a

OleDbException was Unhandled - Unspecified Error.

A copy of the code:

        clsDBConnector dbConnector = new clsDBConnector();
        OleDbDataReader dr;
        string sqlStr;
        dbConnector.Connect();
        sqlStr = " SELECT        MAX(EventID) AS MaxID" +
                 " FROM          Events";
        dr = dbConnector.DoSQL(sqlStr);
        while (dr.Read())
        {
            EventID = Convert.ToInt16( dr[0].ToString());
        }
        dr.Close();
        sqlStr = " SELECT        MemberID" +
                 " FROM          GroupMembers" +
                 " WHERE (GroupID = " + GroupID + ")";
        dr = dbConnector.DoSQL(sqlStr);
        while (dr.Read())
        {
            MemberID = Convert.ToInt16( dr[0].ToString());
            clsDBConnector dbConnectorInsert = new clsDBConnector();
            string cmdStr = "INSERT INTO EventsAtendees " +
                            "(MemberID, EventID) " +
                            "VALUES        (" + MemberID + ", " + EventID + ")";
            dbConnectorInsert.Connect();
            dbConnectorInsert.DoDML(cmdStr);
            dbConnectorInsert.close();
        }
        dr.Close();

I have taken out the custom fields and variables to make this more useable by everyone.

edited Jul 05 '20 at 21:12

Martijn Pieters

1,048,767
296
4,058
3,343

asked Feb 03 '17 at 13:19

PowerBIQuery

Is this the complete SQL you show or an abbreviation? – trailmax Feb 03 '17 at 13:21
@trailmax an abbreviation so that it'd b more accessible to everyone. – PowerBIQuery Feb 03 '17 at 13:21
@trailmax can put in full DML if needed – PowerBIQuery Feb 03 '17 at 13:22
1

You'll need to show more code/context for anybody to be able to answer your question – trailmax Feb 03 '17 at 13:22
@trailmax I've added more context – PowerBIQuery Feb 03 '17 at 13:24
You realy should use sql parameters. – Christian Gollhardt Feb 03 '17 at 13:25
You can really do this as a single SQL Query. Lookup "Insert into select". Something like this: http://www.w3schools.com/sql/sql_insert_into_select.asp – trailmax Feb 03 '17 at 13:29
What is clsDBConnector? – Polyfun Feb 03 '17 at 13:30
@DanielGrist And yes, sql parameters are a must-have: http://csharp-station.com/Tutorial/AdoDotNet/Lesson06 and http://stackoverflow.com/questions/7505808/why-do-we-always-prefer-using-parameters-in-sql-statements – trailmax Feb 03 '17 at 13:31
@Polyfun the connector to the database. – PowerBIQuery Feb 03 '17 at 13:31
The error is occurring in the DML statement – PowerBIQuery Feb 03 '17 at 13:33

OleDbException was Unhandled - Crashes after a mins

0 Answers0