kan man förbättra min metoden?

Hallo igen,

Jag börjar med att visa koden
[PHP]
void printMaxRainWeek(int antalDag, int lista[]) {

int i = 0;
int day = 0;
int sumRain = 0;
int maxSumRain = 0;
double week;
while( i < antalDag ) {
if ( day < 6 && i != (antalDag-1) ) {
sumRain += lista[i];
day++;
} else if ( i == (antalDag-1) ) {
sumRain += lista[i];
if( maxSumRain < sumRain ) {
maxSumRain = sumRain;
}
} else {
sumRain += lista[i];
if( maxSumRain < sumRain ) {
maxSumRain = sumRain;
week = i;
}
sumRain = 0;
day = 0;
}
i++;
}
cout << "Vecka " << ceil(week/7) << " regnade det " << maxSumRain << " mm och det var mest under manaden" << endl;

}
[/PHP]

Metoden ovan går ut på att räkna ut summa regnmängd för varje vecka, som finns sparad i en array. Array:en innehåller regnvärde för 30 dagar. Nu kommer problemet som jag löste på ett mindre snyggt sätt tycker jag. Kan man korta ner koden på nåt sätt?

Det jag har gjort nu är att den kollar på de 4 fyra första veckorna sen har jag en else if där man löser sista veckan också genom att man kollar när man har kommit på slutet av månaden och gör en summering av sista veckan där också.

Jag får inte din beskrivning att gå ihop med koden? Du har regndata över en månad? Vecka 1 startar på dag 1? Du vill hitta den vecka då det regnade mest?


Så här löste jag _det_ problemet:
[PHP]

void print_rainiest_week( int lista[] ) {

int mm_per_week[5] = {0}; // Dela upp nederbörden i veckor
int max = 0; // Maxvärdet för nederbörden
int week; // Den vecka då det regnade mest
// Loopa över listan
for (int i = 0; i < 30; i++) {
mm_per_week[i/7] += lista[i]; // Addera nederbörden för dagen till rätt vecka.
// om det har regnat mer denna veckan ( kan hända på torsdag! ) än tidigare
if ( mm_per_week[i/7] > max ) {
max = mm_per_week[i/7]; // sätt max till veckans nederbörd
week = i/7; // sätt veckan till denna vecka
}
}
cout << "Vecka " << week << " regnade det " << max << " mm" << endl;
}
[/PHP]

PS. Denna kod hanterar inte det fallet då det är en helt torr månad.

hehe, jag undrade ifall det var jag som var otydligt i min beskrivning när man inte fick något svar. verkade vara det också, men du luskade ut det jag var ute efter iaf.

Kan bryta upp det jag gjort:

1. lista[] med regnmängd för okänt antal dagar för en månad läses in. Det ska sedan summera totala regnmängd för varje vecka.

2. den vecka som regnade mest sparas undan som max. även vecka det var på sparas undan.

3. resultat presenteras.

-----------------------

Din lösning blev avsevärt mindre kod än min, och det var precis det jag undrade över. Mah, tycker dessutom det var bra mycket snyggare lösning också (om man kollar på läsbarheten) ^^

Tack och varsågod att använda koden om du skulle vilja det (den är inte vattentät )!

mjo, nog är de det. men det är bra att kolla igenom koden för utbildningssyfte. så att man vet hur man skulle kunna ha gjort det istället.

en annan lösning