[C++] Beräkna medelvärde

Har för mig att ett fält som skall innehålla 20 olika tal börjar från [0] och slutar [19], i din loop så börjar du på 20 vilket är utanför 0-19.

är lite rostig, var ett par år sen jag rörde java/c++

Som WeiQ säger så indexeras en array med 20 element från 0 till 19.

Dessutom är din din for-loop helt fel.

http://www.cplusplus.com/doc/tutorial/control/

Nu gjorde du ju jobbet åt honom, WarWolf.667.

Alla variabler utom i måste vara typade som flyttal som du gjort dina beräkningar i exemplet.

Så borde det vara. Men tar du hänsyn till att de kan innehålla vad som helst från början (eller vad är det som blir fel)?

Ett problem är dock som sagt

vilket borde kunna ställa till det till exempel här

varden_2[i]=varden_2[i-1];

Varför inte ändra loop-huvudet till hur du egentligen menar?

Static är inte lämpligt för att få dess sekundära funktionalitet. Det som static gör (när nyckelordet används framför en lokal variabel inuti en funktion) är att tvinga kvar den variabeln i minnet så länge programmet kör. Detta är inte bra om man inte faktiskt vill detta, och kan skapa otrevliga buggar / minnesläckor.

Dock tycker jag hela koden i sig är knasig, varför skall du konvertera denna? Känns som det är skrivet av en person som verkligen inte har koll, lite noteringar:

for(i=20;i--;i>1)

Ingen initialisering av indexvariabel, går visserligen att använda, men konstig praxis. Dvs inget "int xx = yy" utan bara "xx=yy".
Fel ordning på slutvillkor och uppdatering av variabel (det som perost påpekade).
Om det faktiskt varit korrekt skulle den itererat över 20-1 istället för 19-0, vilket i sig fungerar bättre med hur loopkroppen ser ut, men är konstigt i vilket fall.

	varden_2[i]=varden_2[i-1];
	varden_1[i]=varden_1[i-1];
	medel_2=medel_2 + varden_2[i];
	medel_1=medel_1 + varden_1[i];

Detta är intressant. Det som händer är ju att alla tal flyttas upp ett steg i vektorn, och sedan adderas till "medeltalet". Dvs 0-1-2-3 skulle ge 0-0-1-2 samt icke-delat medeltal 3.

Är du säker på medeltalsberäkning är vad kodbiten är menad att göra? Inte ens en nybörjare skulle väl komma på idén att skriva dessa omvägar för att göra något så simpelt som att beräkna medeltalet i två vektorer?

Koden är ju dock en kombination av medelvärdesuträkning (det sista värdet adderas efteråt) och bufferthantering.

Att skapa en minnesläcka med static är nog inte speciellt troligt, men det går utmärkt utan att tvinga till sig reserverat minne med malloc, även om minnesläckan inte orsakar något mer än att krasha programmet.

Bra poäng, själv kodade jag i C++ före C, och det har därför fallit naturligt för mig!

Ur det perspektivet är koden ännu konstigare. Om man skulle lägga detta i en loop kommer alltså input-värdet läggas på plats 0 i varje loopsteg, och alla andra trycks upp ett steg. Detta sker dock endast upp till 20 platser, maxgränsen för vektorn. Borde då inte medeltalet beräknas m.h.a. det faktiska antal värden som är instoppade (max 20)?

Oavsett hur koden skall och inte skall funka, så känns det inte som att medeltalsberäkning är hela sanningen angående denna kod

Att deklarera variabeln utanför innebär också att man kan se vilket värde den senast hade efter loopen, vilket kan vara användbart ibland.

Det är väl dock ett designbeslut; vi vet ju som sagt inte vad syftet är.

Har väl sisådär två timmars erfarenhet av C++, men till och med jag kan se att ts tydligen inte vet vad han sysslar med

Skickades från m.sweclockers.com

Det var inte det ts frågade om...

Han är här för att lära sig. Sluta klanka ner på människor med onödiga kommentarer som skapar onödiga kommentarer som denna...

Det enda du behöver göra är att addera alla värden och sen dela summan med antalet värden, där har du medelvärdet!