Get content of cell from QTableView

Karoluss96

well.. when I put last code in this topic I found that I forgot add ')' between values and word 'values' :-P

The only one thing to do is change default (non)value python's None to sql's NULL

jsulm

@Karoluss96 I'm really confused now: we were talking about using addBindValue and you post now code which does not use it at all - WHY? Why don't you use addBindValue?!

Karoluss96

For a few moments I did sth different with this code outside of this topic.

Now I try to use addBindValue, but I'm sure wher to put this function

addBindValue is in for loop:

  rowData = [ model.index(row, col).data() for col in range(model.columnCount()) ]
            a=query.addBindValue(rowData)
            print (a)
            iden = rowData[0]
...

JonB

@Karoluss96 said in Get content of cell from QTableView:

Now I try to use addBindValue, but I'm sure wher to put this function

Then you should take the time to search the QSqlQuery doc page for bindValue, where you would come across the examples in subtopic Approaches to Binding Values. It is a really good idea to search the docs for your question before posting to say you do not know how to do something.

  rowData = [ model.index(row, col).data() for col in range(model.columnCount()) ]
            a=query.addBindValue(rowData)

No, there is no loop where you say there is a loop in your code. You cannot just add the whole rowData list in a single addBindValue, you need to add a bound value for each placeholder in the INSERT statement.

Karoluss96

the whole loop looks that:

for col in range(model.columnCount()):
            row1=model.index(row, col).data()
            #print (row1)
            rowData = [ model.index(row, col).data() for col in range(model.columnCount()) ]
            a=query.addBindValue(rowData)
            print (a)
            iden = rowData[0]
            teryt = rowData[1]
...

So As I understand I have to move addBind Value for

eg. ?

  iden =query.addBindValue( rowData[0])

jsulm

@Karoluss96 said in Get content of cell from QTableView:

eg. ?
iden =query.addBindValue( rowData[0])

yes, for every value in your query you want to bind you need to call addBindValue as you can clearly see in the example:

QSqlQuery query;
query.prepare("INSERT INTO person (id, forename, surname) "
                  "VALUES (?, ?, ?)");
query.addBindValue(1001);
query.addBindValue("Bart");
query.addBindValue("Simpson");
query.exec();

JonB

@Karoluss96
Indeed, for each element in rowData in a loop. And you must make sure your INSERT query has the right number of column names and ? (or :<name> if you are using those) placeholders in the VALUES (...) clause to match the bound values added.

Karoluss96

@jsulm said in Get content of cell from QTableView:

