Szukanie największej liczby z tablicy

Question

Dzień dobry,

ogólnie rzecz biorąc jestem po 11 odcinku kursu C++ i pomyślałam sobie, że fajnie byłoby napisać program, który oprócz wyszukania największej liczby, da użytkownikowi możliwość wybrania ich ilości.Oczywiście wszystko z użyciem tablic. Pomimo faktu, że program się włącza, dzieje się następująca rzecz .,.

Kod wygląda tak:

int ile,m;

int main()
{
    cout << "Ile liczb zamierzasz podac: ";
    cin>>ile;

    int *tablica;
    tablica = new int [ile];

    cout<< "Podaj liczby oddzielone spacja: "<<endl;

    for (int i=0; i<ile;i++)
    {
        cin>>*tablica;
        *tablica++;
    }

    m=*tablica;


    for (int i=0;i<ile;i++)
    {
        if (*tablica>m)
        {
            m=*tablica;
        }

        tablica++;
    }
    cout<<"Najwieksza liczba to: "<<m;

    delete []tablica;
    return 0;
}

Prosiłabym o jakąś wskazówkę, co może nie działać, a może także o jakieś uwagi dotyczące napisania tego w prostszy sposób >-< Jestem osobą początkującą, więc zdaję sobie sprawę, że niektóre rzeczy mogę chcieć osiągnąć "na okrętkę", niekoniecznie to zauważając.

Z góry dziękuję za każdą odpowiedź ^-^

Answer 1

W linii 16 przesuwasz wskaźnik tablica, co powoduje, że następna pętla czyta poza zakresem tablicy. Dlaczego nie użyjesz operatora indeksowania []?

Comments

Jeżeli chodzi o coś w stylu

m=tablica[0];


    for (int i=0;i<ile;i++)
    {
        if (tablica[i]>m)
        {
            m=tablica[i];
        }
    }

To to też nie działa .-. Jeśli nie o to, to średnio rozumiem >-< Znaczy.. jest opcja totalnego pominięcia wskaźnika od początku, ale wprowadziłam go tam właśnie dlatego, aby poćwiczyć jego użycie .,.

---

O to mi chodziło. Obie pętle musisz poprawić.

---

No dobra, w ten sposób działa ^-^ Ale nie ma możliwości jakby... ''wyzerowania" tego wskaźnika, żeby znowu wskazywał tablica[0]? Wiem, że się przy tym strasznie upieram, no ale... miałam taką wizję od początku i chciałabym się dowiedzieć >-<

Dziękuję bardzo~

---

Niech pętle operują na kopii wskaźnika tablica. Ewentualnie odwołuj się tak: m = *(tablica + i);

Answer 2

Ok, upierasz się przy wskaźnikach... dobrze:

#include <iostream>
#include <cstddef> // dla std::size_t
 
int main()
{
    std::size_t ile;
    int maksymalna;
    std::cout << "Ile liczb zamierzasz podac: ";
    std::cin >> ile;
 
    int * tablica;
    tablica = new int[ile];
 
    std::cout<< "Podaj liczby oddzielone spacja: \n";
 
    for (std::size_t i = 0; i < ile; ++i)
    {
        std::cin >> *(tablica + i);
    }
 
    maksymalna = *tablica;
 
    for (std::size_t i = 0; i < ile; ++i)
    {
        if (*(tablica + i) > maksymalna)
        {
            maksymalna = *(tablica + i);
        }
    }

    std::cout << "Najwieksza liczba to: " << maksymalna << '\n';
 
    delete []tablica;
}

Na indeksach jest czytelniej:

#include <iostream>
#include <cstddef> // dla std::size_t
 
int main()
{
    std::size_t ile;
    int maksymalna;
    std::cout << "Ile liczb zamierzasz podac: ";
    std::cin >> ile;
 
    int * tablica;
    tablica = new int[ile];
 
    std::cout<< "Podaj liczby oddzielone spacja: \n";
 
    for (std::size_t i = 0; i < ile; ++i)
    {
        std::cin >> tablica[i];
    }
 
    maksymalna = tablica[0];
 
 
    for (std::size_t i = 0; i < ile; ++i)
    {
        if (tablica[i] > maksymalna)
        {
            maksymalna = tablica[i];
        }
    }

    std::cout << "Najwieksza liczba to: " << maksymalna << '\n';
 
    delete []tablica;
}

 

Comments

Na coś takiego bym nie wpadła >-< I tak, wiem, że czytelniej, tylko wersję z indeksami mogłabym sobie napisać na spokojnie, bo je rozumiem, a takich wskaźników.. no jeszcze nie do końca .,. Ale teraz już jestem o te trochę mądrzejsza, dziękuję bardzo ^-^