Как правильно передать void в функцию (C++/Qt)?

Question

[Александр]

Есть вот такой код. В качестве примера пытаюсь вызвать метаметод layout() у наследника от QWidget (Q_INVOKABLE установлен).
В функции call() все нормально работает, но в somefunction() переменная *data приходит сломанной или пустой:

bool call(QObject *qo, QMetaMethod metaMethod, void **data) {
  QGenericReturnArgument returnArgument(metaMethod.typeName(), &data);
  bool ok = metaMethod.invoke(qo, Qt::DirectConnection, returnArgument);
 
  if(ok) {
    QLayout *layout = reinterpret_cast<QLayout *>(data); // Всё ок
    qDebug() << layout ->metaObject()->className(); // QLayout
  }
 
  return ok;
}
 
void somefunction() {
  /* ... */
  void *data;
  bool ok = call( qo, metaMethod, &data );
 
  if(ok) {
    QLayout *layout = reinterpret_cast<QLayout *>(data); // Segfault
    qDebug() << layout ->metaObject()->className(); 
  }
}

Что я делаю не так?

Comments

[Евгений Шатунов]

Я ответ писать не буду, только коммент. У тебя там все не так делается, даже там, где ты написал что все ок, всё все равно не так. Делать из типа **void тип *QLayout - это очень не так и очень нехорошо.
Тебе срочно надо почитать про принципы управления памятью и про работу с объектами в С++.

[Александр]

Да даже если там будет просто void* (и так я тоже пробовал), оно не уходит дальше функции call. А почему не так и почему не хорошо? Компилируется без предупреждений, запускается тоже. Но не передается по ссылке.

[Евгений Шатунов]

Плюсы - это самый лучший язык для того чтобы попасть себе в голову, стреляя себе в ногу, не делая для этого ничего. Поэтому у тебя все нормально компилится, однако в ногу оно стреляет и в голову ты себе исправно попадаешь.
Ты в принципе неправильно пользуешься как приведением типов, так и вообще переменными. Тебе надо читать документацию по плюсам, а не общаться на тостере.

[Александр]

Что значит неправильно =) во всех примерах что я видел войд кастуется в любой тип, тем более, если я на 100% уверен в том, какой тип храниться в нём. Ну не сделал я разыменовывание **void в приведенном коде - это не значит что во всех своих терзаниях я мусолил именно этот подход... Я выложил то, что последнее отработало хотя бы на половину. Проблема то не в касте...

Answer 1

[Александр]

Короче сделал так:

bool call(QWidget *qo, QMetaMethod metaMethod, QGenericReturnArgument *returnArgument)
{
    /* ... */
    return metaMethod.invoke(qo, *returnArgument);
}

void somefunction()
{
    void *vlayout;
    QGenericReturnArgument returnArgument(metaMethod.typeName(), &vlayout);

    if( call(qo, metaMethod, &returnArgument) ) {
        QLayout *layout = reinterpret_cast<QLayout*>(vlayout); 
        /* ... */
    }
}

работает.

Answer 2

[maaGames]

QGenericReturnArgument returnArgument(metaMethod.typeName(), &data)

замена на

QGenericReturnArgument returnArgument(metaMethod.typeName(), *data)

Comments

[Александр]

Если сделать так, то приложение умирает еще в функции call()

[maaGames]

Александр: Учитывая, что конструктор:
QGenericReturnArgument(const char * name = 0, void * data = 0)
, а ты в него передаёшь void***, то правильным было бы именно *data, чтобы передавать void*. А почему падает... Заменить
QLayout *layout = reinterpret_cast(data);

на

QLayout *layout = reinterpret_cast(*data);

[Александр]

maagames.ru: увы, так тоже не работает ;(

Answer 3

[jcmvbkbc]

Думаю, что должно быть так:

bool call(QObject *qo, QMetaMethod metaMethod, void **data) {
  QGenericReturnArgument returnArgument(metaMethod.typeName(), data);
  bool ok = metaMethod.invoke(qo, Qt::DirectConnection, returnArgument);
 
  if(ok) {
    QLayout *layout = reinterpret_cast<QLayout *>(*data); // Всё ок
    qDebug() << layout ->metaObject()->className(); // QLayout
  }
 
  return ok;
}

Т.е. ваша ошибка в том, что вместо переданного из somefunction data вы используете локальную копию data из call.

Comments

[Александр]

ну если так написать в call(), то оно срабатывает, а в somefunction - нет, по прежнему крашится :/

[jcmvbkbc]

Хз, должно работать (: Попробуйте напечатать и сравнить значения layout в somefunction и в call.

[Александр]

jcmvbkbc: так я и не могу обратиться к этому значению в somefunction. Любое обращение приводит к сегфорлту.

[jcmvbkbc]

Не надо к нему обращаться. После успешного вызова метаметода:
qDebug() << layout;
или
qDebug() << *data в call и qDebug() << data; в somefunction.

[jcmvbkbc]

И в somefunction должно было остаться

QLayout *layout = reinterpret_cast<QLayout *>(data);

[Александр]

jcmvbkbc: да, это я понял

[Александр]

сделал пустой проект с двумя функциями. Немного поменял вызов: избавился от указателя на указатель - pastie.org/10333718

Вывод:

CONSTRUCTOR: QVBoxLayout(0x26b9870)
CLICK: QVBoxLayout(0x26b9870)
metaMethod name: "layout"
QGenericReturnArgument
invoke
CALL: 0x26b9870
CALL CAST: QVBoxLayout(0x26b9870)
OK: 0x61e0e565
Программа неожиданно завершилась.

[Александр]

я вижу что приходит левый адрес, но не могу понять почему

[jcmvbkbc]

Александр: так и не будет работать: вы опять используете локальную переменную внутри call..
Если вам не принципиально как именно передавать указатель, проще будет передать его по ссылке. В вашем последнем примере надо поменять единственную строчку:

bool MainWindow::call(QWidget *qo, QMetaMethod metaMethod, void *&vlayout)

[Александр]

jcmvbkbc: Спасибо =) уже по другому сделал

Answer 4

[Андрей Буров]

Внутри call
data у Вас имеет тип void **
а внутри somefunction - void *

QLayout *layout = reinterpret_cast<QLayout *>(&data); // Segfault

Comments

[Александр]

Так я пробовал. Вообще не компилируется:
[code]internal compiler error: in get_polymorphic_call_info, at ipa-devirt.c:1223[/code]

[Андрей Буров]

Александр: а зачем Вы гоняете void * если у Вас везде конкретный тип используется QLayout?

[Александр]

Андрей Буров: не конкретный. Это пример, в котором не работает самый простой метод :-(
В data может быть что угодно, тип определяется из метаметода