Ich setze mich grad ein wenig mit Array & For Schleifen auseinander und habe ne einfach Übung gemacht. Leider kommen beim ausführen diverse Meldungen.

"The variable j is being used without being initialized"

"Unbehandelte Ausnahme bei 0x754eb727 in übung.exe: Microsoft C++- Ausnahme: std::out_of_range an Speicherposition 0x0030f808..

"Run-Time Check Failure #0 - The value of ESP was not properly saved across a function call. This is usually a result of calling a function declared with one calling convention with a function pointer declared with a different calling convention."

Das hier ist mein Code:

#include "stdafx.h"
#include <iostream>
#include <vector>

using namespace std;

void Funktion1()
{
	vector<int> Zahlen(10);
	int j;
	for (size_t i=0;i<Zahlen.size();i++)
	{
		cout << "Bitte geben sie den Wert für das Array" << Zahlen[i] << " an:";
		cin >> Zahlen.at(j);
		system("cls");
		if (i=Zahlen.size())
		{
			for (size_t i = 0;i<Zahlen.size();)
			{
				cout << Zahlen.at (i);
					i++;
			}
		}
	}
}

int _tmain(int argc, _TCHAR* argv[])
{
	Funktion1();
	system("pause");
	return 0;
}

Was habe ich denn nun alles falsch gemacht? Ich vermute es hängt mit meinem Array zusammen, ist es falsch dass ich nur dieses eine benutze?

LG

ps: Ich bringe mir das grad ohne Lehrer selber bei, habe also niemanden zum fragen!

PS: Achja, vermeide system -Aufrufe!

Mehr ist es nicht^^ Ich weiß das meine Programme mit system Aufrufen nicht mehr Linuxfähig sind, aber unser Lehrer in der Schule (Schlaftablette) meinte wir können das für die Sachen in der Schule ruhig benutzen, deswegen nehm ich es.

LG

"The variable j is being used without being initialized"

Sagt doch alles. Du benutzt die Variable j ohne sie jemals initialisiert zu haben, d.h. sie hat keinen vorher bestimmten Wert.
Hier ne kurze Analyse dessen, was da passiert:

- du erstennst eine vector mit der Länge 10. Die 10 Elemente werden default-initialisiert, d.h. sind alle 0.
- du erstellst eine Variable j, für welchen Zweck auch immer.
- Du läufst mit i von 0-10, und machst dabei folgendes:
- Du gibst den Text "Bitte geben sie den Wert für das Array 0 an:", denn zahlen[i] ist immer 0, da wie oben gesagt ja alle Elemente mit 0 initialisiert wurden. Erstens ist das kein Array, sondern jeweils nur ein Element, was du da einliest, zweitens vermute ich mal, dass du den Index in deinem Output stehen haben möchtest, nicht den Wert.
- du liest einen Wert ein und speicherst ihn an der Stelle j des vectors. Oder versuchst es zumindest. j hat nämlich überhaupt nichts mit i zu tun, ist dazu noch uninitialisiert und hat deshlab typischerweise irgendeinen großen Wert. zahlen.at(j) geht daher ziemlich schief und du bekommst deine Exception.

• Eine Funktion ohne Rückgabewert und ohne Parameter
• Einen Vektor erstellen
• Eine Schleife mit einem Durchlaufzähler der bei 0 beginnt und vor Vektor.size() endet
• In der Schleife: Ausgabe für den Nutzen und dazugehörige Eingabe aufnehmen und in Vektor schreiben
• Danach zweite Schleife mit einem Durchlaufzähler der bei 0 beginnt und vor Vektor.size() endet (nach der ersten Schleife, nicht drin)
• In der Schleife: Ausgabe des Vektors

Habe mich jetzt mal an die Anleitung meines Vorgängers gehalten und das programm so geschrieben:

void Funktion1()
{
	vector<int> Array(10);
	for (size_t i=0;i<Array.size();i++)
	{
		cout << "Bitte geben sie den Wert für dieses Array an: " << endl;
			cin >> Array.at(i) ;
			system("cls");
	}
	for (size_t i=0;i<Array.size();i++)
	{
		cout << Array.at(i) << endl;
	}
}

Jetzt Funktioniert es auch! Weiß selber nicht was ich da eig. fürn Müll zusammengeschrieben hatte o.o Aber mal noch ne klitzekleine Frage, ist es falsch mit vector zu arbeiten?

In der Schule meint unser Lehrer immer wir sollen rohen Speicher benutzen, aber ich kann das mit vector besser und finde es ehrlich auch einfacher. Außerdem wird es in meinem Buch was ich mir zur Hilfe gekauft habe auch benutzt.

LG

Aber mal noch ne klitzekleine Frage, ist es falsch mit vector zu arbeiten?

Nein, auf keinen Fall. Er garantiert, dass der Speicher freigegeben wird und arbeitet auf dem Heap (wo man genug Platz hat).

Limatex schrieb:

In der Schule meint unser Lehrer immer wir sollen rohen Speicher benutzen

So ein Depp.

Limatex schrieb:

aber ich kann das mit vector besser und finde es ehrlich auch einfacher.

Genau

ist es falsch mit vector zu arbeiten?

Nein, im Gegenteil, es ist richtig.

In der Schule meint unser Lehrer immer wir sollen rohen Speicher benutzen,

Dein Lehrer hat n Schatten. Es gibt nur ganz wenige Gründe, mit rohem Speicher zu arbeiten:

1. Man kann schon ein gutes Stück C++ und will sich der Vollständigkeit halber mit den Sprachdetails auseinandersetzen, die man seltener braucht. (Nach deinem ersten Code zu urteilen bist du noch lang nicht so weit)

