Zadanko z c++ problemik

Question

Siemka, chcialem napisac program na wskaznikach w ktorym uzytkownik wpisuje  10 cyfr a nastepnie program wypisuje z nich najwiekszą i najmniejszą. Czy robie to dobrze na wskaznikach, czy mozna inaczej. ? Cos w kodzie nie dziala(kompiluje się) ponieważ wypisuje mi randomowe liczby. Czy warunek jest dobry ?

#include <iostream>

using namespace std;

void najliczba(double *x);

int main()
{
    double tab[10];
    cout << "Podaj 10 liczb: " << endl;
    for(int i=0;i<10;i++)
    {
        cin>>tab[i];
    }
    najliczba(tab);
    return 0;
}
void najliczba(double *x)
{
    double maxymalna, minimalna;
    for(int i=0;i<10;i++)
    {
        if(x[i]<x[i+1])
        maxymalna=x[i+1];
    }
    cout<<"Najwieksza liczba sposrod podanych to: "<<maxymalna<<endl;
    for(int i=0;i<9;i++)
    {
        if(x[i]<x[i+1])
        maxymalna=x[i+1];
    }
    for(int j=0;j<9;j++)
    {
        if(x[j]>x[j+1])
        minimalna=x[j+1];
    }
    cout<<"Najmniejsza liczba sposrod podanych to: "<<minimalna<<endl;

}

 

 

Answer 1

 

• int j=0;j<9;j++

  a w innym miejscu 10 zamiast 9. Wiesz, że numerujemy od zera, ale chyba chwilowo zapomniałeś

• Ta druga pętla to chyba nieskasowana kopia?

• Kluczowe, program podaje błędne wyniki względem oczekiwanych, ponieważ algorytm szukania minimum i maksimum jest troszkę inny. Spróbuj przeanazlizować co sie dzieje w tych pętlach, a zobaczysz, że te liczby które ci wyskakują mają pewną zasade - tylko, że inną niż chciałeś uzyskać. Klasyczny błąd - sam go osobiście przerabiałem podczas nauki c++

Podpowiem ci: dla liczb 1,1,1,4,1,1,1,2,1,1 otrzymasz "maksimum" 2, bo nadpisujesz wartość 4 (porównujesz wyłącznie z jedynką ignorując co było wcześniej).

Rozwiązanie: Przed pętlą robisz max=x[0], a później porównujesz kolejne elementy ze zmienną max i gdy znajdziesz coś większego, to wtedy zapisać to pod zmienną max

Z minimum będzie podobnie

Comments

dzieki zrozumialem, mam jeszcze pytanko, czy cos tutaj wiecej da sie zrobic na wskaznikach, ponieważ w poleceniu mam by wykonac to zadanie na wskaznikach dla tablicy 10 elementowej. Czy cos na wskaznikach oplacalo by sie czy wystarczy to co zrobilem? wyslam kod

#include <iostream>

using namespace std;

void najliczba(double *x);

int main()
{
    double tab[10];
    cout << "Podaj 10 liczb: " << endl;
    for(int i=0;i<10;i++)
    {
        cin>>tab[i];
    }
    najliczba(tab);
}
void najliczba(double *x)
{
    double maxymalna = x[0], minimalna = x[0];

    for(int i=0;i<10;i++)
    {
           if(maxymalna < x[i])
    {
        maxymalna = x[i];
    }
            if(minimalna > x[i])
            {
                minimalna = x[i];
            }
    }
    cout<<"Liczba najwieksza to: "<<maxymalna<<endl;
    cout<<"Liczba najmniejsza to: "<<minimalna<<endl;
}

 

---

Przy wcześniejszej odpowiedzi faktycznie zignorowałem wzkaźniki.

Tylko powstaje pytanie co nim chcesz uzyskać? Aktualnie masz gwiazdkę która ostatecznie niczego nie zmienia. Gdybyś wysłał tablice poza funkcje i później jej użył np. do wypisania elementów w funkcji main lub funkcji uruchamianej z poziomu funkcji main to tu masz przypadek praktycznego użycia wzkaźnika.

Inna moja interpretacja to by wysłać wartość minimalną i maksymalną przy użyciu wzkaźnika do funkcji głównej i wysłać komunikat użytkownikowi właśnie z poziomu funkcji main. W ten sposób odseparować kod funkcji najliczba() (logika programu) od wypisywania komunikatu (widok)

Aczkolwiek powyższe lepiej zrobić przez referencję. Uczyliście się referencji? Jeśli nie to prawdopodobnie na najbliższych zajęciach poznasz to rozwiązanie, które ma podobieństwa do wzkaźników (bo na pewnym poziomie jest to jego wyszczególniony przypadek)

 

PS: Oglądałeś tutorial Pana Mirosława Zelenta? http://miroslawzelent.pl/kurs-c++/wskazniki-alokowanie-pamieci

---

Hej, znaczy ogolnie zadanko ma byc zrobione na wskaznikach, ja sie jeszcze w nie dopiero zagłebiam i stworzyłem coś takiego, czy to ma sens ?

#include <iostream>

using namespace std;

    void najliczba(double *x);

int main()
{
    double tab[10];
    double *wsk = tab;
    cout << "Podaj 10 liczb: " << endl;
    for(int i=0;i<10;i++)
    {
        cin >>*(wsk+i);
    }
    najliczba(wsk);
}
    void najliczba(double *x)
{
    double maximum =*x , minimum = *x;
    for(int i=0;i<10;i++)
    {
        if(*(x+i)>maximum)
        maximum = *(x+i);
    }
    cout<<"Najwieksza wartosc sposrod podanych to:"<<maximum<<endl;

    for(int i=0;i<10;i++)
    {
        if(*(x+i)<minimum)
        minimum = *(x+i);
    }
    cout<<"Najmniejsza wartosc sposrod podanych to: "<<minimum<<endl;
}

Ogladalem Mirka, ale jeszcze musze troszke poogarniac :) dlatego pytam Pozdrawiam

---

Sens jakiś ma. Sprawdź sam czy prawidłowo działa natomiast nie wiem co miało znaczyć by zrobić to na wskaźnikach. Powyższy kod pokazuje, że umiesz w wzkaźniki, ale to trochę sztuka dla sztuki.

Zauważ, że mógłbyś zwrócić tablice przez return i uzyskać to samo, a kod byłby czytelniejszy bez nadmiaru gwiazdek. Dlatego wydaje mi się, że większy sens miałoby użycie wzkaźników do wysłania zmiennych minimum i maksimum i komunikat wypisać w funkcji głównej, bo tego przy użyciu return tak po prostu się nie da, więc to daje wartość dodaną.