Forum: PC-Programmierung Funktion MinElem funktioniert nicht

Kann dazu keiner was sagen?

Ich dreh noch durch. Ich verstehe einfach nicht warum ich nicht zwei min
Werte angezeigt bekomme.
Hier in diesem Forum sind doch zahlreiche Cracks oder?

Was heisst da nicht ordentlich programmiert.
Ja wie kann ich den die globalen Variablen verhindern. Die brauche ich
doch bei mir.
Hier in diesem Forum wird nur drum herum geredet.Ok es gibt bestimmt
noch 1000 elegantere Varianten. Aber ich habe nunmal ein Problem mit
meinem C-Code.Ich will nur wissen warum ich keine zwei min Werte
angezeigt bekomme?

So ich habe mal die Funktion MinElem folgendermasen angeändert:

void MinElem(int* ivector1,int* ivector2) //Funktion zum feststellen
der kleinsten Zahl
{
 int imerker_vector1=0, imerker_vector2=0;
 imerker_vector1= ivector1[0];
 imerker_vector2= ivector2[0];

  for(int i=0;i<iwerte;i++)
  {
   if (imerker_vector2>ivector2[i])
   {
    imerker_vector2=ivector2[i];
   }
 }
 printf("min.Element Vector1:%2d\n",imerker_vector2);
}

Wenn ich die Funktion so ausführe, dann bekomme ich auch immer den Wert
0 vom Vector 2 zurück.

Verändere ich diese Funktion so wie unten, dann bekomme ich einen Min.
Wert vom Vector 1 angezeigt.

void MinElem(int* ivector1,int* ivector2) //Funktion zum feststellen
der kleinsten Zahl
{
 int imerker_vector1=0, imerker_vector2=0;
 imerker_vector1= ivector1[0];
 imerker_vector2= ivector2[0];

  for(int i=0;i<iwerte;i++)
  {
   if (imerker_vector1>ivector1[i])
   {
    imerker_vector1=ivector1[i];
   }
 }
 printf("min.Element Vector1:%2d\n",imerker_vector1);
}

Wo kann denn da der Fehler sein?

Hi Tobi vielen Dank für den Tipp.

Ja wo genau müsste ich diese Variable i zurücksetzen?

Ja das mache ich doch schon.
Und zwar in Main nach den einlesen der Werte führe ich ja die Funktion
WriteVec() aus(siehe Quellcode im Anhang).
Ich kann da nix feststellen.

Ich sehe da nichts auffälliges. Ich verzweifle noch.Ich finde den Fehler
nicht.

Hmmm...ok ich kann es jetzt verstehen.Wie könnte man die beiden
Funktionen verbessern?

Ich bekomme es noch als nicht hin.
Bitte bitte helft mir. Ich muss bis morgen das Programm zum laufen
bringen.

Ich habe diesen beiden Variablen von der Funktion MinElem() mal direkt
einen Wert vergeben:

imerker_vector1= 11;
imerker_vector2= 22;

Die For Schleife habe ich auskommentiert.
Die printf Funktion liefert mir dann auch die beiden Werte zurück.

Irgendwas stimmt mit den Vectoren nicht.

grrrrrrrrrrrrrrrrrrrrrr.....

Es muss doch hier einen Crack geben, der mir genau sagen kann wo und wie
ich das Programm abändern muss, damit es läuft.
Ich kann ich noch 100 Stunden sitzen. Leider muss ich ja noch was
anderes tun.

Wenn ich ivector1[0]=55; in der Funktion einfüge und teste, dann erhalte
ich auch den min.Wert von Vector1.
Das vestehe ich absolut nicht.

void MinElem(int* ivector1,int* ivector2) //Funktion zum feststellen
der kleinsten Zahl
{
 imerker_vector1= ivector1[0];
 ivector1[0]=55;   //zusätzlich eingefügt.
 imerker_vector2= ivector2[0];
 for(int m=0;m<iwerte;m++)
 {
  if (imerker_vector1>ivector1[m])
   {
    imerker_vector1=ivector1[m];
   }
  if (imerker_vector2>ivector2[m])
   {
    imerker_vector2=ivector2[m];
   }
 }
 printf("min.Element Vector1:%2d min.Element
 Vector2:%2d\n",imerker_vector1,imerker_vector2);
}

In einem anderen Forum wurde mir das hier vorgeschlagen.

int MinElem(int *a,size_t n){
  int temp;
   for(temp =*a++;--n;++a)
    if(*a<temp)temp =*a;
return temp
}

Also das finde ich total unübersichtlich und vor allem verstehe ich das
nicht.

Mein Programm liest die Werte schon korrekt in die Vectoren ein.
Hab ich ja auch getestet.

Jetzt noch mal:

void MinElem(int* ivector1,int* ivector2) //Funktion zum feststellen
der kleinsten Zahl
{
 ivector1[0]=99;   //zusätzlich eingefügt.
 ivector2[0]=99;   //zusätzlich eingefügt.
 imerker_vector1= ivector1[0];
 imerker_vector2= ivector2[0];
 for(int m=0;m<iwerte;m++)
 {
  if (imerker_vector1>ivector1[m])
   {
    imerker_vector1=ivector1[m];
   }
  if (imerker_vector2>ivector2[m])
   {
    imerker_vector2=ivector2[m];
   }$
 }
 printf("min.Element Vector1:%2d min.Element
 Vector2:%2d\n",imerker_vector1,imerker_vector2);
}

GENAU SO FUNKTIONIERT ES JA!!!!!!

Ja und wie würdest du das machen bzw. programmieren? Ich stehe auf dem
Schlauch.

Achso hier muss ich die Variable i auf NULL setzte!?

 printf("Vector1:\n");
 for(int x=1; x<=iwerte;x++)
 {
  scanf("%2d",&izahl);        //einlesen der Zahlen in einen Vector
  ReadVec(izahl,1);
 }
 i = 0;
 printf("Vector2:\n");
 for(int x=1; x<=iwerte;x++)
 {
  scanf("%2d",&izahl);        //einlesen der Zahlen in einen Vector
  ReadVec(izahl,2);
 }