2. Man muss etwas schreiben, was kompatibel zu C ist, z.B. um eine C-Schnittstelle zu bedienen. (Wobei selbst da in vielen Fällen vector auf der C++-Seite einen guten Job macht)

3. Ein Profiler hat einem glaubhaft machen können, dass der vector ein bottleneck ist und roher Speicher evtl. einen Tick schneller sein könnte. (Wobei man nicht damit rechnen sollte, bei rohem Speicher vs. vector viel rauszukitzeln, zumindest wenn man verantwortungsvoll mit dem vector umgeht, Stichwort reserve(). Wenn man da Performanceprobleme hat muss meist ein anderer Algorithmus her oder statischer Speicher statt Freispeicher benutzt werden, um Allokationen zu vermeiden)

Wenn ihr also im Unterricht nicht grade einiges weiter sein solltet als du scheinbar bist und Nr 1) der Grund ist, kannst du deinem Lehrer sehr gerne sagen, dass das Blödsinn ist, die Benutzung von rohem Speicher zu verlangen.

In der Schule sollen wir das ganze ja ohne globale variablen lösen.

Zb. bekomm ich die Aufgabe die Eingabe und die Ausgabe je in eine Funktion zu bekommen. Da ich aber die Pointer nicht verstanden habe, weiß ich auch nicht wie ich Array(10) in eine neue Funktion (zb. Funktion 2) bekommen würde.

Könnt ihr mir da noch ein wenig helfen? Denn das ist meine Hauptfehlerquelle, ich arbeite immer mit gloabel Variablen was ja auch nicht gut ist!

Gibt es da irgendeine Methode die man da gut verwenden könnte? Oder hat sogar vector eine spezielle Funktion dafür?

LG

void Eingabe(vector<int>* ZeigerAufVektor)
{
  //Einlesen
}
void Ausgabe(const vector<int>* ZeigerAufVektor) //const wegen const-correctness
{
  //Ausgeben
}
int main()
{
  vector<int> MeinVektor(10);
  Eingabe(&MeinVektor);
  Ausgabe(&MeinVektor);
}

Ich würde euch erst Referenzen statt Zeiger beibringen. Was für ein Prof...

vector<int> Eingabe()
{
  //Einlesen
}
void Ausgabe(const vector<int>& ReferenzAufVektor) //const wegen const-correctness
{
  //Ausgeben
}
int main()
{
  vector<int> MeinVektor = Eingabe();
  Ausgabe(MeinVektor);
}

Ausgabe(&MeinVektor);

Sorry wenn ich soviel frage, ist echt nicht so einfach wenn einem die Person nicht gegenüber sitzt ..... ^^

So siehts grad aus, aber sollte ich oben den "ZeigerAufVector nicht auch umbenennen?

void Funktion1(vector<int>* ZeigerAufVector)
{
	for (size_t i=0;i<Array.size();i++)
	{
		cout << "Bitte geben sie den Wert fuer dieses Array an: " << endl;
			cin >> Array.at(i) ;
			system("cls");
	}
}

void Funktion2(const vector<int>* ZeigerAufVektor)
{
		for (size_t i=0;i<Array.size();i++)
	{
		cout << Array.at(i) << endl;
	}
}

int _tmain(int argc, _TCHAR* argv[])
{
	vector<int> Array(10);
	Funktion1(&Array);
	Funktion2(&Array);
	system("pause");
	return 0;
}

LG

Huhu, aber kann das denn so funktionieren?

Wenn ich nichts übersehen habe, ja.

Limatex schrieb:

Da C+ aber von oben nach unten arbeitet, kennt er doch oben mein Array() noch garnicht wenn ich es unten erst deklariere, oder?

Ein Beispiel:

void Funktion(int Integer)
{
  cout << Integer; //Integer bezieht sich auf das Objekt, das als Parameter gegeben wurde
}
int main()
{
  int a = 4;
  Funktion(a); //"a" ist in der Funktion das selbe wie "Integer"
}

Limatex schrieb:

Sorry wenn ich soviel frage, ist echt nicht so einfach wenn einem die Person nicht gegenüber sitzt ..... ^^

Dafür sind wir da.

pumuckl schrieb:

Ausgabe(&MeinVektor);

sry habs korrigiert

Array.at ist aber nicht rot.

Welches "Array" meinst du nun? Irgendwie hab ich mich grad verheddert....^^

Welches "Array" meinst du nun?

Ersetzt jedes " Array " in beiden Funktionen durch " (*ZeigerAufVektor) ".

Und was bedeutet das hier in den Klammern? Ist das der "Wert" der dem Zeiger zugewiesen wird oder wie?

Funktion1(&Array);
	Funktion2(&Array);

Ist das der "Wert" der dem Zeiger zugewiesen wird oder wie?

Ja, ist es. Im Buch deiner Wahl sollte im Zusammenhang mit Zeigern auch der Adressoperator erwähnt werden.

Nachdem ich die beiden = durch == ersetzt habe waren die Klammer zwar weg, aber beim ausführen kam ein neues Fenster:

void Sortieren(const vector<int>*Zeiger) 
 { 
	 int hilf;
     for(size_t i2=0;i2<(*Zeiger).size();i2++) 
     { 
         for(size_t i=0;i<(*Zeiger).size();i++)
         { 
             if((*Zeiger).at(i+1)>(*Zeiger).at(i)) 
             { 
                 hilf=(*Zeiger).at(i); 
                 (*Zeiger).at(i)==(*Zeiger).at(i+1); 
                 (*Zeiger).at(i+1)==hilf; 
             } 
         } 
     } 
 }

http://img232.imageshack.us/img232/8312/capturemk.jpg

LG

EDIT: bzw. kommt jetzt wo ich die normalen[] Arrays zu .at() gemacht habe eine andere reihe von fenster welche aber auch etwas von out of range erzählt!