znajdowanie max i min

pytanie zadane 14 maja 2016 w C i C++ przez Evelek Nałogowiec (28,960 p.)

Mam algorytm na sprawdzanie max i min wartości z pliku tekstowego. Algorytm działa poprawnie, kiedy w pliku tekstowym jest 1000 liczb (w którym znajdują się liczby nawet 250-cyfrowe). Kiedy jest w nim 5000 liczb (max 9-cyfrowe) to wypisuje bzdury. Nie rozumiem tego, bo powinien działać poprawnie...

#include <iostream>
#include <fstream>
#include <string>
using namespace std;
int main()
{
    fstream plik;
    plik.open("dane.txt", ios::in);
    string liczba, pomMax, pomMin;
    int nrMax = 1;
    int nrMin = 1;
    while(!plik.eof())
    {
        plik >> liczba;
        pomMax = pomMin = liczba;

        for(int i=2; i<5001; i++)
        {
            plik >>liczba;
            if(liczba.size() > pomMax.size())
            {
                pomMax = liczba;
                nrMax = i;
            }
            else if(liczba.size()==pomMax.size())
                if(liczba > pomMax)
                {
                    pomMax = liczba;
                    nrMax = i;
                }
            if(liczba.size() < pomMin.size())
            {
                pomMin = liczba;
                nrMin = i;
            }
            else if(liczba.size()==pomMin.size())
                if(liczba < pomMin)
                {
                    pomMin = liczba;
                    nrMin = i;
                }
        }
    }
    cout << "Najmniejsza liczba znajduje sie w linii numer: " << nrMin << endl;
    cout << "Najwieksza liczba znajduje sie w linii numer: " << nrMax << endl;
    cout << pomMax << endl;
    cout << pomMin << endl;
    plik.close();
    return 0;
}

1 odpowiedź

odpowiedź 14 maja 2016 przez jankustosz1 Nałogowiec (35,880 p.)

nie chce mi się na to patrzeć ale tak zgrubsza

ogólnie to ciekawy pomysł sprawdzać który string ma więcej znaków ten przechowuje większa liczbę, ale jest jak oba stringi będą miały po tyle samo znaków (załóżmy 20) to będzie bład bo wyjdzie poza zasięg inta, więc nie pisz że dla 250 znaków pójdzie.

linia 19 już raz przeczytałeś to 3 linie wcześniej, chcesz aby czytało 5 tyś wyrazów z pliku nie ważne czy istnieją czy nie, kompletnie nie rozumiem jaki jest cel robienia tego fora

Jest masa błędów logicznych w każdym jednym ifie.

Najlepiej usuń to i napisz jeszcze raz i pisz komentarze żebyś wiedział co robisz.

komentarz 14 maja 2016 przez Evelek Nałogowiec (28,960 p.)

Liczby nie są przechowywane w int tylko w string. String też umożliwia na ich porównywanie.

Powtórne wczytanie plik >> liczba jest tu konieczne.

Nie ma tu błędów logicznych.