C ++ инициализирующий объект и выделяющий память

Question

C ++ инициализирующий объект и выделяющий память

0

У меня есть структура вроде:

struct myNode{
    Node node; // node object takes just name parameter I mean node(string name);
    int value;
};

И в моей функции main() у меня есть указатель struct myNode, и я хочу сначала прочитать файл, чтобы получить имя объекта и значения узла (просто целое число), а затем создать соответствующую структуру myNode. Для этого я пишу

int main(){
    struct myNode * myRoot;
    ifstream myfile (fileName);
    string line;
    getline(myfile,line);
int value=getValue(line);
myRoot= new myNode();
    myRoot->value=value;
myRoot->node=node a(line);

  return 0;    
}

Тем не менее, я не уверен, это правильный способ или нет, есть ли кто-нибудь, кто мог бы мне помочь?

caesar 27 дек. 2013, в 11:07

Источник

0

Где определен Node ? Покажите нам больше.
legends2k 27 дек. 2013, в 09:31
2

Кроме того, node a(line); очень подозрительно
Kakadu 27 дек. 2013, в 09:32
0

Из того, что ты нам показываешь, мне кажется, все в порядке. Я предполагаю, что myRoot->node = node a(line) является Node a(line) (Node not node)
EoiFirst 27 дек. 2013, в 09:33
1

даже в случае структуры я предпочел бы конструктор. т.е. myNode (int val, Node n): значение (val), node (n) {}. Вы можете использовать его следующим образом: myRoot = new MyNode (value, a (line)); // какой бы ни была (строка).
cageman 27 дек. 2013, в 09:33
1

@EoiFirst: Нет, это не так. a является неопределенным идентификатором. Возможно, myRoot->node=Node(line) подходит.
legends2k 27 дек. 2013, в 09:34

Показать ещё 3 комментария

Теги:

c++

class

oop

2 ответа

Ещё вопросы

Где определен Node ? Покажите нам больше.
Кроме того, node a(line); очень подозрительно
Из того, что ты нам показываешь, мне кажется, все в порядке. Я предполагаю, что myRoot->node = node a(line) является Node a(line) (Node not node)
даже в случае структуры я предпочел бы конструктор. т.е. myNode (int val, Node n): значение (val), node (n) {}. Вы можете использовать его следующим образом: myRoot = new MyNode (value, a (line)); // какой бы ни была (строка).
@EoiFirst: Нет, это не так. a является неопределенным идентификатором. Возможно, myRoot->node=Node(line) подходит.

0x499602D2 · Answer 1 · 2013-12-27T13-28-00.000Z

Среди многих смертоносных проблем в вашем коде, тот, который я вижу, должен быть для вас наиболее неприятным, - это тот, который на этой строке:

myRoot->node = node a(line);

node a(line) является синтаксической ошибкой. Я полагаю, вы это понимали:

myRoot->node = node(line);

или, если Node не имеет explicit конструктора:

myRoot->node = line;

Другими опасными проблемами являются тот факт, что вы используете указатели, выделяя new (оба из которых являются ненужными) и что вы выполняете извлечение в потоке файлов, не проверяя, работает ли он. Все это можно решить, выделив объекты в стеке и проверив, удалось ли выполнить вход:

int main()
{
    myNode myRoot;
    ifstream myfile(fileName);
    string line;

    if (getline(myfile, line))
    {
        int value = getValue(line);
        myRoot.value = value;
        myRoot.node  = line;
    }    
}

WKPlus · Answer 2 · 2013-12-27T07-39-00.000Z

Да, он может работать, кроме:

Вы забыли удалить myRoot перед выходом функции main. Да, os может перерабатывать его, когда вы выходите из программы, но вы можете сделать свой код более изящным.
myRoot->node= node a(line) может быть просто myRoot->node = Node(line) или myRoot->node = line если конструктор конструктора myRoot->node = line может принимать линию как единственный параметр. Я полагаю, вы рассматриваете как временный экземпляр узла.

myRoot-> node = line, если конструктор struct Node может принимать строку как единственный параметр - Нет! Поскольку myRoot->node уже произошло, наличие такого конструктора не поможет. Это будет работать только в том случае, если есть оператор присваивания, принимающий string или сам Node является string .
@ legends2k Извините, я что-то пропустил. Если есть оператор =, который принимает строку или другой экземпляр узла, он будет работать.