Найти номер первого экстремального числа из набора.

[2107]

Добрый день! Помогите разобраться в задаче. Минимальный элемент и его позицию в списке находит нормально, а максимальный не ищет

Дано целое число N и набор из N целых чисел. Найти номер первого экстремального (то есть минимального или максимального) элемента из данного набора.

#include <windows.h>
#include <iostream>
#include <stdlib.h>
#include <fstream>
using namespace std;

 
int main ()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);

    int a;            
    int max,min;      
    int pos, pos2;     
    int i=0;         

    ifstream infile ("file.txt");     
    
    max=min = a;
    
    cout << "Набор чисел: ";
    
    while ( infile >> a )
    {     
          i++;
          
          cout << a << " ";
          
          if ( a > max )
          {
             max=a;
             pos=i;
          }
          
          if (a < min)
          {
             min = a;
             pos2=i;
          }
    }
    
    infile.close();
    
    cout << endl << "Номер первого экстремального элемента: ";
    if (pos < pos2)
    {
          cout << pos << endl;
    }
    else
    {
          cout << pos2 << endl;
    }
    
    
    system ("pause");
    return 0;
}

[ROD]

    int a;               
    max=min = a;

Подсказать чему равно "a" (а так же "min" и "max")?

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

Всегда инициализируйте переменные перед использованием. В задаче поиска минимального или максимального элемента можно приравнять "a" первому (а в принципе вообще любому) элементу из списка, но не приравнивайте его нулю - в списке может не оказаться элементов больше (меньше) нуля и вы получите в качестве ответа этот ноль

p.s.

К стати, system("pause") - тоже не очень хорошая практика (ИМХО).

[2107]

Написал опять вот так:

infile >> a;
max = min = a;

и заработало все, а раньше не работало почему-то ))

спасибо, ROD. И еще такой вопрос, когда считываешь число из массива, предыдущее число будет "a[i - 1]", а след. "a[i + 1]". Есть ли подобный способ записывать пред. и след. числа, когда считываешь не из массива, а из файла?

[BDA]

А Вы ради интереса закомментируйте infile >> a; и выведите значение a.
Самым простым способом будет считывание в буфер 3 чисел и работа с ними, а затем дочитывание из файла и замена значений в буфере.

[2107]

BDA, спасибо, а пример как считывать не приведете?

[BDA]

Без учета того, что в файле может не быть 3 чисел.

int a[3];
infile >> a[0] >> a[1];
while (infile >> a[2]) {
  //тут нужные действия

  //сдвигаем элементы в буфере, выталкивая последний
  a[0] = a[1];
  a[1] = a[2];
}

[2107]

Препод запретил массивы использовать.
Как вот эту задачу можно решить без использования массивов?

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

int main()
{
  SetConsoleCP(1251);
  SetConsoleOutputCP(1251);   
  
  ifstream infile("FILE.txt"); 
        
  int a[3];  // Числа из файла
  
  cout << "Различные элементы набора: ";
 
  infile >> a[0] >> a[1];
  while (infile >> a[2])
  {
        if ((!(a[1]==a[0])) && (!(a[1]==a[2])))
        {  
              cout << a[1] << " ";
        }
        
        a[0] = a[1];
        a[1] = a[2];
  }
  
  cout << endl;
  
  infile.close();
        
  system("Pause");
  return 0;
}

[BDA]

Что-то Вы перемудрили в этой задачке.

int a, b, n;
infile >> n;
if (n) {
    infile >> a;
    cout << a;
}
for (int i = 1; i < n; ++i) {
    infile >> b;
    if (a != b) {
        cout << " " << b;
        a = b;
    }
}

UPD
Пожалуйста

[2107]

работает. спасибо огромное, BDA!

[ROD]

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

Препод запретил массивы использовать.
Как вот эту задачу можно решить без использования массивов?

Я бы, чисто чтобы поиздеваться над таким преподом, решил ее с помощью списков (однонаправленного вполне хватит) или контейнеров.