QSqlQuery problem (insert once but create two rows!!!)

[ditsikts]

When I insert a name, create two rows

Here is insert statement

void wmtDB::addEmployee(QString name)
{
    QSqlQuery qry;
    qry.prepare("INSERT INTO employeestuff (name) VALUES (:name)");
    qry.bindValue(":name", name );
    if (!qry.exec())
        qFatal("Failed to add employee");
}

void wmtDB::addEmployee(QString name)
{
    QSqlQuery qry;
    qry.prepare("INSERT INTO employeestuff (name) VALUES (:name)");
    qry.bindValue(":name", name );
    if (!qry.exec())
        qFatal("Failed to add employee");
}

To copy to clipboard, switch view to plain text mode 

and here the call

void MainWindow::buttonOnClick()
{
    employees.addEmployee(ui->lineEdit->text());
    updateEmployees();
}

void MainWindow::buttonOnClick()
{
    employees.addEmployee(ui->lineEdit->text());
    updateEmployees();
}

To copy to clipboard, switch view to plain text mode 

if db is empty and i run it one time with parameter "john"
the "SELECT * FROM nameTable"
will return
1 "john"
2 "john"

please help me

[nix]

Connection between your button and your slot is done two times, so your slots is call twice. There is one connect in your ui add by QtDesigner probably (pass in signal/slot view to see it), and one in the constructor.

[ditsikts]

thank you, I fix it

[DmitryNik]

For reason of avoiding creating a new thread. Could that line of code "connect(this, SIGNAL(senddata(const QString &)), server, SIGNAL(getdata(const QString &)));" cause the problem, which was described above?

here is the code, what I have in the server-class:

qDebug() << "name: " + name << "staus: " + status << "pass: " + pass;
    bool ok;
    {
        QSqlDatabase db = QSqlDatabase::addDatabase("QMYSQL", "db");
        db.setHostName(this->dbAddress);
        db.setPort(this->dbPort);
        db.setDatabaseName(this->dbName);
        db.setUserName(this->AdminName);
        db.setPassword(this->AdminPass);
        ok = db.open();
        if(ok)
        {
            QSqlQuery query("INSERT INTO people(name, status, pass) VALUES('" + name + "', '"
                            + status + "', '" + pass + "')", db);
            query.exec();
            query.clear();
        }
        db.close();
    }
    QSqlDatabase::removeDatabase("db");

qDebug() << "name: " + name << "staus: " + status << "pass: " + pass;
    bool ok;
    {
        QSqlDatabase db = QSqlDatabase::addDatabase("QMYSQL", "db");
        db.setHostName(this->dbAddress);
        db.setPort(this->dbPort);
        db.setDatabaseName(this->dbName);
        db.setUserName(this->AdminName);
        db.setPassword(this->AdminPass);
        ok = db.open();
        if(ok)
        {
            QSqlQuery query("INSERT INTO people(name, status, pass) VALUES('" + name + "', '"
                            + status + "', '" + pass + "')", db);
            query.exec();
            query.clear();
        }
        db.close();
    }
    QSqlDatabase::removeDatabase("db");

To copy to clipboard, switch view to plain text mode 

Output is:

"name: user" "staus: user" "pass: 12dea96fec20593566ab75692c9949596833adc9"

In my humble opinion, if I got only line with names, passes and statuses, it doesn't. But still I've got every time two new lines in the database table... Where can be a problem?

[stampede]

If you are afraid that connection has been made multiple times, use Qt::UniqueConnection

connect(this, SIGNAL(senddata(const QString &)), server, SIGNAL(getdata(const QString &)),Qt::UniqueConnection);

connect(this, SIGNAL(senddata(const QString &)), server, SIGNAL(getdata(const QString &)),Qt::UniqueConnection);

To copy to clipboard, switch view to plain text mode