[SOLVED] Insert row from Qtableview to another


  • Lifetime Qt Champion

    Again, are you asking us to do consulting for your work ?



  • [quote author="SGaist" date="1375217386"]Again, are you asking us to do consulting for your work ?[/quote]

    sorry, but not this what i mean

    i have consulte the documentation and tried many signal and slots
    but i don't get what i want

    i have find @cellChanged(int row, int column)@ but this is not what i want



  • i have used this Signals but really no this what i want

    @
    cellChanged()
    cellEntered()
    currentCellChanged()
    currentItemChanged()
    itemActivated()
    itemChanged()
    itemEntered()
    @

    i want a signal that allows me to catch this behavior

    when the user enter in cell no thing happened her , but when he start typing her is my goal , "catch the start of editing or typing"

    plz d'ont unserstand me in wrong way, i have read the documentation many and many time but i don't get any thing


  • Lifetime Qt Champion

    This function is not ready made so you'll have to dig a bit deeper, look for QStyledItemDelegate and QEvent::EnterEditFocus


  • Moderators

    Ah, I see now. Sorry, I misunderstood what you wanted the table to do.

    Qt's model/view classes produce signals when you:

    • Select a cell
    • Finish typing into a cell

    However, they don't provide signals when you start typing into a cell. So, you have to implement your own low-level event handlers to detect typing.

    You can subclass the QTableWidget and reimplement "QWidget::keyPressEvent()":http://qt-project.org/doc/qt-5.1/qtwidgets/qwidget.html#keyPressEvent

    @
    class MyTable : public QTableWidget {
    // ...
    };

    void MyTable::keyPressEvent(QKeyEvent* event) {

    // 1. Check if the event is caused by typing
    if (...) {
        qDebug("The user typed something!");
    }
    
    // 2. Pass the event back to the QTableWidget,
    // to store the typed text
    QTableWidget::keyPressEvent(event);
    

    }
    @

    See the "QKeyEvent":http://qt-project.org/doc/qt-5.1/qtgui/qkeyevent.html documentation and choose a way to do #1.

    [quote author="SGaist" date="1375272324"]look for QStyledItemDelegate and QEvent::EnterEditFocus[/quote]Unfortunately, QEvent::EnterEditFocus is not available in normal Qt builds -- the QT_KEYPAD_NAVIGATION macro is not enabled.

    [quote author="advseo32" date="1375270136"]plz d'ont unserstand me in wrong way, i have read the documentation many and many time but i don't get any thing [/quote]I understand that it can be tiring to read, especially if English is not your first language. Your perseverance is very good.

    Please remember: When you ask questions (on qt-project.org, or on any other forum), it is very important to describe what you already tried.

    Earlier, you said "and i need your hints to detect when the user start typing in a cell" but you didn't say what you tried. This was bad, because it looked like you haven't done any work yourself.

    Later on, you explained "i have used this Signals but really no this what i want" and you listed the signals you tried. This was much better, because it showed us that you had done work yourself and tried many things.

    Please continue to be detailed in your future questions.



  • i'm already subclassed the Qtablewidget and i have implemented

    keyPressEvent(QKeyEvent* event)

    this is the code @void MonTableauWidget::keyPressEvent(QKeyEvent *event)
    {

    int LastRow = (this->rowCount() == 0) ? 0 : this->rowCount()-1 ;
    int currentRows = this->currentRow() ;
    switch (event->key()) {
    case Qt::Key_Down :
        if( this->item(LastRow,0) != 0)
          LastRow == currentRows ? this->insertRow(this->rowCount()) : QTableWidget::keyPressEvent(event) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    case Qt::Key_Up :
        if(LastRow > 0 & this->item(LastRow,0) == 0)
         LastRow == currentRows ? this->removeRow(LastRow) : QTableWidget::keyPressEvent(event) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    

    /*
    i will try to do this
    case Qt::(key != Keys(down,up,..etc) :
    setVerticalheader(row,icon);
    */
    default:
    QTableWidget::keyPressEvent(event);
    break;
    }

    }@



  • I tried this for Key_p, but i need this action for other case != Key_Dow,Key_Up

    this is a part of above code

    @void MonTableauWidget::keyPressEvent(QKeyEvent *event)
    {

    int LastRow = (this->rowCount() == 0) ? 0 : this->rowCount()-1 ;
    int currentRows = this->currentRow() ;
    //int row = this->selectionModel()->currentIndex().row();
    QTableWidgetItem * item = new QTableWidgetItem() ;
    switch (event->key()) {
    case Qt::Key_Down :
        if( this->item(LastRow,0) != 0)
          LastRow == currentRows ? this->insertRow(this->rowCount()) : QTableWidget::keyPressEvent(event) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    case Qt::Key_Up :
        if(LastRow > 0 & this->item(LastRow,0) == 0)
         LastRow == currentRows ? this->removeRow(LastRow) : QTableWidget::keyPressEvent(event) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    
    case Qt::Key_P :
    
         item->setIcon( *(new QIcon("imgs/Edit-icon.png")));
         this->setVerticalHeaderItem(currentRows,item);
         QTableWidget::keyPressEvent(event);
        break;
    
    default:
        QTableWidget::keyPressEvent(event);
        break;
    }
    

    ;@



  • Now it's working

    i need your advises about my implementation, (it's bad ? , it's good ?)
    @void MonTableauWidget::keyPressEvent(QKeyEvent *event)
    {

    int LastRow = (this->rowCount() == 0) ? 0 : this->rowCount()-1 ;
    int currentRows = this->currentRow() ;
    QTableWidgetItem * item = new QTableWidgetItem() ;
    switch (event->key()) {
    case Qt::Key_Down :
        if( this->item(LastRow,0) != 0)
          LastRow == currentRows ? this->insertRow(this->rowCount()) : QTableWidget::keyPressEvent(event) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    case Qt::Key_Up :
        if(LastRow > 0 && this->item(LastRow,0) == 0)
         LastRow == currentRows ? this->removeRow(LastRow) : QTableWidget::keyPressEvent(event) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    default:
        item->setIcon( *(new QIcon("imgs/Edit-icon.png")));
        this->setVerticalHeaderItem(currentRows,item);
        QTableWidget::keyPressEvent(event);
        break;
    }
    

    }
    @


  • Lifetime Qt Champion

    [quote author="JKSH" date="1375283945"] Unfortunately, QEvent::EnterEditFocus is not available in normal Qt builds — the QT_KEYPAD_NAVIGATION macro is not enabled.[/quote]

    Right ! I forgot about that one

    Quick code review (not talking about whether it works)
    Your variables name doesn't seem to follow a coding style, use camel case everywhere will make it more readable (there are different styles but that's another story let's use this one). Also even if this is code have a good grammar/orthography:

    @int currentRows = this->currentRow(); @

    currentRows implies that there are several rows implicated which is not the case following your statement.

    @int lastRow = qMax(0, this->rowCount() -1);@

    You should also refactor your tests, using one liner like that doesn't make the code readable, especially if in the end the second choice is the same as the else code.

    @
    if (this->item(LastRow,0) != 0) &&
    lastRow == currentRows)
    this->insertRow(this->rowCount());
    else
    QTableWidget::keyPressEvent(event);
    @

    If you have a test that is done repeatedly you can also use a variable with a meaningful name

    @bool isCurrentRowLast = (lastRow == currentRow);@

    Don't forget that the code you are writing now might be the same code that you'll have to read in a year. Being able to go through it without having to do deep analyzes to understand what is does will prove valuable also for your co-workers.



  • Her is my code , after your advices

    @void MonTableauWidget::keyPressEvent(QKeyEvent *event)
    {

    int LastRow =qMax(0,this->rowCount()-1);
    int CurrentRow = this->currentRow() ;
    bool isCurrentRowLast = (LastRow == CurrentRow);
    bool isCurrentCellEmpty = this->item(LastRow,1) == 0;
    QTableWidgetItem * item = new QTableWidgetItem() ;
    switch (event->key()) {
    case Qt::Key_Down :
        if( !isCurrentCellEmpty)
          (isCurrentRowLast) ? this->insertRow(this->rowCount()) : QTableWidget::keyPressEvent(event) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    case Qt::Key_Up :
        if(LastRow > 0 && isCurrentCellEmpty)
         isCurrentRowLast ? this->removeRow(LastRow) : QTableWidget::keyPressEvent(event) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    default:
        item->setIcon( *(new QIcon("imgs/Edit-icon.png")));
        this->setVerticalHeaderItem(CurrentRow,item);
        QTableWidget::keyPressEvent(event);
        break;
    }
    

    }@


  • Moderators

    1. This is a memory leak:
      @
      item->setIcon( *(new QIcon("imgs/Edit-icon.png")));
      @
      setIcon() will make a copy of your icon, but the original stays in memory forever.

    2. Your variable names have mixed style -- some start with upper-case (e.g. CurrentRow), but some start with lower-case (e.g. isCurrentRowLast). Choose one style and don't mix them.

    In Qt's style, all function names and variable names start with lower-case, and all class names start with upper-case.

    1. Your checking is complicated:
      @
      if( !isCurrentCellEmpty)
      (isCurrentRowLast) ? this->insertRow(this->rowCount()) : QTableWidget::keyPressEvent(event) ;
      else
      QTableWidget::keyPressEvent(event);
      @
      Just write if (!isCurrentCellEmpty && isCurrentRowLast). Your other case has the same issue.


  • Ok

    this my final version (Thank's for your advices )

    @void MonTableauWidget::keyPressEvent(QKeyEvent *event)
    {

    int LastRow =qMax(0,this->rowCount()-1);
    int CurrentRow = this->currentRow() ;
    bool IsCurrentRowLast = (LastRow == CurrentRow);
    bool IsCurrentCellEmpty = this->item(LastRow,1) == 0;
    QTableWidgetItem * item = new QTableWidgetItem() ;
    QIcon *ShowEditSignIcon = new QIcon("imgs/Edit-icon.png");
    switch (event->key()) {
    case Qt::Key_Down :
        if(IsCurrentRowLast && !IsCurrentCellEmpty)
          this->insertRow(this->rowCount()) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    case Qt::Key_Up :
        if(LastRow > 0 && IsCurrentCellEmpty && IsCurrentRowLast)
         this->removeRow(LastRow) ;
        else
          QTableWidget::keyPressEvent(event);
        break;
    default:
        item->setIcon(*ShowEditSignIcon);
        delete ShowEditSignIcon;
        this->setVerticalHeaderItem(CurrentRow,item);
        QTableWidget::keyPressEvent(event);
        break;
    }
    

    }
    @


  • Moderators

    That looks a lot tidier, well done :)

    You don't have to create a QIcon on the heap with new QIcon(). Just create it on the stack (no pointer) with QIcon ShowEditSignIcon("imgs/Edit-icon.png"); -- then you don't have to delete it yourself.



  • Thank's JKSH and SGaist, now i can be said, i have figured out more than 30% of Model/view architecture in Qt

    if i have another question, i will post her , i will create another thread ?


  • Moderators

    You're welcome. Good luck with Qt programming. :)

    You should start new threads for new topics. Remember to provide details of what you already tried.



  • i start use the same strategy as we did with Qtablewidget but

    there is no setVerticalHeaderItem(row,item) function like Qtablewidget(it seems that QtableWidget is very easy than Qtableview

    my question her :

    should i sublcass QHeaderView class and implement neccessary function
    after setting up vertical header @setVerticalHeader(QHeaderView * header)@

    or i can use another solution with out subclassing QheaderView ???



  • i have used this but no thing happened

    i use Qtableview with Qsqlquerymodel

    her is my code
    @ui->tableView->verticalHeader()->model()->
    setData(ui->tableView->verticalHeader()->model()->index(1,1),"**",Qt::DisplayRole) ;
    @


  • Moderators

    [quote author="advseo32" date="1375355486"]if i have another question, i will post her , i will create another thread ?[/quote]Why did you ask this question and then ignore my answer?



  • i have posted this question because is similaire topic but now is Qtableview

    Sorry, if i'm wrong , i will post another thread :)


  • Moderators

    [quote author="advseo32" date="1375434213"]i have posted this question because is similaire topic but now is Qtableview

    Sorry, if i'm wrong , i will post another thread :)[/quote]Yes, please post in a new thread. It is NOT similar.

    Each thread is supposed to have 1 topic, but you already have 5 topics:

    How to automatically insert a row at the end when the user clicks/presses a key there

    How to check if a cell is empty

    How to add custom symbols to the vertical header

    How to check if the user selects a different row

    How to check if the user has started typing into a row

    (Your new topic) How to modify headers

    The title of this thread is "Insert row from Qtableview to another", but topics 2-6 are not related to this.

    Forums let users search old topics to find solutions solutions. But, if a thread has many topics that are not related to the title, it's hard for someone to search.


Log in to reply
 

Looks like your connection to Qt Forum was lost, please wait while we try to reconnect.