query.prepare(

Now I have this:

query = QSqlQuery(db)
 sql = query.prepare('INSERT INTO ZARZADZANIE.ZAMOWIENIA (ID,TERYT,...) VALUES (?,?,...)')       
        index =self.dlg.tableView_3.selectionModel().currentIndex()
        dane= self.dlg.tableView_3.model().index(index.row(),0).data() 
        row = index.row()
        model = self.dlg.tableView_3.model()
        for col in range(model.columnCount()):
            rowData = [ model.index(row, col).data() for col in range(model.columnCount()) ]
            iden =query.BindValue(rowData[0], '')
            print (iden)
...

with error:
AttributeError: 'QSqlQuery' object has no attribute 'BindValue'

db is a global value:

db = QSqlDatabase.addDatabase("QOCISPATIAL")

jsulm

@Karoluss96 said in Get content of cell from QTableView:

AttributeError: 'QSqlQuery' object has no attribute 'BindValue'

Please , isn't this apparent?! Really...

jsulm

@Karoluss96 said in Get content of cell from QTableView:

db is a global value:
db = QSqlDatabase.addDatabase("QOCISPATIAL")

There should not be any such global db variables! Again: it is explained in the documentation which you should read: https://doc.qt.io/qt-6/qsqldatabase.html
"Warning: It is highly recommended that you do not keep a copy of the QSqlDatabase around as a member of a class, as this will prevent the instance from being correctly cleaned up on shutdown. If you need to access an existing QSqlDatabase, it should be accessed with database(). If you chose to have a QSqlDatabase member variable, this needs to be deleted before the QCoreApplication instance is deleted, otherwise it may lead to undefined behavior."

It is even in red color...

Karoluss96

Ah, Maybe I chose the wrong option for bind value from documentation.

I can't change line with db, because it was make by my head-chef and it's important for anothers sql-questions which are in this app.

JonB

@Karoluss96 said in Get content of cell from QTableView:

iden =query.BindValue(rowData[0], '')

AttributeError: 'QSqlQuery' object has no attribute 'BindValue'

Please use your own efforts to resolve an error such as this, instead of just asking every time. The forum is not intended for people to type in responses to every error you might encounter. The method is named and shown as all examples as bindValue(), you now choose to type BindValue(), your own knowledge of Python should tell you what the issue is....

Karoluss96

@JonB said in Get content of cell from QTableView:

Please use your own efforts to resolve an error such as this, instead of just asking every time. The forum is not intended for people to type in responses to every error you might encounter. The method is named and shown as all examples as bindValue(), you now choose to type BindValue(), your own knowledge of Python should tell you what the issue is....

Eh... making so quick can't help in solving problem.

Looking at the subject this discussion goes total out of topic.

Have I continue it here or finish, not to spam for future's users?

JonB

@Karoluss96
If it's still questions related to "Get content of cell from QTableView" you might as well continue in this topic.

Karoluss96

To be honest, I've tried some versions (look at 2 versions of sql variable ), but It haven't work yet.

Now I have:

sql = 'INSERT INTO ZARZADZANIE.ZAMOWIENIA (ID,TERYT,ZAM...) VALUES ('
#sql = query.prepare('INSERT INTO ZARZADZANIE.ZAMOWIENIA (ID,TERYT,ZAM...) VALUES (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)')

and later:

for col in range(model.columnCount()):
            rowData = [ model.index(row, col).data() for col in range(model.columnCount()) ]
            if (rowData[0] is None):
                iden =query.bindValue(rowData[0], '')
                print (iden)
            else:
                iden = rowData[0]
                print (iden)
            if (rowData[1] is None):    
                teryt = query.bindValue(rowData[1],'')
                print (teryt)
            else:
                teryt = rowData[1]
                print (teryt)
            if (rowData[2] is None):    
                zam = query.bindValue(rowData[2],'')
                print (zam)
...

and take here error:

dane2 =  iden + ',\'' + teryt + '\',\'' +zam +  ....+ '\''
        print (dane2)
        sqlCale = sql + dane2 + ')'

can only concatenate str (not "NoneType") to str.

On console for added data (in Table View, from which this question take values) I recive e.g. 7150,1401,UM14 if is fill or NONE it is not.

So still the same, but bindValue separate None from filled attributes

JonB

@Karoluss96 said in Get content of cell from QTableView:

can only concatenate str (not "NoneType") to str.

And which exact line (not 3 lines) does this error report? Does it even have anything to do with bindValue() call? You have already asked this. Python does not let you concatenate (i.e. join with + sign) None into a string, just as one might expect, so do something about it.

Karoluss96

...and that's the problem is in dane2 line

JonB

@Karoluss96
...so that is where you need to deal with the issue, it's a (not unexpected) Python issue you need to address, what else is there to say?

JonB

@Karoluss96
BTW: what are you actually joining together in dane2 = iden + ',\'' + teryt + '\',\'' +zam + ....+ '\'', and why? What are those variables? Since you are now using bindValue() for the values, the only things you would need to join into a comma-separated list are the column names for the INSERT statement, and no column would/should have a None name. So I don't know what you are up to anyway.

Karoluss96

It's a rest from previous version.

I'll continue that tommorow as I forgot about more importat thing: coverting date formats!
(I'm sure that I solve on my own)