C++ pagalba su uždaviniu.

[EnterTheSamurai]

Sveiki,nelabai gerai sekasi išspręsti uždavinį,tad reikia pagalbos.

https://ibb.co/FDtd2Vy - Mano bandymas spręsti uždavinį.

https://ibb.co/s1dxHzh - Uždavinio sąlyga.

 

 

[Zero Two]

pirma susitvarkyk, kad bent jau kompiliuotusi, nes atrodo, kad maigai belekokius mygtukus ir bandai laime. O del klaidu pagrindiniu: pirmam cikle pridejineji A[cn], tai kiekviena karta pridesi 100-aji masyvo elementa, antras ciklas nieko nedaro, kartu ir nesikompiliuoja, vidurki apskaiciuoji du kartus? nu vienu zodziu dar reiktu pamastyti paciam

[EnterTheSamurai]

Sveikas,šiek tiek pratryniau ir patvarkiau tikiuos labiau į temą labiau pataikiau.

Ar trūkska man ten dar kažko ir ką dar galima būtų koreaguoti? Kiek domėjaus reikia funkcija void naudoti,bet neįsivaizduoju ar tiks.

https://ibb.co/Z2vrpXT - Pakeistas.

[Dovias]

#include <fstream>
#include <iomanip>

int main() {
    std::ifstream skaitymas("Duomenys.txt");

    int mKiekis;
    double x1, x2;
    skaitymas >> mKiekis >> x1 >> x2;


    double vSvoris = 0, aMoliugai[mKiekis], avSvoris = 0;
    int amKiekis = 0;
    for (int i = 0; i < mKiekis; i++) {
        double mSvoris;
        skaitymas >> mSvoris;
        vSvoris += mSvoris;

        if (x1 <= mSvoris && mSvoris <= x2) {
            aMoliugai[amKiekis++] = mSvoris;
            avSvoris += mSvoris;
        }

    }
    skaitymas.close();



    std::ofstream rasymas("Rezultatai.txt");
    rasymas << "Visų moliūgų vidutinis svoris: " << std::fixed << std::setprecision(3) << vSvoris / mKiekis << " kg\n";
    if (amKiekis == 0) {
        rasymas << "Į supirkimo punktą atrinktų moliūgų nėra.";
    } else {
        rasymas << "Į supirkimą atrinkta: " << amKiekis << '\n';
        rasymas << "Vidutinis vieno atrinkto moliūgo svoris: " << avSvoris / amKiekis << " kg\n";
        rasymas << "Atrinktų moliūgų svorių sąrašas:";
        for (int i = 0; i < amKiekis; i++) {
            rasymas << ' ' << aMoliugai[i];
        }
    }
    rasymas.close();
    return 0;
}

Mano sprendimas. Nėra pats efektyviausias, galima kaikuriose vietose optimizuot bet veikia. Pati principą skaitymo bei išrašymo prasme atrodo darai gerai, bet pati duomenų logika yra neegzistuojanti tavo sprendime. Na pavyzdžiui tu sukuri masyvą dydžiu cn, o moliūgo svorį išsaugai į kintantį i, bet pridedi moliugo svorį iš indekso cn, kas yra 100 tavo atvejyje. Tu tikrini reikšmę už masyvo ribų. jeigu turi masyvą dydžiu 100, tavo paskutinis masyvo indeksas bus 99, kadangi tu tikrini masyvo indeksą didesnį nei pats masyvo dydis, programa gauna random reikšmę, nes ji neužtikrina kokias reikšmes tu išgausi už jo dydžio, todėl svorio kintamasis išvis neveiks ir mes milijonines reikšmes (tikriausiai taip ir daro tavo atveju).

Antra, kai tu deklaruoji kintamuosius, kintamasis nebūna automatiškai nustatytas į 0. Taip yra ne visose kalbose (pvž Java visą laiką deklaruojant nustato reikšmę į 0), bet būtent pagal C ir C++ specifikacijas kintamasis turės random reikšmes, kurias tu gali perrašyti į kokią norimą reikšmę, todėl jeigu tu naudosi kintamąjį pvž skaičiuojant svorį ar randant dydį, tu turi jas pirmiausia nustatyti į 0, tai yra antra priežastis kodėl gauni nesąmones savo kode.

Na tai keleta priežasčių kurias greitai pastebėjau. Sėkmės mokantis programavimo :