Форум программистов, компьютерный форум CyberForum.ru

Работа с calloc и free в C++ - C++

Восстановить пароль Регистрация
 
Рейтинг: Рейтинг темы: голосов - 10, средняя оценка - 4.90
kapitan-kuljok
0 / 0 / 0
Регистрация: 01.02.2009
Сообщений: 44
14.02.2012, 13:13     Работа с calloc и free в C++ #1
Всем добрый день!! Уважаемые программисты посмотрите пожалуйста, корректно ли я использую эти функции для динамического выделения памяти, прога вроде работает, но есть сомнения, вот код:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
struct Item
{
    unsigned int numberSite; //номер участка
    std::string address;
    unsigned int numberOfVoters;//количество поголосовавших
    Item *next;// указатель на следующий элемент списка
};
 
class Elections
{
    public:
        Elections(int number);// конструктор класса number - кол-во записей.
        void Add(unsigned int number, const char * address, unsigned int Voters);//метод добавления данных в односвязный список
        int Search(); // поиск  номера участка, в котором больший процент проголосовавших. Возвращает порядковый номер элемента, в котором находится данный изберательный участок
        void Print(int number);// входной параметр является i-ым элементом односвязного списка
        ~Elections();// деструктор класса. Удаляет весь список по элементно
    private:
        int count;// кол-во элементов
        Item *head; // указатель на начало списка
        Item *prev; // указатель на последней элемент
};
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
#include "stdafx.h"
#include <conio.h>
#include <iostream>
#include <stdlib.h>
#include "elections.h"
 
 
Elections::Elections(int number)
{
    count = number;
    head = NULL;
}
 
void Elections::Add(unsigned int number, const char*  address , unsigned int voters)
{
    Item* elem;
    elem = (Item*) (calloc(count, sizeof(Item)));
    //Item *elem=new Item();// создаем новый элемент односвязного списка
    if (head==NULL)
        head = elem;
    else 
        prev->next = elem;
        elem->numberSite = number;
        elem->address = address;
        elem->numberOfVoters = voters;
        elem->next = NULL;
        prev = elem;
}
 
int Elections::Search()
{
    Item *elem = head;
    unsigned int temp = elem->numberOfVoters;//temp - кол-во проголосовавших в первом учаске
    int tempCount = 1; 
    for (int i = 2; i<=count; i++)
    {
        elem = elem->next;
        if (temp < elem->numberOfVoters)
        {
            temp = elem->numberOfVoters;
            tempCount = i;
        }
    }
    return tempCount;
}
 
void Elections::Print(int number)
{
    setlocale(LC_ALL,"rus");
    Item *elem = head;
    for (int i = 1; i <= count; i++)
    {
        if (number == i)
        {
            std::cout<<"n\ Номер изберательного участка: "<<elem->numberSite;//номер изберательного участка, где проголосовало максимальное количество
            std::cout<<"\n Его адрес: "<<elem->address.c_str();
            std::cout<<"\n Количество проголосовавших: "<<elem->numberOfVoters;// количество изберателей
            getch();
            break;// выход из цикла, потому что найденный избер. участок может находится в в первом эл-те списка, а количество участков может быть большим. Нужно, чтобы не бегать по всем элементам.
        }
        elem = elem->next;// elem присваеваем адрес на следующий элемент
    }
}
Elections::~Elections()
{
    while (head != NULL)
    {
        Item *elem = head;
        head = head->next;
        free (elem);
    }
}
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include "stdafx.h"
#include <conio.h>
#include <iostream>
#include "elections.h"
#include <string>
using namespace std;
 
 
int _tmain(int argc, _TCHAR* argv[])
{
    setlocale(LC_ALL,"rus");    
    //setlocale(LC_ALL,"ru_RU");
 
    int count = 0; // количество записей
    unsigned int numberSite = 0; //номер участка
    std::string address = "";
    unsigned int numberOfVoters = 0;//количество поголосовавших
    
    cout<<"Введите количество изберательных участков: ";
    cin>>count;
    if (count == 0)
    {
        cout<<"Неккоректный ввод данных, нажмите любую клавишу!!";
        getch();
        exit(1);
    }
 
    Elections type(count);// создаешь объект Elections и вызываешь конструктор с входным параметром count
 
    for (int i = 1; i <= count; i++)
    {
        cout<<"\n Номер изберательного участка: ";
        cin >> numberSite;
        cout<<"\n Его адресс: ";
        cin>>address;
        cout<<address;
        cout<<"\n Количество проголосовавших: ";
        cin>> numberOfVoters;
        type.Add(numberSite,address.c_str(),numberOfVoters);
    }
    
    int temp = type.Search();
    type.Print(temp);
    
 
    return 0;
}
Добавлено через 45 минут
Посмотрите пожалуйста кто - нибудь!!
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
14.02.2012, 13:13     Работа с calloc и free в C++
Посмотрите здесь:

