Выход за пределы массива, устранение

@EwgenYYY · Регистрация: 28.03.2018

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

Всем, привет!

Дано задание: Определить последовательность элементов массива, возрастает, убывает или не возрастает и не убывает

Сама программа работает, однако сравнение элементов выходит за пределы массива, объясните почему так?

ПС: скрин прилагается, в желтой рамке самой программы, с каким числом сравнивается заданный элемент, а в желтой рамке в самом коде (i = 0), мне говорят, что там должна быть единица.

C++

#include "stdafx.h" 
#include <iostream> 
#include <conio.h> 
#include <stdio.h> 
#include <cstdlib> 
 
using namespace std;
 
 
int main()
{
    setlocale(LC_ALL, "RUS");
 
    int n, i, *a, k_voz = 0, k_ub = 1;
 
    cout << "Введите количество элементов последовательности: ";
    cin >> n;
 
    a = new int[n];
    cout << "Введите элементы последовательности\n";
    for (i = 0; i < n; i++)
    {
        cin >> a[i];
                cout << "a[" << i - 1 << "]=" << a[i - 1] << ", a[" << i << "]=" << a[i] << "\n";
        if (a[i - 1] < a[i])
        {
            k_voz = k_voz + 1;
        }
        if (a[i - 1] > a[i])
        {
            k_ub = k_ub + 1;
        }
    }
    cout << "Заданная последовательность:\n";
    for (i = 0; i < n; i++)
    {
        cout << a[i] << " ";
    }
    if (k_voz == n)
    {
        cout << "\nПоследовательность возрастает\n";
    }
    else
    {
        if (k_ub == n)
            cout << "\nПоследовательность убывает\n";
        else
        {
            cout << "\nПоследовательность не возрастает и не убывает\n";
        }
    }
    system("pause");
    return 0;
}

@SomniPhobia · 27.01.2019, 11:35

В массиве нет элемента с индексом -1.

@EwgenYYY · 27.01.2019, 11:43 **[ТС]**

Всем, привет!

Дано задание: Определить последовательность элементов массива, убывает, возрастает или не убывает и не возрастает

Сам код работает как часы, однако первый элемент сравнивается с отрицательным числом, тем самым выходит за пределы массива, объясните почему так и как это исправить

Пс: скрин прилагается, в желтой рамке самого кода число с которым сравнивается первый заданный элемент массива, в самом коде в желтой рамке ( i = 0 ) мне говорят там должна быть единица.

C++

#include "stdafx.h" 
#include <iostream> 
#include <conio.h> 
#include <stdio.h> 
#include <cstdlib> 
 
using namespace std;
 
 
int main()
{
    setlocale(LC_ALL, "RUS");
 
    int n, i, *a, k_voz = 0, k_ub = 1;
 
    cout << "Введите количество элементов последовательности: ";
    cin >> n;
 
    a = new int[n];
    cout << "Введите элементы последовательности\n";
    for (i = 0; i < n; i++)
    {
        cin >> a[i];
                cout << "a[" << i - 1 << "]=" << a[i - 1] << ", a[" << i << "]=" << a[i] << "\n";
        if (a[i - 1] < a[i])
        {
            k_voz = k_voz + 1;
        }
        if (a[i - 1] > a[i])
        {
            k_ub = k_ub + 1;
        }
    }
    cout << "Заданная последовательность:\n";
    for (i = 0; i < n; i++)
    {
        cout << a[i] << " ";
    }
    if (k_voz == n)
    {
        cout << "\nПоследовательность возрастает\n";
    }
    else
    {
        if (k_ub == n)
            cout << "\nПоследовательность убывает\n";
        else
        {
            cout << "\nПоследовательность не возрастает и не убывает\n";
        }
    }
    system("pause");
    return 0;
}

Заранее благодарен!

@SomniPhobia · 27.01.2019, 11:48

EwgenYYY, вот моя версия решения.

C++

#include <iostream>
#include <windows.h>
 
using namespace std;
 
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    system("color 0A");
 
    cout << "Укажите длину последовательности ";
    size_t n;
    cin >> n;
    auto a = new int[n];
    cout << "Введите значения элементов" << endl;
    for (size_t u = 0u; u < n; ++u)
    {
        cout << "a[" << u << "] = ";
        cin >> a[u];
    }
    auto up = true;
    auto down = true;
    for (size_t u = 1u; u < n; ++u)
    {
        up = a[u - 1] < a[u] && up;
        down = a[u - 1] > a[u] && down;
    }
    cout << "Последовательность является ";
    cout << (
        up ? "возрастающей" :
        down ? "убывающей" :
        "не возрастающей и не убывающей"
        ) << endl;
 
    system("pause");
    return 0;
}

@Yetty · 27.01.2019, 11:54

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

мне говорят, что там должна быть единица

