using an isntance of QSqlDatabase for connection defiinition

[anda_skoa]

Now that you point it out that makes sense will move open() call inside if() statement, since the if statement is checking for validation would you also pass the instance to QSqlQuery inside the if() block?

No, I would leave the QSqlQuery outside.

do I need to close or remove the database?

No, you should be able to keep more than one connection at any given time.


Now I need help fixing my function in my other class

QSqlQuery selectQuery("SELECT id, userName, eventMessage, dateTime FROM userlogevents");

QSqlQuery selectQuery("SELECT id, userName, eventMessage, dateTime FROM userlogevents");

To copy to clipboard, switch view to plain text mode 

You likely want to use the m_selectDataBase for this query, no?

while (selectQuery.next()){
        //---Instance of userEventLogMsg Class Struct---//
        userEventLogMsg msg;
        //---Instance of userEventlog Class---//
        UserEventLog model;
        //---Add query data to the msg class struct---//
        msg.id = selectQuery.value(0).toString();
        msg.username = selectQuery.value(1).toString();
        msg.eventmessage = selectQuery.value(2).toString();
        msg.datetime = selectQuery.value(3).toString();
        //---Use model object to access an call addEvent()---//
        model.addEvent(msg);
    }

while (selectQuery.next()){
        //---Instance of userEventLogMsg Class Struct---//
        userEventLogMsg msg;
        //---Instance of userEventlog Class---//
        UserEventLog model;
        //---Add query data to the msg class struct---//
        msg.id = selectQuery.value(0).toString();
        msg.username = selectQuery.value(1).toString();
        msg.eventmessage = selectQuery.value(2).toString();
        msg.datetime = selectQuery.value(3).toString();
        //---Use model object to access an call addEvent()---//
        model.addEvent(msg);
    }

To copy to clipboard, switch view to plain text mode 

This doesn't look at all what I assume you want.
This creates a new model instance in every loop iteration, adds one record and then destroys the model (when it goes out of scope).

Cheers,
_