C++ malloc vs. calloc
calloc, free. C++
calloc vs malloc C++
C++ calloc
C++ Работа с памятью через malloc,realloc и free
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
rangerx
1908 / 1517 / 139
Регистрация: 31.05.2009
Сообщений: 2,876
14.02.2012, 13:17     Работа с calloc и free в C++ #2
Функции языка С для работы с памятью нельзя использовать для работы с конструкциями несвойственными этому языку, т: е. с классами. Эти функции ничего не знают о конструкторах, следовательно ни сам объект по отношению к которому применяются эти функции, ни его поля не будут сконструированы. Используйте операторы new и delete.
zss
Модератор
Эксперт С++
 Аватар для zss
5953 / 5558 / 1787
Регистрация: 18.12.2011
Сообщений: 14,204
Завершенные тесты: 1
14.02.2012, 13:26     Работа с calloc и free в C++ #3
Не факт, что не будет работать.
Во всяком случае, под переменные память выделяться будет.
kapitan-kuljok
0 / 0 / 0
Регистрация: 01.02.2009
Сообщений: 44
14.02.2012, 13:42  [ТС]     Работа с calloc и free в C++ #4
В начале я естественно выделял память с помощью new delete, но преподаватель сказал, что в тему этой лабороторной работы не входит использование этих операторов, и сказал использовать calloc free. Я тоже читал о том что эти функции C с классами не дружат, но у меня всё работает, вот по этому и есть сомнения..
rangerx
1908 / 1517 / 139
Регистрация: 31.05.2009
Сообщений: 2,876
14.02.2012, 14:01     Работа с calloc и free в C++ #5
zss2, сколько объектов по-вашему будет сконструировано в следующем примере?
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
#include <iostream>
#include <cstdlib>
 
class A
{
  int* p;
public:
  A() : p(new int[100])  { std::cout << "A()\n"; }
  //...
};
 
class B
{
  A a;
public:
  B() { std::cout << "B()\n"; }
};
 
struct C
{
  A a;
  B b;  
};
 
int main()
{
  A* a = (A*)malloc(sizeof(A) * 10);
  B* b = (B*)malloc(sizeof(B) * 10);
  C* c = (C*)malloc(sizeof(C) * 10);
 
  free(a);
  free(b);
  free(c);
}
Добавлено через 10 минут
Цитата Сообщение от kapitan-kuljok Посмотреть сообщение
В начале я естественно выделял память с помощью new delete, но преподаватель сказал, что в тему этой лабороторной работы не входит использование этих операторов, и сказал использовать calloc free. Я тоже читал о том что эти функции C с классами не дружат, но у меня всё работает, вот по этому и есть сомнения..
В таком случае замените std::string address(который у вас на самом деле не создаётся) в структуре Item на простой символьный массив.
kapitan-kuljok
0 / 0 / 0
Регистрация: 01.02.2009
Сообщений: 44
14.02.2012, 14:04  [ТС]     Работа с calloc и free в C++ #6
rangerx, можете пояснить смысл этой замены
rangerx
1908 / 1517 / 139
Регистрация: 31.05.2009
Сообщений: 2,876
14.02.2012, 14:33     Работа с calloc и free в C++ #7
kapitan-kuljok, я уже пояснил смысл этой замены двумя предыдущими постами. Память под объект std::string adress выделена будет, но сам объект не будет сконструирован(его конструктор не будет вызван), а это знаичт, что и конструкторы всех полей этого(и т.д.) класса так же не будут вызваны. Думаю нетрудно догадаться, что работать корректно такой объект не будет.
-=ЮрА=-
Заблокирован
Автор FAQ
14.02.2012, 18:16     Работа с calloc и free в C++ #8
Цитата Сообщение от kapitan-kuljok Посмотреть сообщение
Всем добрый день!! Уважаемые программисты посмотрите пожалуйста, корректно ли я использую эти функции для динамического выделения памяти, прога вроде работает, но есть сомнения, вот код:
- в плюсах думаю следует использовать плюсы, а не делать суржик С/С++(ИМХО malloc/realloc в С++ программе будет свидетельством смеси языков - т.е суржика) см сюда как можно сделать динамику на плюсах Изменить программу
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
14.02.2012, 20:01     Работа с calloc и free в C++
Еще ссылки по теме:

C++ Использование calloc/realloc/free
Чем заменить calloc? C++
C++ Malloc / calloc

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
zss
Модератор
Эксперт С++
 Аватар для zss
5953 / 5558 / 1787
Регистрация: 18.12.2011
Сообщений: 14,204
Завершенные тесты: 1
14.02.2012, 20:01     Работа с calloc и free в C++ #9
Цитата Сообщение от rangerx Посмотреть сообщение
zss2, сколько объектов по-вашему будет сконструировано в следующем примере?
Поскольку память выделена через malloc, то надо понимать, что конструктор не выполниться.
Т.е. в классе "a" память выделилась под указатель p, который не будет проинициализирован.

Вопрос о sizeof(A)*10. Зачем остальные 9 шт?
Yandex
Объявления
14.02.2012, 20:01     Работа с calloc и free в C++
Ответ Создать тему
Опции темы

Текущее время: 08:28. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru