Problem with virtual functions

[eekhoorn12]

Hey all

I am trying to create a treeview in which the parent holds different information than its children, so i created two seperate classes, postheaderitem and postdataitem, for this and one abstract class, abstractpostitem, which also serves as a rootitem.

From a dialog I create an entire new tree item with its child items and return that. That item I want to append to my rootitem. Before I append it I want to set the parent of the new item so it points to rootItem.

void PostModel::addPost(AbstractPostItem newPost)
{
    newPost.setParent(rootPost);
    this->insertRows(rootPost->childCount(),1,newPost,QModelIndex());
}

void PostModel::addPost(AbstractPostItem newPost)
{
    newPost.setParent(rootPost);
    this->insertRows(rootPost->childCount(),1,newPost,QModelIndex());
}

To copy to clipboard, switch view to plain text mode 

The type created in my dialog is of postheaderitem and that is passed to this function as newPost. But the setParent function is called in the abstractpostitem and not in the postheaderitem.

So I am wondering if I should cast newPost to a postheaderitem or if I am doing something else wrong.

Regards
eekhoorn12

[wysota]

Hmm... what exactly is the problem again? What is postheaderitem? What is AbstractPostItem? Are you sure you don't need to use pointers anywhere?

[eekhoorn12]

I have changed my code some to make use of pointers, the previous problem is solved but I still have a problem.

I will try to explain the situation some more.

I have a treeview which I would like to populate. It is going to be a treeview with a headerbar en below that one level that describes a certain group with its own information and it has child items to have some other information.

To create this I tried to make one abstract class named AbstractPostItem, this class als functions as the root item of the treeview.

For the description items in the treeview I created PostHeaderItem which inherits AbstractPostItem and for the child items of PostHeaderItem I created PostDataItem which also inherits AbstractPostItem.

Now from a dialog I try to create a new PostHeaderItem with its children, for that I use the following code:

PostHeaderItem::PostHeaderItem(QStringList files, QString subject, QString group)
{
    foreach(QString i, files)
    {
        PostDataItem *newItem = new PostDataItem(i);
        newItem->setSubject(subject);
        newItem->setGroup(group);
        newItem->setParent(this);
        addItem(newItem);
    }
    setSubject(subject);
    setGroup(group);
}

PostHeaderItem::PostHeaderItem(QStringList files, QString subject, QString group)
{
    foreach(QString i, files)
    {
        PostDataItem *newItem = new PostDataItem(i);
        newItem->setSubject(subject);
        newItem->setGroup(group);
        newItem->setParent(this);
        addItem(newItem);
    }
    setSubject(subject);
    setGroup(group);
}

To copy to clipboard, switch view to plain text mode 

Which is called from:

AbstractPostItem *NewPost::getPost()
{
    PostHeaderItem *newItem = new PostHeaderItem(files, ui->inputSubject->text(), ui->inputUsenetGroup->text());
    return newItem;
}

AbstractPostItem *NewPost::getPost()
{
    PostHeaderItem *newItem = new PostHeaderItem(files, ui->inputSubject->text(), ui->inputUsenetGroup->text());
    return newItem;
}

To copy to clipboard, switch view to plain text mode 

Which is called from:

void PostView::newPost()
{
    AbstractPostItem *tmpAbstractPostItem;
    postScreen = new NewPost(this);
    postScreen->setWindowFlags(Qt::Window);
    postScreen->setAttribute(Qt::WA_ShowModal);
    if(postScreen->exec() == QDialog::Accepted)
    {
        tmpAbstractPostItem = postScreen->getPost();
        postsList->addPost(tmpAbstractPostItem);
        postScreen->close();
    }
}

void PostView::newPost()
{
    AbstractPostItem *tmpAbstractPostItem;
    postScreen = new NewPost(this);
    postScreen->setWindowFlags(Qt::Window);
    postScreen->setAttribute(Qt::WA_ShowModal);
    if(postScreen->exec() == QDialog::Accepted)
    {
        tmpAbstractPostItem = postScreen->getPost();
        postsList->addPost(tmpAbstractPostItem);
        postScreen->close();
    }
}

To copy to clipboard, switch view to plain text mode 

Now my problem is that in the first and second code bit everything works just as expected, but when I try to call functions from tmpAbstractPostItem in the third code bit it uses the functions in the AbstractPostItem class instead of the PostHeaderItem class which is what I want.

So my question is how can I accomplish this?

Update:
Added the files as attachment.

[wysota]

Server isn't always running, usually during european day time and evening it is up.

I have deleted links to the code from your post. Please do it the proper way - attach files to your post instead of linking to external sites, especially ones that might not work. If you manage to squeeze into 6 hours from posting the original post, you can (and should) edit it and add the attachment there.

[eekhoorn12]

Found out what my problem was. The problem was that the function declaration in the AbstractPostItem and PostHeaderItem were not the same. The one in PostHeaderItem was a const and the one in AbstractPostItem was not. When I changed that it worked.