Помогите исправить ошибки в классе

@b0n4b · Регистрация: 16.11.2011

Студворк — интернет-сервис помощи студентам

Здравствуйте. Наконец то дописываю класс, но опять столкнулся с кое какими проблемами.
Задание: написать класс очередь через массив. Вот что у меня получилось:

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
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
#include<iostream>
#include<stdlib.h>
#include<string>
 
using namespace std;
 
class Queue
{
    int *queue;
    int tail, head, siz, x;
 
public:
    Queue();
    ~Queue();
    void push(int n);   // добавление элемента
    void clear();   // очистка очереди
    void pop(); // удаление 1го элемента   
    void size();  // количество элементов в очереди
    void front();  // узнать знач 1-ого элемента не удаляя его ????!!!!
};
 
Queue::Queue(int queue)
{
    queue = new int[256];
    tail=0;
    head=0;
    siz=0; 
}
 
Queue::~Queue()
{
    delete[]queue;
}
 
void Queue::clear()
{
    head=tail;
    cout <<"ok\n";
}
 
void Queue::size()
{
    siz=0;
    for(int i=head; i<tail; i++)
    {
        siz++;
    }
    cout << siz << "\n";
}
 
void Queue::front() 
{
    if(head==tail)
    {
        cout << "error\n";
    }
    else
    {
        cout << queue[head] << "\n";
    }
}
 
void Queue::push(int n)
{
    cin>>n;
    queue[tail]=n;
    tail++;
}
 
void Queue::pop()
{
    if(head==tail)
    {
        cout << "error\n";
    }
    else
    {
        cout << queue[head];
        queue[head]=x;
        head++;
        cout <<"\n";
    }
    
}
 
int main()
{
    Queue q(6);
    string s;
    for(;;)
    {
        cin>>s;
        if(s=="push")
        {
            int n;
            cin>>n;
            q.push(n);
        }
        else if(s=="pop")
        {
            q.pop();
        }
        else if (s=="clear")
        {
            q.clear();
        }
        else if (s=="size")
        {
            q.size();
        }
        else if (s=="front")
        {
            q.front();
        }
        else if (s=="exit")
        {
            cout <<"bye";
            return 1;
        }
    }
}

Программа не компилируется, выдаёт 2 ошибки.
Помогите исправить ошибки, недочёты, и распределение памяти.
До этого писал класс с динамической памятью, но столкнулся с проблемой нехватки памяти, боюсь, что и тут имеется такая ошибка.

@p_r_a_g_m_a · 14.04.2012, 00:42

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

int *queue;

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

Queue::Queue(int queue)
{
queue = new int[256];
tail=0;
head=0;
siz=0;
}

Первое что попалось на глаза

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

queue[head]=x;

x не инициализирована

@b0n4b · 14.04.2012, 00:56 **[ТС]**

Что-то поправил, вот что вышло.

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
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
#include<iostream>
#include<stdlib.h>
#include<string>
 
using namespace std;
 
class Queue
{
    int *queue;
    int tail, head, siz, x;
    int max;
 
public:
    Queue(int n);
    ~Queue();
    void push(int n);   // добавление элемента
    void clear();   // очистка очереди
    void pop(); // удаление 1го элемента   
    void size();  // количество элементов в очереди
    void front();  // узнать знач 1-ого элемента не удаляя его ????!!!!
};
 
Queue::Queue(int m)
{
    max=m;
    queue = new int[max];
    tail=0;
    head=0;
    siz=0; 
}
 
Queue::~Queue()
{
    delete[]queue;
}
 
void Queue::clear()
{
    head=tail;
    cout <<"ok\n";
}
 
void Queue::size()
{
    siz=0;
    for(int i=head; i<tail; i++)
    {
        siz++;
    }
    cout << siz << "\n";
}
 
void Queue::front() 
{
    if(head==tail)
    {
        cout << "error\n";
    }
    else
    {
        cout << queue[head] << "\n";
    }
}
 
void Queue::push(int n)
{
    queue[tail]=n;
    tail++;
    cout << "ok\n";
 
}
 
void Queue::pop()
{
    if(head==tail)
    {
        cout << "error\n";
    }
    else
    {
        cout << queue[head];
        queue[head]=x;
        head++;
        cout <<"\n";
    }
    
}
 
