Kleinsten Wert finden, Variablen vergleichen und auswerten

[Matze (Gast)]

Hallo,

Sitze grade vor einem unverständlichen Problem:

Von 4 Double-Variablen soll die kleinste gefunden werden.
Wenn die andrern Varaiblen um > 0,2 größer sind soll eine Aktion 
ausgeführt werden.

Es kommt jedoch überall # raus.
-->Dass sollte eigentlich garnicht möglich sein.

Seht ihr den Fehler?

void autobalance(uint8_t ueb)
{
  if (ueb==0)  //Automatisches balancing
  {
    //Finden der kleinsten Spannung
    uint8_t spg_klein=0;
    if ((spg1<spg2) && (spg1<spg3) && (spg1<spg4))
    {
      spg_klein=spg1;
    }
    else if ((spg2<spg1) && (spg2<spg3) && (spg2<spg4))
    {
      spg_klein=spg2;
    }
    else if ((spg3<spg1) && (spg3<spg2) && (spg3<spg4))
    {
      spg_klein=spg3;
    }
    else
    {
      spg_klein=spg4;
    }
    //----------Ein/Abschalten der Balancing R's
    if (spg1>(spg_klein+0.2))
    {
      zeile1[6]='#';
      balance(0,1);
    }
    else
    {
      zeile1[6]=' ';
      balance(0,0);
    }
    if (spg2>(spg_klein+0.2))
    {
      zeile1[14]='#';
      balance(1,1);
    }
    else
    {
      zeile1[14]=' ';
      balance(1,0);
    }
    if (spg3>(spg_klein+0.2))
    {
      zeile2[6]='#';
      balance(2,1);
    }
    else
    {
      zeile2[6]=' ';
      balance(2,0);
    }
    if (spg4>(spg_klein+0.2))
    {
      zeile2[14]='#';
      balance(3,1);
    }
    else
    {
      zeile2[14]=' ';
      balance(3,0);
    }
  }
}

[Edi R. (edi_r)]

Was mir aufgefallen ist: spg1 bis spg4 sollen double-Variablen sein, 
aber spg_klein ist ein uint8_t.

[Matze (Gast)]

Edi R. schrieb:
> Was mir aufgefallen ist: spg1 bis spg4 sollen double-Variablen sein,
> aber spg_klein ist ein uint8_t.

Stimmt Danke.

Doch nun wird die größte als kleinste erkannt!
Alle kleineren haben ein '#'

[Loddar (Gast)]

Matze schrieb:
> Doch nun wird die größte als kleinste erkannt!
> Alle kleineren haben ein '#'

beschreib doch mal den Fehler:
welche Werte haben die Spannungen und was wird ausgegeben und was ist an 
der Ausgabe falsch

[Matze (Gast)]

Es geht um ein Balancing-System, dabei soll die höchste Spannung nur 
0,2V größer als die anderen sein.

-->Im Bedarfsfall wird ein Parallelwiderstand geschaltet.
-->Dann soll dies durch ein '#' auf dem Display angezeigt werden.

Spg 1 = 7,88V
Spg 2 = 6,36V
Spg 3 = 6,52V
Spg 4 = 6,11V

Aktuell werden alle Spannungen außer Spg 1 mit '#' angezeigt und 
entladen.

Habe die Spannungen nun in ein Array gepackt, doch am Problem ändert 
sich nichts.

void autobalance(uint8_t ueb)
{
  if (ueb==0)  //Automatisches balancing
  {
    double spg_klein=100;
    for (int i=0;i<4;i++)
    {
      if (spg[i]<spg_klein)
      {
        spg_klein=spg[i];
      }
    }

    //----------Ein/Abschalten der Balancing R's
    if (spg[0]>(spg_klein+0.2))
    {
      zeile1[6]='#';
      balance(0,1);
    }
    else
    {
      zeile1[6]=' ';
      balance(0,0);
    }
    if (spg[1]>(spg_klein+0.2))
    {
      zeile1[14]='#';
      balance(1,1);
    }
    else
    {
      zeile1[14]=' ';
      balance(1,0);
    }
    if (spg[2]>(spg_klein+0.2))
    {
      zeile2[6]='#';
      balance(2,1);
    }
    else
    {
      zeile2[6]=' ';
      balance(2,0);
    }
    if (spg[3]>(spg_klein+0.2))
    {
      zeile2[14]='#';
      balance(3,1);
    }
    else
    {
      zeile2[14]=' ';
      balance(3,0);
    }
  }
}

[Loddar (Gast)]

Matze schrieb:
> Es geht um ein Balancing-System, dabei soll die höchste Spannung nur
> 0,2V größer als die anderen sein.

und warum suchst du dann nach der kleinsten Spannung?

[Matze (Gast)]

Loddar schrieb:
> Matze schrieb:
>> Es geht um ein Balancing-System, dabei soll die höchste Spannung nur
>> 0,2V größer als die anderen sein.
>
> und warum suchst du dann nach der kleinsten Spannung?

Hab's falsch formuliert,
Es soll die kleinste Spannung ermittelt werden.
Alle anderen Spannungen sollen so geregent werden, dass sie maximal 0,2V 
größer sind.

[Loddar (Gast)]

Matze schrieb:
> if (spg[0]>(spg_klein+0.2))
>     {
>       zeile1[6]='#';
>       balance(0,1);
>     }
>     else
>     {
>       zeile1[6]=' ';
>       balance(0,0);
>     }

ändere das doch Mal in

 if ( spg[0]==spg_klein )
       zeile1[6]='#';
     else
       zeile1[6]=' ';

usw.
um zu kontrollieren was als kleinste Spannung gefunden wird,
Programmieren besteht zu einem großen Teil in systematischer Fehlersuche 
...