Правильная ли программа? С++

[Вероника99]

Не уверена в правильности соответствия задания и кода. Обычно ведь очередь через структуру делается. Задание следующее:
Создать класс - очередь. Определить конструктор, деструктор, функции для ввода элемента в очередь и изъятия элемента из очереди, вычисление количества элементов в очереди, суммы и произведения элементов.
Код:
main.cpp

#include "stdafx.h"
#include "queue.h"
#include <iostream>
#include <conio.h>
using namespace std;
 
int main()
{
    int n;
    queue a; //object
    cout<<"Enter numbers:";
    for(int i=0;i<100;i++)
    { 
        cin>>n; 
        if(n==0)
            break;
        a.qput(n);  
    }
    a.print();
    return 0;
}

queue.h

#include "stdafx.h"
#ifndef QUEUE_H
#define QUEUE_H
 
class queue
{
public:
    queue();
    ~queue();
    void qput( int m );
    void qget();
    void print();
private:
    int a[100];
    int start,end;
};
#endif

queue.cpp

#include "stdafx.h"
#include "queue.h"
#include <iostream>
using namespace std;
 int k;
 int sum=0,dob=1;
 
queue::queue()
{
    start=0;
    end=0;
}
 
void queue::qget()
{
 
    if ( start == end )
    {
        cout<<"Queue is empty!";
        return;
    }
    cout<<"Element "<<a[end]<<"\n";
    end++;
    
}
void queue::qput( int m )
{
    if ( 100 == start )
    {
        cout<<"Queue is full!";
        return;
    }
    a[start] = m;
 
    start++;
    k=start;
    sum+=m;
    dob*=m;
}
 
void queue::print()
{
    for(int i=0;i<k;i++)
        cout<<" "<<a[i];
}
 queue::~queue()
{
    cout<<"\nGeneral count "<<k<<"\nSum="<<sum<<"\nProizved"<<dob<<endl;
}

[p51x]

Вы забыли указать: от нас то чего хотите? или просто хвастаетесь?

[challengerr]

В данном классе количество элементов фиксированное, поэтому вычисление количества элементов не требуется.
Отсутствуют функции подсчета суммы и произведения элементов
Синтаксические ошибки не смотрел

[Smogg]

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

В данном классе количество элементов фиксированное, поэтому вычисление количества элементов не требуется.

Отчего же? Разве count = start - end; не похоже на вычисление?

А по делу:
а) если end == 80 и start == 100, то очередь не очень то фулл.
б) Вычислять суммы надо, как указали, отдельными функциями. К тому же сейчас у вас, при вынимании элемента, переменные произведений и суммы не уменьшаются (изменяются).

[Вероника99]

Я не хвастаюсь)Просто я нашла в интернете образец очереди и немного переделала его под задание,дело в том,что никак не могу разобраться с очередями,поэтому спросила правильно ли здесь написано,т.к раньше в основном сталкивалась с очередями через структуры.Спасибо за ответы,я сделаю отдельные функции для вычисления произведения и суммы)

[Вероника99]

Мне кажется,что здесь вышло обычное заполнение массива числами,а не очередь

[BDA]

Сам код мне не нравится (это не реализация очереди на кольцевом массиве, а какая-то заготовка). Почему Вы считаете, что это не очередь? Объект ведет себя как очередь, а уж как он там внутри реализован, не должно никого волновать. Другое дело, удовлетворяет ли такая реализация очереди условию. Вам нужно реализовать очередь "бесконечного" размера или фиксированного?

[Вероника99]

BDA, фиксированного размера. Меня сбило с толку,то что здесь нет структуры. Я доделала в соответствии с условием. Вот что получилось:

#include "stdafx.h"
#include "queue.h"
#include <iostream>
using namespace std;
 int k;
 int sum=0,dob=1;

queue::queue()
{
    start=0;
    end=0;
}
 
void queue::qget()
{

    if ( start == end )
    {
        cout<<"Queue is empty!";
        return;
    }
   // cout<<"Element "<<a[end]<<"\n";
    end++;
	if(start==100)
		start=0;
	
}
 
void queue::qput( int m )
{
    if ( 100 == start )
    {
        cout<<"Queue is full!";
        return;
    }

    a[start] = m;

    start++;
	k=start;
	
}

void queue::suma()
{
	for(int i=0;i<k;i++)
	{
		sum+=a[i];
		dob*=a[i];
	}
}

void queue::size()
{
	for(int i=start;i<end;i++)
		k++;

}
void queue::print()
{
	for(int i=0;i<k;i++)
		cout<<" "<<a[i];
	
}
 queue::~queue()
{
	cout<<"\nGeneral count "<<k<<"\nSum="<<sum<<"\nProizvedenie "<<dob<<endl;
}

[BDA]

#include <iostream>
using namespace std;

enum { SIZE = 10 };

class queue
{
public:
    queue() : start(0), end(0), sum(0), pr(1) {}
    ~queue() {}
    int qput(int m);
    int qget();
    void print(ostream &f);
    int qsum();
    int qpr();
    int qcount();
private:
    int a[SIZE];
    int start, end;
    int sum;
    int pr;
};
 
int queue::qget()
{
    if (start == end)
        return 0;
    int tmp = start;
    start = (start + 1) % SIZE;
    sum -= a[tmp];
    pr /= a[tmp];
    return a[tmp];
}
 
int queue::qput(int m)
{
    if ((end + 1) % SIZE == start)
        return 1;
    a[end] = m;
    sum += m;
    pr *= m;
    end = (end + 1) % SIZE;
    return 0;
}

void queue::print(ostream &f)
{
    for (int i = start; i < end; i = (i + 1) % SIZE)
        f << a[i] << " ";
    f << endl;
}

int queue::qsum()
{
    return sum;
}

int queue::qpr()
{
    if (qcount())
        return pr;
    else
        return 0;
}

int queue::qcount()
{
    return (end + SIZE - start) % SIZE;
}

int
main()
{
    int n;
    queue a;
    cout << "Enter numbers:";
    for(int i = 0; i < 15; ++i) {
        cin >> n; 
        if(n == 0)
            break;
        if (a.qput(n))
            cout << "queue full" << endl;        
    }
    a.print(cout);
    cout << "Sum: " << a.qsum() << " Product: " << a.qpr() << " Count: " << a.qcount() << endl;
    for(int i = 0; i < 15; ++i) {
        n = a.qget();
        if (!n)
            cout << "queue empty" << endl;
        else
            cout << "Element: " <<  n << " Sum: " << a.qsum() << " Product: " << a.qpr() << " Count: " << a.qcount() << endl;
    }
}

В очереди ни в коем случае не должно оказаться нулей, иначе очередь "сломается".

[Вероника99]

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

В очереди ни в коем случае не должно оказаться нулей, иначе очередь "сломается".

Смысле нулей не должно оказаться?При удалении элементов нельзя менять на 0 или что Вы имеете в виду?