Что означает строка?

Question

[Максим Руднев]

Я новичок, код не мой, пытаюсь разобраться с синтаксисом. Интересует 26, 27, 28, 29, 30 строка
Если тут ясно, что в скобках параметры

Get::Get(const QString &peer, const QString &objectId, QObject *parent)

то далее не ясно :

QObject(parent),
    peer(peer),
    objectId(objectId),
    socket(new QUdpSocket(this))

что происходит выше?

#include "Get.h"

#include "Types/DataTypeFactory.h"

#include "Types/ObjectIdentifier.h"
#include "Types/Null.h"

#include "Types/Sequence.h"
#include "Types/SnmpMessage.h"

#include "Types/SnmpVersion.h"
#include "Types/OctetString.h"

#include "Types/Varbind.h"
#include "Types/VarbindList.h"

#include "Types/GetRequestPDU.h"
#include "Types/GetResponsePDU.h"

#include <QUdpSocket>

#ifdef QT_SNMP_DEBUG
#include <QDebug>
#endif

Get::Get(const QString &peer, const QString &objectId, QObject *parent) :
    QObject(parent),
    peer(peer),
    objectId(objectId),
    socket(new QUdpSocket(this))
{    
    socket->bind(10500);
    connect(socket,SIGNAL(readyRead()),SLOT(readPendingDatagram()));

}

void Get::execute()
{
    SnmpVersion *version = new SnmpVersion(SnmpVersion::SNMPv1);
    OctetString *community = new OctetString("public"); //#TODO: Make configurable.

    Integer *requestId = new Integer(1);
    Integer *error = new Integer(0);
    Integer *errorIndex = new Integer(0);

    ObjectIdentifier *objectIdentifier = new ObjectIdentifier(objectId);
    AbstractSyntaxNotationOne *value = new Null();
    VarbindList *varbindList = new VarbindList(SequenceData{new Varbind(objectIdentifier, value)});

    ProtocolDataUnit *getRequestPDU = new GetRequestPDU(requestId, error, errorIndex, varbindList);

    SnmpMessage snmpMessage(version, community, getRequestPDU);
    QByteArray datagram = snmpMessage.encode();
    socket->writeDatagram(datagram, QHostAddress(peer), 161);
}

QStringList Get::getResponse() const
{
    return response;
}

void Get::readPendingDatagram()
{
    while (socket->hasPendingDatagrams()) {
        QByteArray datagram;
        datagram.resize(socket->pendingDatagramSize());
        QHostAddress sender;
        quint16 senderPort;
        socket->readDatagram(datagram.data(), datagram.size(), &sender, &senderPort);

        QDataStream stream(&datagram, QIODevice::ReadOnly);
        Type::AbstractSyntaxNotationOneType type;
        stream >> type;

        if (type != Type::Sequence)
            throw SnmpException("Wrong response");

        quint8 bytesRead = 0;
        SnmpMessage sequence;
        bytesRead += sequence.decode(stream);

        QStringList responses;
        VarbindList *varbindList = sequence.getProtocolDataUnit()->getVarbindList();
        foreach (Varbind *varbind, varbindList->getVarbinds()) {
            responses += QString("%1 = %2 : %3")
                    .arg(varbind->getObjectIdentifier()->toString())
                    .arg(DataTypeFactory::getTypeName(varbind->getValue()->getType()))
                    .arg(varbind->getValue()->toString());

            response.append(varbind->getValue()->toString());
        }

        qDebug() << QString("SNMP datagram has been read!\n"
                            "Version: %1\n"
                            "Community: %2\n"
                            "Responses: %3")
                    .arg(sequence.getSnmpVersion()->getVersion() + 1)
                    .arg(sequence.getCommunity()->getValue())
                    .arg(responses.join("\n"));
    }
}

вот GET.h

#pragma once

#include <QObject>
#include <QStringList>

class QUdpSocket;
class Get : public QObject
{
    Q_OBJECT

    QString peer;
    QString objectId;

    QUdpSocket *socket;

    QStringList response;
public:
    Get(const QString &peer, const QString &OID, QObject *parent = 0);

    void execute();

    QStringList getResponse() const;

private Q_SLOTS:
    void readPendingDatagram();
};

Comments

[sitev_ru]

опубликуйте Get.h - тогда станет понятнее

[Максим Руднев]

sitev_ru: добавил. прокомментируйте

[sitev_ru]

Free_ze всё уже расписал

Answer 1

[Толстый Лорри]

Это инициализация полей класса.

Если бы вы писали в Java-стиле - так (QObject пока опустим):

Get::Get(const QString &peer, const QString &objectId, QObject *parent) :
    QObject(parent)
{    
    this->peer = peer;
    this->objectId = objectId;
    this->socket = new QUdpSocket(this);
    ...

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

Что же касается QObject, он - родительский класс для типа Get, потому на нем тоже необходимо вызвать конструктор QObject(QObject *parent)

Comments

[sitev_ru]

Мне такой вариант, как вы написали в Java-стиле больше по душе. Почему это два действия?

[Антон Жилин]

sitev_ru: Ещё до начала работы конструктора все поля класса уже обязаны быть инициализированы. Если мы не добавим такие инициализаторы, то компилятор инициализирует из за нас конструктором по умолчанию, то есть будет выполнять дополнительную работу в рантайме.
P.S. Встроенные типы можно оставить неинициализированными, но это плохая практика.

[Толстый Лорри]

sitev_ru: Потому что это C++ и в нем такие правила =) Члены класса обязательно должны быть инициализированы до начала исполнения тела конструктора. Если этого не сделаете вы явно, то за вас это сделает компилятор (сгенерировав код вызова умолчательных конструкторов).

[sitev_ru]

Вот так вот писать на разных языках, упускаешь такие тонкости... Хотя, с другой стороны, наверняка, начальная инициализация - это просто выделение участка памяти под объект где-нибудь в "куче" (которе и так нужно нам) и по времени мы не теряем НИЧЕГО

[Толстый Лорри]

sitev_ru: В Java объекты (ссылочные типы же) тоже инициализируются дефолтным значением - null, примитивные типы - нулями и т.п., а потом уже применяются ваши конструкторы. В C++ при использовании указателей они тоже будут иметь значение nullptr. Но ссылочность не всегда имеет смысл. Что же касается базового класса, то в Java конструктор базового типа (super) тоже должен быть явно вызван в начале конструктора, т.е. разница чисто синтаксическая.

[Толстый Лорри]

Если что-то можно размещать на стеке (маленький объект, часто создается-удаляется), то лучше это там. Память в куче без GC - это отдельная проблема. Нужно писать десруктор, т.е. вступает в игру "правило пяти", соблюдать принцип RAII. Для объектов в стеке подходит умолчательный деструктор и вызывается он автоматически с родительским деструктором. Это проще =)

[sitev_ru]

Написал проверочный код для vs2013. Если поле класса описать, например, как целое, то начальное значение будет случайным, значит время на инициализацию не тратиться. Если добавляем указатель, то все поля обнуляются (и указатели и не указатели), видимо участок, выделяемый под класс тупо обнуляется. Предположу, что если есть хотя бы одно не инициализированное поле и есть указатель, то на практике (а не в теории), компилятор в любом из двух случаев особо не парится и вначале выделяет память под класс, а потом эту всю память обнуляет. Получается, что в любом случае мы тратим одно и тоже время. В крайнем случае, если будем писать в Java-стиле мы проиграем несколько процессорных тактов :)

[Толстый Лорри]

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

Да и если хочется писать в Java-style больше, чем писать производительный код, может лучше и писать на Java? =)

[Максим Руднев]

Free_ze: могу я попросить объяснить на примере попроще? на примере какого-нибудь дефолтного и простого класса. Просто первый раз вижу подобный синтаксис и не могу разложить по полкам в голове

[Толстый Лорри]

Максим Руднев: Когда объект (не указатель и не примитивный тип) создается, то _всегда_ вызывается какой-то конструктор. Если пользователь не определил никаких конструкторов, то вызывается конструктор по умолчанию (который автоматически сгенерировал компилятор).

Члены класса - это подобъекты, для которых тоже как-то нужно вызывать разные интересные конструкторы. Для этого есть список инициализаторов конструктора (не путать с std::initializer_list).

Попробуйте разобраться в этом: pastebin.com/s0rySQ0f