Проблема при работе с массивом

[kAND]

Приветствую всех. Вобщем ситуация следующая, переписывая лабу с паскаля на си, столкнулся с кое-какой проблемкой.
Вот написанный мной код:

#include <stdio.h>
#include <conio.h>
void main()
{
int a[14];
int n=0,s=0,i;

clrscr();
printf("Enter elements of the array\n");
for (i=0;i<=14;i++)
  {
  scanf("%d",&a[i]);
  if (a[i]>a[s]) s=i;
  else
    if (a[i]==a[s]) n++;
  }
printf("There are %d max values.\nMax value is: %d",n,a[s]);
getch();
}

Код этой же программы на паскале:

program L6_2;
uses crt;
var n,s,i:integer;
    a:array[1..15] of integer;
begin
  clrscr;
  s:=1;
  n:=0;
  writeln('Enter array elements');
  for i:=1 to 15 do
    begin
      readln(a[i]);
      if a[i]>a[s] then s:=i
      else
        if a[i]=a[s] then n:=n+1;
    end;
  writeln('There are ',n,' max values.',#10,#13,'Max value is: ',a[s]);
  readln;
end.

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

Проблема заключается в следующем: код на паскале работает как часики, а вот на си... на последнем шаге работы цикла в переменную n невообразимым способом попадает некоторое число равное приблизительно a[i]

Программу тестировал следующими входными данными:
1 2 3 4 5 6 7 8 9 1 2 3 4 5 6
по идее максимальное число 9, их кол-во - 1, а программа пишет, что их 6

Уже довольно долго сижу и не могу понять в чем дело...

[Sazary]

по идее максимальное число 9, их кол-во - 1, а программа пишет, что их 6

хм.. А у меня все работает..

[kAND]

Хм... интересно. Дабы не быть голословным добавил аттачи.

P.S. Аналогичная программа без ввода элементов, работает нормально.
Вот текст:

#include <stdio.h>
#include <conio.h>
#include <math.h>
void main()
{
static int a[]={1,2,3,4,5,6,7,8,9,1,2,3,4,5,6};
int n=0,s=1,i;

clrscr();
for (i=0;i<=14;i++)
  {
  if (a[i]>a[s]) s=i;
  else  if (a[i]==a[s]) n++;
  }
printf("There are %d max values.\nMax value is: %d",n,a[s]);
getch();
}

[Sazary]

Попробуйте так:

#include <stdio.h>
#include <conio.h>
int main()
{
int a[14];
int n=0,s=0,i;

printf("Enter elements of the array\n");
for (i=0;i<=14;i++)
  {
  scanf("%d",&a[i]);
  if (a[i]>a[s])
   {
    n=1;  // <-- Если нашли новый максимум, то он пока один
    s=i;
   }
  else
    if (a[i]==a[s]) n++;
  }
printf("There are %d max values.\nMax value is: %d",n,a[s]);
getch();
}

Кстати, присваивать n единицу при нахождении нового максимума по-любому надо.
Иначе будут ошибки при подобном вводе:
1 2 3 4 4 4 4 5 6 7 8 8 8 8 9 9

[kAND]

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

Уберите амперсант в scanf

Хм... думаю с ним было лучше

2 Sazary, ответ выдает не верный n=1, max=7. Но за поправку в алгоритме спасибо, не учел

[Sazary]

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

ответ выдает не верный n=1, max=7

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

[kAND]

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

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

попробовал другим компилятором, результат тот же Попробую на другом компе, может дело в нем...
------------
нет, проблема не в компе
------------
Спасибо всем за помощь, буду разбираться в чем проблема, может найду причину. Если найду, отпишусь

[Sazary]

kAND Я компилю в MinGW.

Специально проверил в borland C++ 3.1.
Для
1 2 3 4 5 6 7 8 9 1 2 3 4 5 6
говорит, что максимум =7 (а количество 1).
Этого НЕ должно происходить и я не знаю почему так получается.

А вот вообще бред:
1 2 3 3 4 5 6 6 6 9 7 6 5 4 3
There are 2 max values.
Max value is: 3
=========

#include <stdio.h>
#include <conio.h>
int main()
{
clrscr();
int a[14];
int n=0,s=0,i;

printf("Enter elements of the array\n");
for (i=0;i<=14;i++)
  {
  scanf("%d",&a[i]);
  if (a[i]>a[s])
   {
  //  printf("a[%d]>a[%d]! a[%d]=%d, a[%d]=%d\n",i,s,i,a[i],s,a[s]);
  //  printf("n=%d, s=%d\n",n,s);
    n=1;  // <-- Если нашли новый максимум, то он пока один
    s=i;
   }
  else
    if (a[i]==a[s]) n++;
  }
printf("s= %d\n",s);
printf("There are %d max values.\ns=%d\nMax value is: %d",n,s,a[s]);
getch();
return 0;
}

Как видно из результата, индекс максимума он все-таки считает правильно.
Но, почему-то, выводить элемент массива с этим индексом не хочет.
==========
поправка:
все-таки проблема не только в этом.

[MaTBeu]

Сделайте пошаговую отладку. Выясните на каком этапе у вас ошибка.
----------------------------------------------------------
Мда... ошибка коварная... у вас цикл от 0 до 14(включительно), тоесть 15 итераций, а элементов всего 14

#include <stdio.h>
#include <conio.h>
int main()
{
clrscr();
int a[14];
int n=0,s=0,i;

printf("Enter elements of the array\n");
for (i=0;i<14;i++) <-----------вот тут ошибка (была)
  {
  scanf("%d",&a[i]);
  if (a[i]>a[s])
   {
  //  printf("a[%d]>a[%d]! a[%d]=%d, a[%d]=%d\n",i,s,i,a[i],s,a[s]);
  //  printf("n=%d, s=%d\n",n,s);
    n=1;  // <-- Если нашли новый максимум, то он пока один
    s=i;
   }
  else
    if (a[i]==a[s]) n++;
  }
printf("s= %d\n",s);
printf("There are %d max values.\ns=%d\nMax value is: %d",n,s,a[s]);
getch();
return 0;
}

[Sazary]

MaTBeu Да, действительно, ошибка коварная )

Кстати, хотелось бы заметить, что в Dev-Cpp работает даже в таком виде.(даже если максимальный элемент будет 14-м)