1

mi problema es que al ejecutar el siguiente código:

void MainWindow::on_loginButton_clicked() {
  QString userName = ui->userLineEdit->text();
  QString userPassword = ui->passwordLineEdit->text();

  QSqlQuery q;
  q.exec("SELECT name FROM users.name WHERE name = 'javier'");

  QString nombre = q.value(0).toString();
  qDebug() << nombre;
}

Me aparece el siguiente error al presionar el llamado loginButton:

QSqlQuery::value: not positioned on a valid record ""
4
  • Creo que tu query deberia ser "SELECT name FROM users WHERE name = 'javier'"
    – alanfcm
    el 5 abr. 2018 a las 18:04
  • Al principio probé eso, pero creeme que es lo mismo, el problema para mi que no es el sql
    – zuninool04
    el 5 abr. 2018 a las 18:16
  • 1
    Intenta añadir q.first() antes de acceder los datos, o sea despues de tu query.
    – alanfcm
    el 5 abr. 2018 a las 18:18
  • Gracias, esa era la razón
    – zuninool04
    el 5 abr. 2018 a las 18:29

1 respuesta 1

1

Tu programa tiene dos errores:

La consulta es incorrecta

Como te han comentado en los comentarios, debería ser:

"SELECT name FROM users WHERE name = 'javier'"

La sintaxis de SELECT es:

SELECT [COLUMNAS] FROM [TABLAS] WHERE [FILTROS]

Solo pueden existir nombres de columnas es los campos [COLUMNAS] y [FILTROS].

Iteración de consultas

Una vez que has ejecutado la consulta tienes que moverte por los diferentes registros devueltos. Si te fijas en tu código, para acceder a los valores devueltos usas:

QString nombre = q.value(0).toString();

Donde q es la propia QSqlQuery, es decir, el objeto sobre el que has ejecutado la consulta... Entonces, si la consulta devuelve 3 registros... ¿Cómo podemos acceder a cada uno? No existe algo tipo q.value([REGISTRO],[COLUMNA]), así que... ¿cómo se hace?

En la respuesta que has borrado dices de usar q.first()... esa sería una solución muy a medida. Hay una más genérica que permite trabajar con tantos registros como te devuelva la aplicación y es mediante el método next():

QSqlQuery q;

if(! q.exec("SELECT name FROM users.name WHERE name = 'javier'") )
{
  qDebug() << "Error en la consulta: "
           << q.lastError().text();

  return;
}

while( q.next() )
{
  QString nombre = q.value(0).toString();
  // ...
}

Si solo interesa un único registro se puede cambiar el while por un if... así consigues un código bastante homogéneo independiente de la cantidad de resultados obtenidos.

Por cierto, nota que es recomendable comprobar el valor de retorno de exec... no vaya a ser que la consulta no se haya podido ejecutar por alguna razón.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.