Обращение к структуре в классе с выделением памяти

16.09.2011, 18:37. **Просмотров** 634. **Ответов** 3

Добрый день уважаемые форумчане.
Столкнулся с такой проблемой. Есть класс:

C++

class Components {
    struct  comps{ // -- База компонентов
        char name[50];
    };
    comps  *components;
 
    int ReadComponents();
    int ReadProducts();
};
 
 
int Components::ReadProducts() {
    int b = ReadComponents();
 
    printf ("Результат: %s",components[5].name);
    // Выводится полная фигня, такое ощущение, что память выделяется только 
        //   в пределах метода, а после уничтожается
}
 
int Components::ReadComponents() {
    comps  *components = (comps*)malloc(sizeof(comps)*10);
    strcpy(components[5].name,"Prostotak");
}

и

C++

int main(int argc, char **argv) {
    Components components;
    int a = components.ReadProducts();
}

Ожидаемый ответ работы программы: Prostotak
Однако выводится полная ерунда...

Похоже на то, что метод ReadComponents выделяет память, отрабатывает верно и после завершения эту память освобождает, таким образом к Components::ReadProducts возвращается полная ерунда, которую он и выводит.

Объясните пожалуйста, как правильно выделить память или где это сделать, чтобы программа отработала верно.
Заранее спасибо за помощь.

@OstapBender · 16.09.2011, 19:02

Сообщение от Spliver

// Выводится полная фигня, такое ощущение, что память выделяется только
// в пределах метода, а после уничтожается

так и есть, вы создали локалную переменную comps *components

надо писать просто

C++

1	components = (comps)malloc(sizeof(comps)10);

и да, почему не new ?

@insolent · 16.09.2011, 19:15

Spliver, в не void ф-ция всегда должен быть return. Из вне можно обращаться только к public методам или полям класса..
Так более правильнее..

C++

class Components {
    struct  comps{ // -- База компонентов
        char name[50];
    };
    comps  *components;
 
public:
    int ReadComponents();
    int ReadProducts();
};
 
 
int Components::ReadProducts() {
    int b = ReadComponents();
 
    printf ("Результат: %s",components[5].name);
    // Выводится полная фигня, такое ощущение, что память выделяется только 
    //   в пределах метода, а после уничтожается
    return 0;
}
 
int Components::ReadComponents() {
    components = (comps*)malloc(sizeof(comps)*10);
    strcpy(components[5].name,"Prostotak");
 
    return 0;
}

16.09.2011, 19:27

Сообщение от OstapBender

так и есть, вы создали локалную переменную comps *components

надо писать просто

C++

1	components = (comps)malloc(sizeof(comps)10);

и да, почему не new ?

Все заработало Большое спасибо!
Я портирую эту программу с C, еще не до конца разобрался с new

To insolent: конечно я везде ставлю return, просто здесь я делал вырезку кода, чтобы понятнее объяснить свою проблему

Опции темы