Функция меню - ошибка в коде

@peva · Регистрация: 12.02.2017

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

Здравствуйте, уважаемое сообщество. Пожалуйста, укажите, в чем моя ошибка.

Задание следующее. Нужно создать 4 функции - заполнение массива случайными числами, вывод массива на экран, поиск минимума и меню выбора из этих трех вариантов.

С первыми тремя функциями справилась. Вот код - работает:

C++

#include <iostream>
#include <time.h>
 
using namespace std;
 
void createMas(int masI[], int length);
void masOut(int masI[], int length);
void masMin(int masI[], int length);
 
 
void main()
{
    setlocale(LC_ALL, "rus");
    srand(time(NULL));
 
    const int razmer = 15;
    int arr[razmer] = {};
    cout << "\n\n";
 
    createMas(arr, razmer);
    masOut(arr, razmer);
    masMin(arr, razmer);
}
 
void createMas(int mas[], int length)
{
    for (size_t i = 0; i < length; i++)
    {
        mas[i] = -50 + rand() % 101;
    }
}
 
void masOut(int mas[], int length)
{
    for (size_t i = 0; i < length; i++)
    {
        cout << "\t[" << i << "] " << mas[i] << "\n";
    }
    cout << "\n";
}
 
void masMin(int mas[], int length)
{
    int min = mas[0];
    for (size_t i = 0; i < length; i++)
    {
        if (mas[i] < min)
        {
            min = mas[i];
        }
    }
    cout << "min = " << min << endl;
}

Добавляю меню - и просто пустая консоль. Дальше привожу код. В чем моя ошибка?

C++

#include <iostream>
#include <time.h>
 
using namespace std;
 
void menu(int masI[], int length);
 
void createMas(int mas[], int length);
void masOut(int mas[], int length);
void masMin(int mas[], int length);
 
 
void main()
{
    setlocale(LC_ALL, "rus");
    srand(time(NULL));
 
    const int razmer = 15;
    int arr[razmer] = {};
 
    void menu(int arr, int length);
 
    cout << "\n";
}
 
void createMas(int mas[], int length)
{
    for (size_t i = 0; i < length; i++)
    {
        mas[i] = -20 + rand() % 45;
    }
}
 
void masOut(int mas[], int length)
{
    for (size_t i = 0; i < length; i++)
    {
        cout << "\t[" << i << "] " << mas[i] << "\n";
    }
    cout << "\n";
}
 
void masMin(int mas[], int length)
{
    int min = mas[0];
    for (size_t i = 0; i < length; i++)
    {
        if (mas[i] < min)
        {
            min = mas[i];
        }
    }
    cout << "min = " << min << endl;
}
 
void menu(int mas[], int length)
{
    int choice;
    
    cout << "\t1 - Создать целочисленный массив.\n";
    cout << "\t2 - Вывести целочисленный массив на экран.\n";
    cout << "\t3 - Найти минимальный элемент целочисленного массива.\n";
    cout << "\t0 - Выход из программы.\n\n";
 
    int f = 0;
    do
    {
        do
        {
            cout << "Сделайте выбор:\n";
            cin >> choice;
        } while (choice < 0 || choice > 3);
 
        switch (choice)
        {
        case 1:
            f = 1;
            createMas(mas, length);//создание
            break;
 
        case 2:
            if (f == 1)
            {
                masOut(mas, length);//вывод на экран
                cout << "\n\n";
            }
            else
            {
                cout << "Массив не заполнен!\n";
            }
            break;
 
        case 3:
            if (f == 1)
            {
                masMin(mas, length);//поиск минимума
            }
            else
            {
                cout << "Массив не заполнен!\n";
            }
            break;
        }
    } while (choice != 0);
}

nd2 · 07.04.2017, 23:45

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

C++

1	void menu(int arr, int length);

Это не вызов функции.

@peva · 07.04.2017, 23:54 **[ТС]**

nd2, спасибо Вам огромное!

@anapshy · 08.04.2017, 00:25

C++

int main(void)
{
    setlocale(LC_ALL, "rus");
    srand(time(NULL));
 
    const int razmer = 15;
    int arr[razmer] = {};
 
    menu( arr, length);
 
    cout << "\n";
    return 0;
}

Если массив нигде не изменяется, то лучше передавать его везде по ссылке.

C++

#include <iostream>
#include <time.h>
 
