3. Noobie braucht dringend Hilfe!!! (Zeiger/Zeigerübergabe)

Noobie braucht dringend Hilfe!!! (Zeiger/Zeigerübergabe)

  • S

    ok, und in der Funktion arbeite ich dann mit dem Synonym "durchschnitt", womit auch immer die Variable, die ich an die Funktion übergeben habe, global verändert wird. richtig? wobei ich ja auch gleich in der Funktion mit der globalen Variable arbeiten könnte, oder?

    0
  • Z

    Sonja80 schrieb:

    ok, und in der Funktion arbeite ich dann mit dem Synonym "durchschnitt", womit auch immer die Variable, die ich an die Funktion übergeben habe, global verändert wird. richtig? wobei ich ja auch gleich in der Funktion mit der globalen Variable arbeiten könnte, oder?

    Eigentlich schon aber globale variablen sind bööööööse.
    Kurt

    0
  • S

    Ok, dank ZuK hab ichs endgültig kapiert... THX!!!

    0
  • S

    ZuK schrieb:

    Eigentlich schon aber globale variablen sind bööööööse.

    War das ironisch gemeint?

    0
  • Z

    eigentlich nicht. denk nur daran wass passiert wenn du lokale variablen mit dem gleichen namen definierst.
    Kurt

    0
  • P

    mit Sicherheit nicht. globale Variablen sind wirklich schrecklich, sobald der Code über 20 Zeilen deutlich hinausgeht 😉

    der Clou an den Referenzen als Parameter ist ja, dass du an der Stelle des Aufrufs festlegen kann, in welche Variable deine Funktion den Durchschnitt speichern soll (muss ja nicht immer die selbe sein). Das kann durchaus auch eine sein, die irgendwo ganz versteckt in einem sehr lokalen Gültigkeitsbereich existiert (z.B. in einer anderen Funktion)

    0
  • S

    Kacke. Wollt grad die ganzen Funktionen im Hauptprogramm ausprobieren, und schon scheiterts wieder. Wird die Funktion ausgeführt, erscheint nach eingabe der Matrikelnummer die fehlermeldung, dass das Programm beendet werden musste. Ich hol gleich die Axt!!! Die Fehlersuche erzählt was von Zugriffsverletzung (Segmentation Fault)
    Mein verbesserter Quellcode dieser Funktion:

    int initialize(student *&pA)
{   
    int n;
    cout<<"Wieviel neue Studenten möchten sie zur Liste hinzufügen?";
    cin>>n;
    if (n==0)
        return 0;
    else
    {  
       for (int i=1; i<=n; i++) 
       {
        student * phelp;
        phelp = new student;
        cout << "Nachname: ";
        cin >> phelp->name;
        cout << "Vorname: ";
        cin >> phelp->vorname;
        cout << "Studienfach: ";
        cin >> phelp->fach;
        cout <<"Matrikelnummer: ";
        cin >> phelp->matrikelnummer;
        for (int j=0; j<20; j++) phelp->noten[j]=0; 
        phelp->pnext = 0;

        if ((pA==0) || (strcmp(phelp->name, pA->name)<0))
        {
            phelp->pnext = pA;
            pA = phelp;
        }
        else
    	{
        	student *prun;
        	prun = pA;

            while((prun->pnext != 0) && (strcmp(prun->pnext->name, phelp->name)<0))
        	  prun = prun->pnext;      
        	phelp->pnext = prun->pnext;
        	prun->pnext = phelp;
        }
       }   
    }     
}

    Eure geschulten Augen entdecken doch bestimmt gleich den Fehler oder?

    0
  • S

    alles klar, fehler gefunden, pA war nicht initialisiert, da ging der Vergleich schief

    0
  • W
     

    #include <iostream>
using namespace std;

struct student {
	char name[40];
	char vorname[20];
	char fach[40];
	int matrikelnummer;
	float noten[20];
	student * pNext;

	};

