Wskazniki! Czyli co ja tu źle robię?

Question

Witam,

Ostatnio dodałem posta z moim zadaniem do Challangu z forum. Spróbowałem przekształcić go żeby używał wskazników. Program kompiluje sie ale kompletnie nie działa. Gdzie jest błąd?

 

#include <iostream>

using namespace std;

int main()
{
    int l_ucz,m,n;
    cin>>l_ucz;
    string *imie;
    imie= new string[l_ucz];
    float *srednia;
    srednia= new float[l_ucz];
    float ocena=0,suma=0,maxi,mini,suma_cala;
    for(int i=0; i<l_ucz; i++)
    {
        cin>>imie[i];
        for (int j=0; j<5; j++)
        {
            cin>>ocena;
            suma+=ocena;
            ocena=0;


        }
        srednia[i]+=(suma/5);


                if (i>0)
                {if (srednia[i]>srednia[i-1]) {maxi=srednia[i]; m=i;}}
                   else {maxi=srednia[0]; m=0;}



                    if (i>0)
                {if (srednia[i]<srednia[i-1]) {mini=srednia[i];n=i;}}
                   else {mini=srednia[0]; n=0;}

        suma_cala+=suma;
        suma=0;
    }
    delete [] imie , srednia;
    cout<<endl<<imie[m]<<endl<<maxi;
    cout<<endl<<imie[n]<<endl<<mini;
    cout<<endl<<(suma_cala/(5*l_ucz));

    return 0;
}

 

Answer 1

Znalazłem swój błąd... nie dość że wymazałem te tablice przed ich wypisaniem to jeszcze w niewłaściwy sposób -..-

Answer 2

A co ten program miał robić w ogóle? oprócz tego, że liczy średnią?

Comments

Miał wybrać najlepszego i najgorszego ucznia. Tu jest link https://forum.pasja-informatyki.pl/11228/challenge-%232-pani-agnieszka-nauczycielka-z-powolania

Answer 3

tu są błędy - for(int i=0; i<l_ucz; i++) :>

zrób - for(int i=0; i<l_ucz-1; i++)

Pamiętaj, że liczymy po amerykańsku. 

 

Pozdro! ;D

Comments

a nie, tu jest dobrze jednak xD

---

No ale u mnie jest mniejsze a nie mniejsze bądź równe. No i znowu doświadczyłem sytuacji ze ten program zadziałał na ideone a u mnie w Code Blocks nie...

---

No własnie xd

---

no właśnie zauważyłem to w pewnej chwili... xD Może to nie wina Code::Blocksa, tylko kompilatora? ;> Może masz inny niż gcc ustawiony i masz inną skadnię troszkę :> Albo masz to przez przypadek jako projekt C? Spróbuj z tymi operatorami z C do operacji na pamięci... :> Wydaje mi się, że masz wszystko dobrze... :/

 

Pozdro! :D

Answer 4

Nie ma to jak usunć tablicę i wyświetlić co jest w niej.

 

delete [] imie;

delete [] srednia;

Powinna być przed return 0;

Comments

Wczoraj wieczorem już znalazłem swój błąd ;) Ale dzięki za odp :)

Answer 5

Jeśli mógłbym mieć prośbę to czy ktoś z was mógłby wkleić ten kod u siebie z danymi z challenga i sprawdzić czy działa?

 

#include <iostream>


using namespace std;

int main()
{
    int l_ucz,m,n;
    cin>>l_ucz;
    string *imie;
    imie= new string[l_ucz];
    float *srednia;
    srednia= new float[l_ucz];
    float ocena=0,suma=0,maxi,mini,suma_cala;
    for(int i=0; i<l_ucz; i++)
    {
        cin>>imie[i];
        for (int j=0; j<5; j++)
        {
            cin>>ocena;
            suma+=ocena;
            ocena=0;


        }
        srednia[i]+=(suma/5);


                if (i>0)
                {if (srednia[i]>srednia[i-1]) {maxi=srednia[i]; m=i;}}
                   else {maxi=srednia[0]; m=0;}



                    if (i>0)
                {if (srednia[i]<srednia[i-1]) {mini=srednia[i];n=i;}}
                   else {mini=srednia[0]; n=0;}

        suma_cala+=suma;
        suma=0;
    }

    cout<<endl<<imie[m]<<endl<<maxi;
    cout<<endl<<imie[n]<<endl<<mini;
    cout<<endl<<(suma_cala/(5*l_ucz));
    delete [] imie;
    delete [] srednia;
   cin.sync();
   cin.get();


    return 0;
}

Strasznie mnie to denerwuje ze na Ideone wszystko zawsze działa xd