Ich habe eine Klasse (-> Zahlenliste) und ein private Datenelement zahlen vom STL Typ vector<int>.

Nun soll ich einen Konstruktor mit einem int Parameter für die Anzahl der enthaltenen Zahlen (die Liste soll n Elemente haben , die je eine zufällige Zahl sind) erstellen.

Kann ich mit meinem vector , der zahlen heisst, so umgehen wie mit einem Array?Oder warum gibt mein Code hier einen Fehler?

#include<iostream>
#include<vector>
#include<algorithm>
using namespace std;

class Zahlenliste{
	  private:
	  		  vector<int> zahlen;

	  public:
	  		 Zahlenliste(int n):zahlen(n){
			 				 zahlen=new int[n];
			 				 for(int i=0;i<n;i++)
			 				 zahlen[i]=(rand()%n)+1;
							  }
	  };

Lg caro

Hallo !

Ich habe eine Klasse (-> Zahlenliste) und ein private Datenelement zahlen vom STL Typ vector<int>.

Nun soll ich einen Konstruktor mit einem int Parameter für die Anzahl der enthaltenen Zahlen (die Liste soll n Elemente haben , die je eine zufällige Zahl sind) erstellen.

Kann ich mit meinem vector , der zahlen heisst, so umgehen wie mit einem Array?Oder warum gibt mein Code hier einen Fehler?

#include<iostream>
#include<vector>
#include<algorithm>
using namespace std;

class Zahlenliste{
	  private:
	  		  vector<int> zahlen;
	  
	  public:
	  		 Zahlenliste(int n):zahlen(n){
			 				 zahlen=new int[n];
			 				 for(int i=0;i<n;i++)
			 				 zahlen[i]=(rand()%n)+1;
							  }
	  };

Lg caro

die Zeile

zahlen=new int[n];

ist unnötig. Ein Vector ist dynamisch und du musst die Größe nicht angeben

Schau mal hier:
http://www.cppreference.com/wiki/stl/vector/start

Da findest du alle Methoden und Beispiele zur Benutzung.

Ich habe das jetzt so gemacht...macht man das so?

Btw: Am Besten niemals den std-Namensraum auflösen

1. Niemals in Headern
2. Gilt das nicht nur für std sondern für jeden Namensraum!

@fischldi:
In dem Fall ist das unnötige CPU-Zeit, die du da verbrätst.
In der Initialisierungsliste füllst du deinen vector mit (defaultkonstruierten) Werten. Im Konstruktorbody weist du denen dann nochmal neue Werte zu.

Es wäre besser, wenn du deinem Vector genügend Speicher gibst und dann einfach mit push_back arbeitest:

ZahleListe::ZahlenListe( size_t size )
{
    zahlen.reserve(size); // verhindert unnötiges Kopieren, da der vector
    // falls er zu klein ist sich selbstständig vergrößert
    for( size_t i = 0; i < size; i++ ) {
        zahlen.push_back((rand()%size));
    }
}

Folgene Möglichkeiten:

#include<vector>
#include <cstdlib> // for 'rand()'

class Zahlenliste
{
private:
  vector<int> zahlen;

public:
  Zahlenliste(int n)
  : zahlen(n) //Groesse vorbelegt
  {
    for(int i = 0; i < n; ++i)
      zahlen[i] = rand() % n; //vorinitialisierte Elemente aendern
  }
};

#include<vector>
#include <cstdlib> // for 'rand()'

class Zahlenliste
{
private:
  vector<int> zahlen;

public:
  Zahlenliste(int n)
  : zahlen() //leerer vector
  {
    for(int i = 0; i < n; ++i)
      zahlen.push_back(rand()%n);
  }
};