[SOLVED] Insert row from Qtableview to another

SGaist

You can also use an event filter to catch that event, this really is up to you to choose the solution that meets your need

advseo32

i have implemented keypressEvent function

and it works perfectly, but when i start to somme conditions the programme crashes

her is my code @void MonTableauWidget::keyPressEvent(QKeyEvent *event)
{
int LastRow = this->rowCount();

switch (event->key()) {
case Qt::Key_Down :
// the problem occured  when i added the line below if(!this->item(LastRow,0)->text().isEmpty())
    this->insertRow(this->rowCount());
    break;
case Qt::Key_Up :
 this->removeRow(1);
    break;
case Qt::Key_0 :

    QMessageBox::information(this,"Row coutn",QString::number( LastRow)) ;
    break;
default:
    QTableWidget::keyPressEvent(event);
    break;
}

}
@

SGaist

@removeRow(1); << What would happen if you don't have a row number 1 ?@

advseo32

i have used the debugger to determine what's the issue
and i think the issue is located her
because when i delete this line every thing works perfectly
@this->item(LastRow,0)->text().isEmpty() ;@

this not refering to the customQtableWidget it's reffering to the
QtableWidgetItem objet

advseo32

[quote author="SGaist" date="1374875944"]@removeRow(1); << What would happen if you don't have a row number 1 ?@[/quote]

nothing is happened , means no row have to removed , but i have correct this issue

@this->removeRow(this->rowCount()-1); @
and this should works , and finally it works

thank's for the hint

advseo32

her is my finall code @void MonTableauWidget::keyPressEvent(QKeyEvent *event)
{
int LastRow = this->rowCount()-1;

// bool ok = this->item(LastRow,0)->text().isEmpty() ;

switch (event->key()) {
case Qt::Key_Down :
    if(LastRow)
        this->insertRow(this->rowCount());
    break;
case Qt::Key_Up :
 this->removeRow(LastRow);
    break;
default:
    QTableWidget::keyPressEvent(event);
    break;
}

@

advseo32

i lost hope to get the solution from my self,

the only problem now is how to test if Cell is Empty

i use this

@this->item(row,col)->text().isEmpty(); @

but the programme crashes , what the probleme her ??

JKSH

[quote author="advseo32" date="1374923562"]
@this->item(row,col)->text().isEmpty(); @

but the programme crashes , what the probleme her ??[/quote]If there is no data in the cell, then this->item() returns NULL.

Your program will crash if you dereference a null pointer.

advseo32

[quote author="JKSH" date="1374928103"][quote author="advseo32" date="1374923562"]
@this->item(row,col)->text().isEmpty(); @

but the programme crashes , what the probleme her ??[/quote]If there is no data in the cell, then this->item() returns NULL.

Your program will crash if you dereference a null pointer.

[/quote]

Yeah! that's true , because QTableWidgetItem was created only when the user fill in the cell

do you have another way to check if the cell if not empty

JKSH

[quote author="advseo32" date="1374942257"]do you have another way to check if the cell if not empty [/quote]If there is no data in the cell, then this->item() returns NULL :)

advseo32

[quote author="JKSH" date="1374942566"][quote author="advseo32" date="1374942257"]do you have another way to check if the cell if not empty [/quote]If there is no data in the cell, then this->item() returns NULL :)[/quote]

Probably , you have give the answer from the the begning , but i haven't pay attention to it

yes it's works
@if (this->item(LastRow,0) != 0)@

advseo32

now

i want to change the behavior of vertical header in Qtableview

this is what i mean:

when user start fill table cells, i want this sign(*) present in vertical header instead of numbers

when user moves beteween rows , i want also this sign(>) present in vertical header instead of numbers

what function i need to implement

SGaist

Did you read the documentation of the various classes you are using ? For example "cellChanged":http://qt-project.org/doc/qt-4.8/qtablewidget.html#cellChanged ?

advseo32

yes i did, but what functions are responsible for doing this

sign(*) present in vertical header instead of numbers

sign(>) present in vertical header instead of numbers

JKSH

[quote author="SGaist" date="1374952377"]Did you read the documentation of the various classes you are using ?
[quote author="advseo32" date="1374964095"]yes i did, but what functions are responsible for...[/quote][/quote]Please read more thoroughly.

Go to the "documentation":http://qt-project.org/doc/qt-5.1/qtwidgets/qtablewidget.html again and search for "header". Try different functions with good names to see what they do.

advseo32

i have try many functions but no result

Her is my try
this function called when the user start editing cell "CellChanged Signal"
@
void Dialog::newfunction()
{
QModelIndex index ;
index = tableau->model()->index(0,0) ;
tableau->verticalHeader()->model()->setData(index,QPixmap("imgs/arrow.png"),Qt::DecorationRole);
}

@

JKSH

Good on you for trying.
[quote author="advseo32" date="1375041952"]
Her is my try
this function called when the user start editing cell "CellChanged Signal"
@
void Dialog::newfunction()
{
QModelIndex index ;
index = tableau->model()->index(0,0) ;
tableau->verticalHeader()->model()->setData(index,QPixmap("imgs/arrow.png"),Qt::DecorationRole);
}

@
[/quote]There are two major issues:

If you haven't set any header data, tableau->verticalHeader()->model() returns NULL

You have 2 models here. tableau->model() is the table model, tableau->verticalHeader()->model() is the header model. You created an index for the table model, so you can't use that index to set data in the header model.

Hint: If you use setHeaderData(), you don't need an index

advseo32

i have use setHeaderData but no thing shown

@tableau->verticalHeader()->model()->setHeaderData(0,Qt::Vertical,"***", Qt::DisplayRole) ;@

SGaist

Since you are using QTableWidget, did you take a look at this part of the "QTableWidget documentation":http://qt-project.org/doc/qt-4.8/qtablewidget.html#verticalHeaderItem ?

JKSH

[quote author="advseo32" date="1375120030"]i have use setHeaderData but no thing shown

@tableau->verticalHeader()->model()->setHeaderData(0,Qt::Vertical,"***", Qt::DisplayRole) ;@[/quote]You are calling setHeaderData() on the header's header. Read my point #2 again.

Call setHeaderData on the table's header.

Read SGaist's link too.