int main()
{
    Queue q(6);
    string s;
    for(;;)
    {
        cin>>s;
        if(s=="push")
        {
            int n;
            cin>>n;
            q.push(n);
        }
        else if(s=="pop")
        {
            q.pop();
        }
        else if (s=="clear")
        {
            q.clear();
        }
        else if (s=="size")
        {
            q.size();
        }
        else if (s=="front")
        {
            q.front();
        }
        else if (s=="exit")
        {
            cout <<"bye";
            return 1;
        }
    }
}

Всё компилируется, только не уверен правильно ли я сделал с памятью.

@p_r_a_g_m_a · 14.04.2012, 01:06

Да, с памятью все норм.

@b0n4b · 14.04.2012, 01:07 **[ТС]**

p_r_a_g_m_a, просто контестер в универе выдавал ошибку. Не хватало памяти, надеюсь что сейчас всё в порядке.

@p_r_a_g_m_a · 14.04.2012, 01:10

256 переменных целочисленных, не так и много)

@b0n4b · 14.04.2012, 01:14 **[ТС]**

Я правильно понимаю, что "max" это число элементов которые выделяются под память? Но не ясно чему оно равно.

@p_r_a_g_m_a · 14.04.2012, 01:17

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

Queue q(6);

6 целочисленных или 24 байта (для x86).

@b0n4b · 15.04.2012, 12:29 **[ТС]**

Является ли этот вариант более правильным?

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
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
#include<iostream>
#include<stdlib.h>
#include<string>
 
using namespace std;
 
class Queue
{
    int *queue;
    int tail, head, siz, siz2, x;
    int max;
 
public:
    Queue(int n);
    ~Queue();
    void push(int n);   // добавление элемента
    void clear();   // очистка очереди
    void pop(); // удаление 1го элемента   
    void size();  // количество элементов в очереди
    void front();  // узнать знач 1-ого элемента не удаляя его ????!!!!
};
 
Queue::Queue(int m)
{
    siz2=1;
    max=m;
    queue = new int[siz2];
    tail=0;
    head=0;
    siz=0; 
}
 
Queue::~Queue()
{
    delete[]queue;
}
 
void Queue::clear()
{
    head=tail;
    cout <<"ok\n";
}
 
void Queue::size()
{
    siz=0;
    for(int i=head; i<tail; i++)
    {
        siz++;
    }
    cout << siz << "\n";
}
 
void Queue::front() 
{
    if(head==tail)
    {
        cout << "error\n";
    }
    else
    {
        cout << queue[head] << "\n";
    }
}
 
void Queue::push(int n)
{
    if (siz2==tail)
    {
        int *new_queue=new int[siz2*2];
        for (int i=0; i<siz2; i++)
        {
            new_queue[i]=queue[i];
        }
        delete[] queue;
        queue=new_queue;
        siz2*=2;
    }
    queue[tail]=n;
    tail++;
    cout << "ok\n";
 
}
 
void Queue::pop()
{
    if(head==tail)
    {
        cout << "error\n";
    }
    else
    {
        cout << queue[head];
        queue[head]=x;
        head++;
        cout <<"\n";
    }
    
}
 
int main()
{
    Queue q(6);
    string s;
    for(;;)
    {
        cin>>s;
        if(s=="push")
        {
            int n;
            cin>>n;
            q.push(n);
        }
        else if(s=="pop")
        {
            q.pop();
        }
        else if (s=="clear")
        {
            q.clear();
        }
        else if (s=="size")
        {
            q.size();
        }
        else if (s=="front")
        {
            q.front();
        }
        else if (s=="exit")
        {
            cout <<"bye";
            return 1;
        }
    }
}

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@p_r_a_g_m_a 112 / 64 / 23 Регистрация: 08.03.2012 Сообщений: 211
	14.04.2012, 01:06
	Да, с памятью все норм. 1

@b0n4b 3 / 3 / 2 Регистрация: 16.11.2011 Сообщений: 100
	14.04.2012, 01:07 [ТС]
	p_r_a_g_m_a, просто контестер в универе выдавал ошибку. Не хватало памяти, надеюсь что сейчас всё в порядке. 0

@p_r_a_g_m_a 112 / 64 / 23 Регистрация: 08.03.2012 Сообщений: 211
	14.04.2012, 01:10
	256 переменных целочисленных, не так и много) 0

@b0n4b 3 / 3 / 2 Регистрация: 16.11.2011 Сообщений: 100
	14.04.2012, 01:14 [ТС]
	Я правильно понимаю, что "max" это число элементов которые выделяются под память? Но не ясно чему оно равно. 0