Почему моя функция печати выбрасывает мусор, а текущая функция работает неправильно?

Question

вот заголовочный файл http://pastebin.com/g0z7LkeN вот файл реализации http://pastebin.com/USHbjbYC

и вот файл тестового отладчика (не должен его изменять)

// FILE: sequence_test.cpp
// An interactive test program for the new sequence class
#include <cctype>       // Provides toupper
#include <iostream>     // Provides cout and cin
#include <cstdlib>      // Provides EXIT_SUCCESS
#include "sequence2.h"  // With value_type defined as double
using namespace std;
using namespace CISP430_A2;

// PROTOTYPES for functions used by this test program:
void print_menu( );
// Postcondition: A menu of choices for this program has been written to cout.

char get_user_command( );
// Postcondition: The user has been prompted to enter a one character command.
// The next character has been read (skipping blanks and newline characters), 
// and this character has been returned.

void show_sequence(sequence display);
// Postcondition: The items on display have been printed to cout (one per line).

double get_number( );
// Postcondition: The user has been prompted to enter a real number. The
// number has been read, echoed to the screen, and returned by the function.


int main( )
{
    sequence test; // A sequence that we’ll perform tests on
    char choice;   // A command character entered by the user

    cout << "I have initialized an empty sequence of real numbers." << endl;

    do
    {
        print_menu( );
        choice = toupper(get_user_command( ));
        switch (choice)
        {
            case '!': test.start( );
                      break;
            case '+': test.advance( );
                      break;
            case '?': if (test.is_item( ))
                          cout << "There is an item." << endl;
                      else 
                          cout << "There is no current item." << endl;
                      break;
            case 'C': if (test.is_item( ))
                           cout << "Current item is: " << test.current( ) << endl;
                      else
                          cout << "There is no current item." << endl;
                      break;
            case 'P': show_sequence(test);
                      break;
            case 'S': cout << "Size is " << test.size( ) << '.' << endl;
                      break;
            case 'I': test.insert(get_number( ));
                      break;
            case 'A': test.attach(get_number( ));
                      break;
            case 'R': test.remove_current( );
                      cout << "The current item has been removed." << endl;
                      break;     
            case 'Q': cout << "Ridicule is the best test of truth." << endl;
                      break;
            default:  cout << choice << " is invalid." << endl;
        }
    }
    while ((choice != 'Q'));

    return EXIT_SUCCESS;
}

void print_menu( )
// Library facilities used: iostream.h
{
    cout << endl; // Print blank line before the menu
    cout << "The following choices are available: " << endl;
    cout << " !   Activate the start( ) function" << endl;
    cout << " +   Activate the advance( ) function" << endl;
    cout << " ?   Print the result from the is_item( ) function" << endl;
    cout << " C   Print the result from the current( ) function" << endl;
    cout << " P   Print a copy of the entire sequence" << endl;
    cout << " S   Print the result from the size( ) function" << endl;
    cout << " I   Insert a new number with the insert(...) function" << endl;
    cout << " A   Attach a new number with the attach(...) function" << endl;
    cout << " R   Activate the remove_current( ) function" << endl;
    cout << " Q   Quit this test program" << endl;
}

char get_user_command( )
// Library facilities used: iostream
{
    char command;

    cout << "Enter choice: ";
    cin >> command; // Input of characters skips blanks and newline character

    return command;
}

void show_sequence(sequence display)
// Library facilities used: iostream
{
    for (display.start( ); display.is_item( ); display.advance( ))
        cout << display.current( ) << endl;
}

double get_number( )
// Library facilities used: iostream
{
    double result;

    cout << "Please enter a real number for the sequence: ";
    cin  >> result;
    cout << result << " has been read." << endl;
    return result;
}

когда я нажимаю опцию P в отладчике, чтобы распечатать всю последовательность, она никогда не работает правильно и часто выплевывает мусор, а когда я нажимаю c в меню отладчика после ввода или добавления 2 чисел, кажется, что тоже не работает правильно. я проанализировал свои алгоритмы и не понимаю, почему они не работают правильно ...

Answer 1

Резюме:

Вы создаете экземпляр последовательности классов, используя конструктор по умолчанию, который не выделяет массив, в котором хранятся числа, это приведет к некорректному поведению при вызове любой из функций из меню, поскольку они зависят от массива быть правильно инициализированным. Вам нужно создать экземпляр теста, используя другой конструктор, а именно тот, который принимает аргумент размера.

Оригинальный текст

Посмотрев «файл тестового отладчика» выше, я заметил, что вы вызываете конструктор по умолчанию для последовательности классов:

sequence test; // A sequence that we’ll perform tests on

Глядя на реализацию, я не могу найти, что ее реализовали. Я думаю, что конструктор:

sequence(size_type entry=CAPACITY )

- это то, что вам нужно для того, чтобы «создать» новые данные массива и правильно установить индексы при запуске. Текущий код, например, при вызове show_sequence (test) считывает данные из неинициализированной случайной памяти.