using namespace std;
 
void menu(int *(&mas), int length);
 
void createMas(int *(&mas), int length);
void masOut(int *(&mas), int length);
void masMin(int *(&mas), int length);
 
 
int main(void)
{
    setlocale(LC_ALL, "rus");
    srand(time(NULL));
 
    const int razmer = 15;
    int *arr = new int[razmer];
 
    menu( arr, razmer);
 
    cout << "\n";
    return 0;
}
 
void createMas(int *(&mas), int length)
{
    for (size_t i = 0; i < length; i++)
    {
        mas[i] = -20 + rand() % 45;
    }
}
 
void masOut(int *(&mas), int length)
{
    for (size_t i = 0; i < length; i++)
    {
        cout << "\t[" << i << "] " << mas[i] << "\n";
    }
    cout << "\n";
}
 
void masMin(int *(&mas), int length)
{
    int min = mas[0];
    for (size_t i = 0; i < length; i++)
    {
        if (mas[i] < min)
        {
            min = mas[i];
        }
    }
    cout << "min = " << min << endl;
}
 
void menu(int *(&mas), int length)
{
    int choice;
 
    cout << "\t1 - Создать целочисленный массив.\n";
    cout << "\t2 - Вывести целочисленный массив на экран.\n";
    cout << "\t3 - Найти минимальный элемент целочисленного массива.\n";
    cout << "\t0 - Выход из программы.\n\n";
 
    int f = 0;
    do
    {
        do
        {
            cout << "Сделайте выбор:\n";
            cin >> choice;
        } while (choice < 0 || choice > 3);
 
        switch (choice)
        {
        case 1:
            f = 1;
            createMas(mas, length);//создание
            break;
 
        case 2:
            if (f == 1)
            {
                masOut(mas, length);//вывод на экран
                cout << "\n\n";
            }
            else
            {
                cout << "Массив не заполнен!\n";
            }
            break;
 
        case 3:
            if (f == 1)
            {
                masMin(mas, length);//поиск минимума
            }
            else
            {
                cout << "Массив не заполнен!\n";
            }
            break;
        }
    } while (choice != 0);
}

nd2 · 08.04.2017, 00:48

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

Если массив нигде не изменяется, то лучше передавать его везде по ссылке.

Тут указатель на начало массива передаётся. И передавать по ссылке его нужно, как раз, тогда, когда указатель в функциях изменяется.

@anapshy · 08.04.2017, 00:53

nd2, а разве не создается каждый раз как-бы копия этого массива (если без ссылки)?! Или я заблуждаюсь?
А ну хотя после завершения функции всё исчезнет. Все верно?

nd2 · 08.04.2017, 01:10

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

а разве не создается каждый раз как-бы копия этого массива (если без ссылки)?! Или я заблуждаюсь?

Нет, массивы при передаче в функции не копируются. Передаётся или указатель на начало массива, или ссылка на массив, но это тогда так (размер массива должен быть указан, и только массив указанного размера может передаваться):

C++

1	void menu(int (&masI)[15]);

, или указатель на массив (размер массива должен быть указан, и только массив указанного размера может передаваться):

C++

1	void menu(int (*masI)[15]);

@anapshy · 08.04.2017, 01:21

nd2, а если:

C++

1	void menu(int masI[]);

nd2 · 08.04.2017, 01:29

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

а если:

Это равнозначно (просто по-другому написано):

C++

1	void menu(int* masI);

Передаётся указатель на начало массива.

@peva 0 / 0 / 1 Регистрация: 12.02.2017 Сообщений: 55
	07.04.2017, 23:54 [ТС]	3
	nd2, спасибо Вам огромное! 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	08.04.2017, 00:48	5
	Сообщение от anapshy Если массив нигде не изменяется, то лучше передавать его везде по ссылке. Тут указатель на начало массива передаётся. И передавать по ссылке его нужно, как раз, тогда, когда указатель в функциях изменяется. 1

@anapshy 243 / 268 / 219 Регистрация: 14.11.2016 Сообщений: 1,043
	08.04.2017, 00:53	6
	nd2, а разве не создается каждый раз как-бы копия этого массива (если без ссылки)?! Или я заблуждаюсь? А ну хотя после завершения функции всё исчезнет. Все верно? 1

	08.04.2017, 01:29

Функция меню - ошибка в коде

Решение