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

Собственная реализация стека. Критика - C++

Войти
Регистрация
Восстановить пароль
Другие темы раздела
C++ Посоветуйте компилятор для C++ http://www.cyberforum.ru/cpp-beginners/thread773926.html
Пользуюсь VS 2008 express,но с ним возникли проблемы при использовании чтении данных из файлов,может руки кривые и не знаю,как кодировку настроить правильно,если объясните,то буду благодарен,ну а так качаю eclipse,может с ним такой беды не будет,ну или посоветуйте чего ещё,у меня XP
C++ Ввод/вывод баг или нет? #include <iostream> #include <iterator> #include <string> using namespace std; ostream_iterator<string> oo(cout); istream_iterator<string> ii(cin); int main() { http://www.cyberforum.ru/cpp-beginners/thread773898.html
Кривое считывание с файла C++
#include <iostream> #include <fstream> #include <string> #include <locale.h> using namespace std; #include <windows.h> int main() { setlocale(LC_ALL,"Russian"); string str="Vremya-velikiy ychitel,no,yvi, ono ybivaet svoih ychenikov. Berlioz";
C++ Коменты
Может кто написать коменты на каждую строку? #include <iostream> #include <vector> using namespace std; void printy (int a,int b) { for (int i=0; i<b+1; i++) { cout << a << ' ';
C++ Подсчёт количества чисел в строке http://www.cyberforum.ru/cpp-beginners/thread773848.html
Подсчитать количество чисел в первой строке из текстового файла: код написала, всё хорошо считывается из текстового файла и выводится на экран, но увы с подсчётом чисел пока не получается реализовать алгоритм:( В текстовом файле находятся например: 1-я строка : 0,1 0,2 0,3 0,4 2-я строка: 0,5 0,6 0,7 0,8 0,9 #include <stdio.h> #include <stdlib.h> int main(int argc, char *argv){ FILE...
C++ code::blocks не находит .h файл Написал в code::blocks пример из книги Х.Дэйтел "как программировать на C++" при компиляции выдает: Fatal error: time1.h: No such file or dirrectory очевидно, что компилятор не знает пути к .h файлу. Где его прописать или что сделать чтобы программа скомпилировалась? Имеется вот что (все файлы в одном каталоге): // TIME1.H подробнее

Показать сообщение отдельно
MrGluck
Модератор
Эксперт CЭксперт С++
7210 / 4376 / 638
Регистрация: 29.11.2010
Сообщений: 11,887

Собственная реализация стека. Критика - C++

28.01.2013, 21:55. Просмотров 1182. Ответов 20
Метки (Все метки)

Покритикуйте пожалуйста реализацию. Самому мне не очень нравится момент с завершением работы программы в catch блоке, но не знаю как обойти возврат мусора в функции, возвращающей T&. Ведь данные могут быть и обработаны какой-нибудь другой функцией, принимающей результат работы в качестве аргумента. Но если этого нет, то, в принципе, программа остается дееспособной.

Stack.h
Кликните здесь для просмотра всего текста
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
#ifndef _STACK_H_
#define _STACK_H_
#include <cstddef>
#include <iterator>
#include <stdexcept>
#include <cstdlib>
 
template <class T>
class Stack
{
    public:
        Stack();
        template <class InputIterator>
        Stack(InputIterator beg, InputIterator end);
        ~Stack();
 
        bool empty() const;
        void push(const T& obj);
        void pop();
        std::size_t size() const;
        const T& top() const;
        T& top();
 
    private:
        Stack(const Stack &);             // prohibit copy
        Stack& operator= (const Stack &); // and assignment
        T& getTopData() const;
 
        struct Node
        {
            T data_;
            Node *next_;
            Node(const T &data, Node *nextNode);
            Node(const Stack::Node &);             // prohibit copy
            Node& operator= (const Stack::Node &); // and assignment
        } *Top;
        std::size_t counter_;
};
 
 
template <class T>
Stack<T>::Stack() : Top(nullptr), counter_(0) {}
 
template <class T>
template <class InputIterator>
Stack<T>::Stack(InputIterator beg, InputIterator end) : Stack()
{
    while(beg != end)
    {
        push(*beg);
        ++beg;
    }
}
 
template <class T>
Stack<T>::~Stack()
{
    while(Top)
        pop();
}
 
template <class T>
bool Stack<T>::empty() const
{
    return Top == nullptr;
}
 
template <class T>
void Stack<T>::push(const T& obj)
{
    Top = new Node(obj, Top);
    counter_++;
}
 
template <class T>
void Stack<T>::pop()
{
    if (!Top)
        return;
    Node *tmp = Top;
    Top = Top->next_;
    delete tmp;
    counter_--;
}
 
template <class T>
std::size_t Stack<T>::size() const
{
    return counter_;
}
 
template <class T>
const T& Stack<T>::top() const
{
    return getTopData();
}
 
template <class T>
T& Stack<T>::top()
{
    return getTopData();
}
 
template <class T>
T& Stack<T>::getTopData() const
{
    try
    {
        if (!Top) throw std::out_of_range("Trying to access to nothing");
        return Top->data_;
    }
    catch (const std::exception &e)
    {
        std::cerr << e.what() << std::endl;
        exit(1);
    }
}
 
template <class T>
Stack<T>::Node::Node(const T &data, Node *next) : data_(data), next_(next) {}
 
#endif


main.cpp (для теста)
Кликните здесь для просмотра всего текста
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <iostream>
#include "Stack.h"
 
class A
{
    public:
        A() { std::cout << "A()\n"; }
        A(const A &a) { std::cout << "A(A &)\n"; }
        ~A() { std::cout << "~A()\n"; }
};
 
int main()
{
    Stack<A> s;
    s.push(A());
    s.push(A());
 
    while(!s.empty())
    {
        s.pop();
    }
    s.top();
}
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
 
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru