Jak znaleźć minimalną wartość dla n-ciągu liczb. Help!

pytanie zadane 14 listopada 2015 w C i C++ przez Peeter_101 Nowicjusz (160 p.)
edycja 14 listopada 2015 przez Peeter_101

Niżej zamieszczam moje wypociny:

#include <iostream>

using namespace std;
int  n, a, mn, y;
int main()
{
 cin>>n;
 

    for(int i=1;i=n-1;i++)
    {
        cin>>a;
        cin>>y
     if(mn>a) mn=y cout<<y;

        else cout<<a;

    }
return 0;

Dodam, że powinna być to metoda elementarna.

1 odpowiedź

odpowiedź 14 listopada 2015 przez emSon Stary wyjadacz (10,480 p.)

Żeby się kompilowało:

Wstaw średniki na końcu paru instrukcji.
Zamknij instrukcję dla i'fów w bloku. Jeśli if ma więcej niż jedną instrukcje do wykonania, należy zamknać je klamrami.

Żeby działało

Popraw pętle. Narazie warunkiem jej skończenia jest .. no właśnie, co?

Żebyśmy ci pomogli

Powiedz co to ma dokładnie robić.
Sformatuj kod.
Porządnie nazwij zmienne. m mówi mi tyle co koślawa dupa...

komentarz 14 listopada 2015 przez Peeter_101 Nowicjusz (160 p.)

Program ma znaleźć najmnieszą liczbę w ciągu liczb, gdzie n jest liczbą określającą ilość liczb w ciągu.

komentarz 14 listopada 2015 przez emSon Stary wyjadacz (10,480 p.)

Najpierw popraw pętle for. Powinna ona wyglądać tak:

for(int i=0; i<n; i++){...}

W pętli nie podajesz dwóch liczb. Powinieneś mieć dwie zmienne zadeklarowane na zewnątrz. Do jednej przypisujesz liczbę, a w drugiej przechowujesz najmniejszą. Dokładasz if'a i wszystko gra. Jak czegoś nie rozumiesz jeszcze to pisz. I błagam... popraw ten kod.

komentarz 14 listopada 2015 przez Peeter_101 Nowicjusz (160 p.)

Wyrzuciłem to na zewnątrz i poprawiłem warunek, jest błąd w linii 14. jak powinienem to zapisać:

#include <iostream>

using namespace std;
int  n, a, mn, y;
int main()
{
 cin>>n;
 cin>>a;
 cin>>mn;
 cin>>y;

    for(int i=1;i<n;i++)
    {
        if(mn>a) mn=y cout<<y};
        else cout<<a};
    }
return 0;
}

komentarz 14 listopada 2015 przez emSon Stary wyjadacz (10,480 p.)

mówiłem ci, że musisz zamknąć te dwie instrukcje przy ifie w klamrach. ty dałeś tylko jedną, zamykajacą. tak samo w else, chociaż tam już nie musiałeś zamykać bo wykonuje on tylko jedną instrukcję.

i pamiętaj: JEDNA INSTRUKCJA , JEDNA LINIA

więc powinno to mniej więcej tak wyglądać tak:

 if(najmniejszaLiczba>podanaLiczba){
    najmniejszaLiczba = podanaLiczba;
    cout << najmniejszaLiczba << endl;
}
else{ 
   cout << podanaLiczba << endl; 
}

pozmieniaj nazwy zmiennych