если поставите 1 не будет вводиться первый элемент или перед циклом сin >>a[0]; ставить. но лучше так:

C++

1
2
3

for (i = 0; i < n-1; i++)
if (a[i] < a[i+1])...
else ...

второй if не нужен применяйте else. не забывайте что ещё случай равенства. ставить строгое неравенство или нестрогое зависит от того, какую последовательность считать возрастающей (т.е. когда следующий больше предыдущего или больше или равен предыдущему)
k_voz = k_voz + 1; лучше так: up++;
и строку 24 уберите - не делайте вывод после cin

@EwgenYYY · 27.01.2019, 11:56 **[ТС]**

А число отрицательное разве может быть, где a [i] = 0

И там указанная последовательность из 4 элементов, а я ввел 3

@EwgenYYY · 27.01.2019, 12:01 **[ТС]**

24 строка нужна, т.к без нее преподаватель не примет работу (она сама ввела эту строку)

@Yetty · 27.01.2019, 12:16

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

И там указанная последовательность из 4 элементов, а я ввел 3

Вы прочитали пост ? чтобы не путаться сделайте ввод и сравнение в разных циклах

насчёт неравенства только определитесь - строгое или не строгое

Добавлено через 4 минуты

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

без нее преподаватель не примет работу

нельзя вывести 2 элемента если введён один. и не делается вывод сразу после cin. если хотите что-то вывести на экран - делайте отдельный цикл (или организуйте ввод случайных значений)

EwgenYYY, напишите Ваше определение возрастающей последовательности

@ReDoX · 27.01.2019, 12:18

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

i = 0;

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

a[i - 1]

К какому элементу получаете доступ?

@EwgenYYY · 27.01.2019, 12:36 **[ТС]**

Последовательность возрастает, когда следующий элемент больше предыдущего элемента или больше или равен предыдущему

@Yetty · 27.01.2019, 13:06

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

Последовательность возрастает, когда следующий элемент больше предыдущего элемента или больше или равен предыдущему

C++

#include <iostream>
using namespace std;
 
int main()
{
    int n, up=0, down=0;
    cout <<"n="; cin >>n;
 
    double*a = new double[n];
 
    cout <<"Enter "<<n<<" elements:\n";
    for (int i = 0; i < n; i++)
      cin >>a[i];      
 
    for (int i = 0; i < n-1; i++)
      {
      if (a[i]<=a[i+1]) up++;
      if (a[i]>=a[i+1]) down++;
      }
    
    if (up==n-1 && down!=n-1) cout <<"UP\n";
    else if (down==n-1 && up!=n-1) cout <<"DOWN\n";
    else cout <<"No UP, No DOWN\n";
 
    delete[]a;
system("pause");
return 0;
}

@EwgenYYY 0 / 0 / 1 Регистрация: 28.03.2018 Сообщений: 48
	27.01.2019, 12:36 [ТС]	10
	Последовательность возрастает, когда следующий элемент больше предыдущего элемента или больше или равен предыдущему 0

@SomniPhobia 601 / 438 / 136 Регистрация: 22.11.2017 Сообщений: 1,379
	27.01.2019, 11:35	2
	В массиве нет элемента с индексом -1. Миниатюры 0

@EwgenYYY 0 / 0 / 1 Регистрация: 28.03.2018 Сообщений: 48
	27.01.2019, 11:56 [ТС]	6
	А число отрицательное разве может быть, где a [i] = 0 И там указанная последовательность из 4 элементов, а я ввел 3 Миниатюры 0

@EwgenYYY 0 / 0 / 1 Регистрация: 28.03.2018 Сообщений: 48
	27.01.2019, 12:01 [ТС]	7
	24 строка нужна, т.к без нее преподаватель не примет работу (она сама ввела эту строку) 0

@Yetty 7437 / 5029 / 2892 Регистрация: 18.12.2017 Сообщений: 15,692
	27.01.2019, 12:16	8
	Сообщение от EwgenYYY И там указанная последовательность из 4 элементов, а я ввел 3 Вы прочитали пост ? чтобы не путаться сделайте ввод и сравнение в разных циклах насчёт неравенства только определитесь - строгое или не строгое Добавлено через 4 минуты Сообщение от EwgenYYY без нее преподаватель не примет работу нельзя вывести 2 элемента если введён один. и не делается вывод сразу после cin. если хотите что-то вывести на экран - делайте отдельный цикл (или организуйте ввод случайных значений) EwgenYYY, напишите Ваше определение возрастающей последовательности 0

@ReDoX 447 / 333 / 172 Регистрация: 01.07.2015 Сообщений: 1,162
	27.01.2019, 12:18	9
	Сообщение от EwgenYYY i = 0; Сообщение от EwgenYYY a[i - 1] К какому элементу получаете доступ? 0

	27.01.2019, 13:06