int initialize(student *&pA)
{
int i=0, d=0, r=0;
student *pH, *pNeu, *ptmp, *pPrev;

while (i<2) //studenten einlesen
{
pNeu=new student; //listenelement für übergabe

/* EINLESEN */
cout << endl;
cout << "Name:  ";
cin >> pNeu->name;
/*
cout << "Vorname:  ";
cin >> pNeu->vorname;

cout << "Fach:  ";
cin >> pNeu->fach;

cout << "Matrikelnummer:  ";
cin >> pNeu->matrikelnummer;
*/
//noten 0 setzen
for(int c=0; c<20; c++)
{
pNeu->noten[c]=0;
}

if(!pA)
{
//cout << "Liste noch nicht vorhanden" << endl;
pA=pNeu;
pH=pA;
pH->pNext=NULL;
}
else
{

pH=pA; //immer vorn anfangen

//WHILE SCHLEIFE
while(( (strcmp(pNeu->name, pH->name)>0) && (pH->pNext!=NULL)) ) //solange eingabe groeÃer & nicht last element
{
	pPrev=pH;
	pH=pH->pNext;
	cout << "while - d: " << d << "pNext: " << pH->pNext << endl;
	d++;
}

if(pH->pNext==NULL)
	{
	if (strcmp(pNeu->name, pH->name)<0)
	{
		if(d!=0)
		{
//		cout << "schleife" <<endl;
		pPrev->pNext=pNeu;
		pNeu->pNext=pH;
		}
		else
		{
		pNeu->pNext=pH;
		pA=pNeu;
		}
	//pPrev->pNext=pNeu;
	}
	else
	{
//	cout << "last element" << endl;

	//neues ele
	pH->pNext=pNeu;
	pH=pH->pNext; //pNeu->pNext=NULL müste auch gehen
	pH->pNext=NULL;

	//andere sachen auch übertragen
//	cout << "pH->name von pNeu :" << pH->name << endl;
	}
	}

if(pH->pNext!=NULL)
{
//	cout << "not last element" << endl;
//	cout << "pH-pNext :" << pH->pNext << endl;

	if (pA==pH)
	{
	//element kommt vor anfang
//	cout << "pH==pA" << endl;
//	cout << "pH :" << pH << endl;
//	cout << "pA :" << pA << endl;
	pNeu->pNext=pA;
	pA=pNeu;
	}
	else
	{
	pNeu->pNext=pH;
	pPrev->pNext=pNeu;

	}

}
}	
i++;
}

r=1;
return r;
}

int output(student *&pA)
{
student *pP;
int i;
pP=pA;

for(i=0;i<2;i++)
{
cout << pP->name << "\t" << pP->vorname << "\t" << pP->fach << "\t" << pP->matrikelnummer <<endl; 
pP=pP->pNext;
}

}

