Vektorer C++ Hjälp

Tja!

Kan inte C++ överhuvudtaget, men om jag tolkar dig rätt så låter det som att vektorn inte behöver vara dynamisk (ändras i storlek över tiden), eftersom man på förhand vet hur många tal som kommer matas in. Det dynamiska är snarare att programmet skapar en vektor med storlek baserat på hur många mätningar användaren säger att den ska mata in, om det make:ar sense

Du allokerar en array av int till 'x' men sen använder du aldrig den. Istället använder du 'temperatures' som du inte allokerat.

@Skogskotten: Jag antar att din kompilator klagar på att "sizeof temperatures isn't known eller liknande. Så ska du allokera något, så får du väl allokera "size" också, och i sånt fall efter "cin >> size".

Men som föregående inlägg, du använder aldrig "x" till något.

Så här typ:

int main()
{
float sum = 0;

cout << "Hur många m\x84tningar har gjorts?\n";
int size;

cin >> size;
int* temperatures = new int[size];

for (int i = 0; i < size; i++)
{
cin >> temperatures[i];
sum = sum + temperatures[i];
}

cout << "Summan blir: " << sum;
float avg = sum / size;
cout << "Medeltemperaturen blir: " << avg;

cin.get();
return 0;
}

Borde fungera, men kan ha något fel också, det är trots allt söndag :). Sen behöver du kanske heller inte allokera nåt på heapen, men i och med att du bara kör programmet en gång så behöver du inte oroa dig för minnesläckor.

#include <iostream> //For cin, cout
#include <vector> //For vector
#include <numeric> //For accumulate

int main() {

//Define what we need
std::vector<float> measurements;
unsigned int numberOfMeasurements = 0;

std::cout << "How many measurments do you want to log?" << std::endl;

std::cin >> numberOfMeasurements;

//Log all the temperatures
for (unsigned int i = 0; i < numberOfMeasurements; i++) {
std::cout << "What is temperature number: " << i << std::endl;
float measurement;
std::cin >> measurement;
measurements.push_back(measurement);
}

//Calculate the average
float average = std::accumulate(measurements.begin(), measurements.end(), 0.0) / measurements.size();

//Print individual temperatures
std::cout << "Temperatures: " << std::endl;
for (unsigned int i = 0; i < numberOfMeasurements; i++) {
std::cout << measurements.at(i);
}

//print the average temperature
std::cout << "Average temperature: " << average << std::endl;

return 0;
}

Eftersom det är dynamiska vektorer som efterfrågas är det std::vector du vill använda

Går även loopa över std::vector lite snyggare...

Om vec är vår vector:

std::vector<float> vec;

for (auto i = vec.begin(); i != vec.end(); i++) {
std::cout << Temp: << *i << std:endl;
}

*i har här typen float

@Skogskotten: Hoppas det hjälper

@Skogskotten: "Krister" är konstant i.o.m att du skrivit in det direkt, så det kommer aldirg gå att ändra, funktionen Hello, förväntar sig en char array som den kan mutera (ändra)

@Skogskotten: Om det är C++ du ska använda skulel jag rekommendera std::string istället:

#include <iostream>
#include <string>

void Hello(std::string name)
{
std::cout << "Hej " << name << "!";
}

int main()
{
Hello("Krister");

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