Перекрашивание одинаковых чисел. Подскажите, где ошибка

@~~cr0n~~ · 18.10.2014, 22:26. **Показов** 340. **Ответов** 11

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

программа должна перекрасить одинаковые числа , но почему то не работает и не могу найти ошибку

C++

#include <conio.h>
#include <stdlib.h>
#include <iostream.h>
 
bool IsRepeatable(int * p, int i, int n)
{
        int j;
    for (int j = 0; j<n; ++j){
        if (p[i]==p[j])
        {
            return 1;
        }
    }
    return 0;
}
 
 
void main(int argc, char* argv[])
{
  int i,n;
  int * p;
  cout << "Masiva elementu daudzums N= ";
  cin >> i;
  p= new  int[i];
 
    for (n=0; n<i; n++)
    {
      cout << "Ievadiet vertibas : ";
      cin >> p[n];
    }
    cout << "Jusu ievadtie skaitli ir : ";
    for (n=0; n<i; n++)   {
    {
      if (IsRepeatable(p,i,n))
      {
        textcolor(LIGHTRED);
      }
     
      cout << p[n] << ", ";}
      textcolor(WHITE);
    }
     printf("\n");
     system("pause");
 
}

@John Prick · 18.10.2014, 22:30

Потому что путаница с аргументами функции IsRepeatable: передаёшь в неё число элементов массива, а используешь в функции как индекс. Там у тебя вообще выход за пределы массива получается.

@zss · 18.10.2014, 22:32

И еще

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

if (p[i]==p[j])

C++

1	if (p[i]==p[j] && i!=j)

@John Prick · 18.10.2014, 22:34

zss, у него в main i - это число элементов массива, а n - индекс, а в функции наоборот.

@zss · 18.10.2014, 22:37

Я потому и написал

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

И еще

Мораль - нечего отступать от общепринятых i - индекс массива, n - его размер!

Советую поменять их местами и переписать функцию main()

@~~cr0n~~ · 18.10.2014, 22:39 **[ТС]**

и как это выглядит если исправить ?

@John Prick · 18.10.2014, 22:44

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

и как это выглядит если исправить ?

В функции main просто замени i на n, а n на i.
И не забудь проверку, про которую zss писал.

@~~cr0n~~ · 18.10.2014, 22:51 **[ТС]**

Сообщение от John Prick

В функции main просто замени i на n, а n на i.
И не забудь проверку, про которую zss писал.

Изменил, но все осталось по прежнему

@John Prick · 18.10.2014, 22:52

cr0n, ну покажи ,как заменил.

@~~cr0n~~ · 18.10.2014, 22:56 **[ТС]**

Сообщение от John Prick

cr0n, ну покажи ,как заменил.

C++

#include <conio.h>
#include <stdlib.h>
#include <iostream.h>
 
 
 
bool IsRepeatable(int * p, int i, int n)
{
        int j;
    for (int j = 0; j<n; ++j){
        if (p[i]==p[j] && i!=j)
        {
            return 1;
        }
    }
    return 0;
}
 
 
void main(int argc, char* argv[])
{
  int i,n;
  int * p;
  cout << "Masiva elementu daudzums N= ";
  cin >> n;
  p= new  int[n];
 
    for (i=0; i<n; i++)
    {
      cout << "Ievadiet vertibas : ";
      cin >> p[i];
    }
    cout << "Jusu ievadtie skaitli ir : ";
    for (i=0; i<n; i++)   {
    {
      if (IsRepeatable(p,n,i))
      {
        textcolor(LIGHTRED);
      }
     
      cout << p[i] << ", ";}
      textcolor(WHITE);
    }
     printf("\n");
     system("pause");
 
}

@John Prick · 18.10.2014, 22:58

Пардон, я не уточнил.
Здесь менять не надо было:

C++

1	IsRepeatable(p,n,i)

Т.е. эту строчку оставить, как была:

C++

1	IsRepeatable(p,i,n)

@~~cr0n~~ · 18.10.2014, 23:24 **[ТС]**

Сообщение от John Prick

Пардон, я не уточнил.
Здесь менять не надо было:

C++

1	IsRepeatable(p,n,i)

Т.е. эту строчку оставить, как была:

C++

1	IsRepeatable(p,i,n)

хоть уже и многие ошибки помогли исправить все равно отказывает перекрашивать

@John Prick 2268 / 1761 / 739 Регистрация: 27.07.2012 Сообщений: 5,233
	18.10.2014, 22:30	2
	Потому что путаница с аргументами функции IsRepeatable: передаёшь в неё число элементов массива, а используешь в функции как индекс. Там у тебя вообще выход за пределы массива получается. 0

@John Prick 2268 / 1761 / 739 Регистрация: 27.07.2012 Сообщений: 5,233
	18.10.2014, 22:34	4
	zss, у него в main i - это число элементов массива, а n - индекс, а в функции наоборот. 0

@zss Модератор 13496 / 10751 / 6406 Регистрация: 18.12.2011 Сообщений: 28,687
	18.10.2014, 22:37	5
	Я потому и написал Сообщение от zss И еще Мораль - нечего отступать от общепринятых i - индекс массива, n - его размер! Советую поменять их местами и переписать функцию main() 0

@~~cr0n~~ Заблокирован
	18.10.2014, 22:39 [ТС]	6
	и как это выглядит если исправить ? 0

@John Prick 2268 / 1761 / 739 Регистрация: 27.07.2012 Сообщений: 5,233
	18.10.2014, 22:44	7
	Сообщение от cr0n и как это выглядит если исправить ? В функции main просто замени i на n, а n на i. И не забудь проверку, про которую zss писал. 0

@~~cr0n~~ Заблокирован
	18.10.2014, 22:51 [ТС]	8
	Сообщение от John Prick В функции main просто замени i на n, а n на i. И не забудь проверку, про которую zss писал. Изменил, но все осталось по прежнему 0

@John Prick 2268 / 1761 / 739 Регистрация: 27.07.2012 Сообщений: 5,233
	18.10.2014, 22:52	9
	cr0n, ну покажи ,как заменил. 0

	18.10.2014, 23:24