/*
//set_marks
int set_marks(student *const pA, int matrnr, int index, float note)
{
int r=0;
student *pP;

pP=pA;

while(pP->matrikelnummer!=matrnr && pP->pNext!=NULL)
	{
	pP=pP->pNext;
	}

if (pP->matrikelnummer==matrnr)
	pP->noten[index]=note;

if (pP->matrikelnummer!=matrnr && pP->pNext==NULL)
	r=-1;

return r;
}

//--- get_mean ---
int get_mean(student * pA, int matrnr, float & durchschnitt)
{
int r=0, anz=0;
float sum=0;
student *pP;

pP=pA;

while(pP->matrikelnummer!=matrnr && pP->pNext!=NULL)
	{
	pP=pP->pNext;
	}

if (pP->matrikelnummer!=matrnr && pP->pNext==NULL)
	r=-1;

if (pP->matrikelnummer==matrnr)
	{
	for(int i=0;i<20;i++)
		{
		if (pP->noten[i]!=0)
			{
				sum=sum+pP->noten[i];
				anz++;
			}
		}
	durchschnitt=sum/anz;

	cout << durchschnitt;
	}

return r;

}

*/
main () 
{
student *pA;
int mat, index;
int exit=0;
int eng;
//float note;

//while(exit==0)
//{
/*-- MENU -- */
cout << endl << "---------------------------" << endl;
cout << "1 - \tListe initilisieren" << endl; 
cout << "2 - \tNoten bearbeiten" << endl;
cout << "3 - \tListe ausgeben " << endl;
cout << "4 - \tDurchschnitt berechnen" << endl;
cout << "5 - \tExmatrikulieren" << endl;
cout << "6 - \tBeenden" << endl;
cout << endl << "EINGABE :";

cin >> eng;
switch(eng)
{
case 1:
		initialize(pA);
	//	if (initialize(pHelp)==0)
		cout << "FEHLER beim Aufbau der Liste!" << endl;
		//else 
		cout << "LISTE OK";
		break;
case 2:
/*		cout << endl << "Matrikelnummer :";
		cin >> mat;
		cout << "Index :";
		cin >> index;
		cout << "Note :";
		cin >> note;
	//	if (set_marks(pA,mat,index,note)==-1)
		cout << endl << "Matrikelnummer nicht vorhanden!" << endl;

		break;

case 3:
		output(pA);
		break;
case 4:
		cout << endl << "Matrikelnummer :";
		cin >> mat;
	//	if(get_mean(pA,matrnr,durchschnitt)==-1)
		cout << endl << "Matrikelnummer nicht vorhanden!" << endl;
		break;
*/				
case 6:
		exit=1;
		break;
}

//}

return 0;
}

    hi leute ich hab toller weise genau den gleichen beleg zu machen. 😉
    ich krieg ein SEGFAULT irgendwas hab ich mit den pointern falsch gemacht.

    die initializie funktion ging ohne probleme als ich aber die anderen funktionen noch geschrieben hab, kommt ein segfault. ich glaube er kommt an der stelle

    if(!pA) ....
    ich hab aber absolut keine ahnung warum
    BITTE BITTE HELFT!!!!
    DANKE martin

    0
  • ?

    pH->name ist nicht initialisiert.
    Den segmentationfault löst dann while(( (strcmp(pNeu->name, pH->name)>0)...
    aus (erstes Drittel von init...)

    sagt mein Debugger.
    Dein Quellcode lädt ja nicht gerade zum Lesen ein.

    Ich hoffe das hilft dir weiter

    0
  • ?

    pH->name ist nicht initialisiert.
    Den segmentationfault löst dann while(( (strcmp(pNeu->name, pH->name)>0)...
    aus (erstes Drittel von init...)

    sagt mein Debugger.
    Dein Quellcode lädt ja nicht gerade zum Lesen ein.

    Ich hoffe das hilft dir weiter

    0
  • W

    DANKE,
    aber nachdem ich das erste mal einen namen eingegeben hab springt er doch eigentlich in

    if(!pA)
{
//cout << "Liste noch nicht vorhanden" << endl;
pA=pNeu;
pH=pA;
pH->pNext=NULL;
}
else
{
....

    und da wird doch pH. zugewiesen.

    aber dein debugger lügt ja sicher nicht.
    wenn ich pA aber nur als pointer deklariert hab sollte
    !pA doch true sein oder?

    Danke! 🙂
    martin

    0
  • ?

    Du mußt einen Zeiger auch initialisieren. Einfach nur Student* pA zu schreiben reicht nicht. Somit erzeugst Du einen Zeiger. Welcher Wert dort steht ist nicht vorhersehbar. Das da allerdings NULL steht, ist mehr als unwahrscheinlich. Somit scheitert die Prüfung !pA. Ergo: Student* pA = NULL;. Dann klappt das auch.

    0
  • P

    gib mal bitte deinen quellcode nochmal mit den entsprechenden Einrückungen etc. So wie er dort steht wird sich das kaum einer antun... (am besten auch noch unwichtige teile auskürzen)

    0
Anmelden zum Antworten