C++ nybörjare funktioner och vektorer

float medel(int v[], int n)
{
	...
	return (float)medel; // för att inte returnera medel som en int sätter man (float)/(double) framför för en typomvandling
}

(float)medel har ingen effekt här, medel är ju redan en float.

	for (int i = 0; i <=10; i++)
	{
		std::cout << "Skriv in tio tal: ";
		std::cin >> n[i];
	}

Den här for-loopen börjar på 0 och fortsätter så länge som i är mindre eller lika med 10. Räkna på fingrarna från 0 till 10 och se hur långt du kommer

		medel(n, 10); //Här kallar jag på min funktion
		std::cout << "Medelvärdet blir : " << summa/10;

Här anropar du funktionen, men du tilldelar inte resultatet till något. Sen skriver du ut variabeln summa som du tidigare satt till 0, så utskriften kommer så klart säga 0.

Låt mig för övrigt citera mig själv från en av dina tidigare trådar:

Ah, det missade jag. Felet beror på att du försöker använda n som en vektor, men det är ju v som är vektorn i funktionen.

#include <iostream>
#include <clocale>

float medel(int v[], int n)
{
	float summa = 0.0; //Här nollställer man summan så att vi börjar additionen från noll
	float medel = 0.0;
	for (int i = 0; i <=9; i++)

	{
		summa = summa + v[i];
	}
	medel = summa / v[9];
	return medel; 
}
int main()
{
	setlocale(LC_ALL, "swedish");

	int n[10];
	int summa = n[10];
	for (int i = 0; i <=9; i++)
	{
		std::cout << "Skriv in tio tal: ";
		std::cin >> n[i];
	}
		medel(n, 10); //Här anropar jag på min funktion
		std::cout << "Medelvärdet blir : " << summa;


	std::cin.ignore();
	std::cin.get();
	return 0;
}

Först och främst bör du omedelbart åtgärda din indentering. Det är mycket jobbigare för oss att läsa kod med felaktig indentering eller annan inkonsekvent formatering. Men till problemen:

1. I main där du anropar medel tar du aldrig tillvara på resultatet. Förslagsvis sparar du det i en variabel med namnet average eller liknande.

   float average = medel(n, 10);

   Du kan även skriva ut medelvärdet direkt om du inte behöver spara det.

   std::cout << medel(n, 10) << std::endl;

2. I medel dividerar du med arrayens 9:e element istället för antal element n, så den kommer inte ge något vettigt resultat.

3. Du har hårdkodat längden till 10 i medel. Se till att längden n du får som parameter används istället.

4. Namnet n på din array är inte särskilt beskrivande. Använd tal eller något i den stilen så man förstår vad den används till.

5. Varför skapar du variabeln summa och sätter värdet på den till det 10:e elementet i n? Särskilt illa blir det eftersom n bara har längden 10 och därmed inte har något 10:e element, så du läser något till synes slumpartat ur minnet och använder det till absolut ingenting förutom att skriva ut där du borde skriva ut medelvärdet. Du ska inte beräkna någon summa i main, så ta bort den.

@MrDoggo: Jag utgår ifrån att du läst hela tråden och funderat över var och varför TS gör tankevurpor. Självklart slipper hen en del av problematiken genom att använda funktionalitet i standardbiblioteket, men får du inte intrycket att några av de saker som TS behöver öva på är just for-loopar och array-indexering? På vilket sätt anser du att ditt inlägg förde TS framåt i sin utveckling?

@MrDoggo: Asdfghs har i sitt inlägg huvudsakligen bytt ur for-looparna mot range-baserade for-loopar men behåller så mycket av TS lösning så det är lätt att känna igen sig och kunna ta till sig den nya typen av loop och se fördelar med att man själv inte behöver hålla reda på storleken utan att kompilatorn fixar det åt dig. Den nya slumptalsgeneratorn är dock lite överkurs.

Din lösning består bara av överkurs. Trådens rubrik är "C++ nybörjare funktioner och vektorer" så det är för TS, sannolikt, helt okända delar av standardbiblioteket och en helt ny konstruktion i form av lambdat. Du kan hävda att du också bara ville illustrera andra sätt att lösa samma problem, men du har på intet sätt förklarat vad accumulate eller generate gör eller fördelarna med att använda dem. Detta hjälper varken TS eller andra intresserade av ämnet att lära sig nya saker.