mehrere Rückgabewerte, PIC, zwei 7 Segment Anzeigen

[Lukas K. (totemsteiner)]

Hallo,

ich habe zwei 7 Segment Anzeigen, die ich per Tastendruck hochzählen 
lasse, funktioniert auch alles soweit ganz gut. Nun möchte ich die 
"hochzählenFunktion" gerne auslagern, mit hochzaehlen.h und 
hochzaehlen.c.

Folgendes habe ich probiert, leider funktioniert es mit meinem 
PIC16F1937 und dem Taster nicht.
hier die main.c:

int einersekunde=1 ;
int zehnersekunde=1;
int startwerteiner=0;
int startwertzehner=0;
#include "mcc_generated_files/mcc.h"
#include "hochzaehlen.h"
...//alle weiteren #includes sind natürlich vorhanden
...

while (1)
    {
        while (PORTCbits.RC1==1)
        {   
            AnzeigeEiner(einersekunde);
            AnzeigeZehner(zehnersekunde);
            if(!Taster1_GetValue())
            { 
                __delay_ms(200);
                hochzaehlen(); //hier rufe ich die Funktion auf
               
                startwerteiner=einersekunde;
                startwertzehner=zehnersekunde;
            }
        ....//usw.

hier meine hochzaehlen.h:

void hochzaehlen (int einersekunde, int zehnersekunde);

und hier die hochzaehlen.c:

#include "mcc_generated_files/pin_manager.h"



void hochzaehlen(int einersekunde, int zehnersekunde)
{            
                einersekunde++;
                if(einersekunde==11)
                {
                    einersekunde=1;
                    zehnersekunde++;
                    if(zehnersekunde==10)
                    {
                        einersekunde=1;
                        zehnersekunde=1;
                    }
                }  
               
}

Dass man mehrere Rückgabewerte über Zeiger auf die jeweiligen Adressen 
zurückgeben kann, ist mir bekannt. Leider weiß ich nicht, wie ich das 
hier anwenden kann, da ich ja eigentlich nichts zurückgeben muss oder?
Jemand eine Idee wie sich die Auslagerung realisieren lässt?
Ich arbeite mit MPLAB v.3.6.
LG
totem

[Karl M. (Gast)]

Ein Struktur mit den benötigten Daten, dann eine Variable mit diesem 
Datentyp und einen Zeiger darauf.

[Sebastian K. (sek)]

Lukas K. schrieb:


> Leider weiß ich nicht, wie ich das
> hier anwenden kann, da ich ja eigentlich nichts zurückgeben muss oder?

Naja, du willst doch die neuen Werte von einersekunde und zehnersekunde 
zurückgeben, oder nicht? So wie die hochzaehlen Funktion derzeit 
ausgestaltet ist, bewirkt sie rein praktisch garnichts. Sie zählt die 
Variablen auf dem Stack hoch, was genauso effektiv ist wie die CPU Zeit 
in einer Warteschleife zu verbraten :-)

hochzaehlen müsste so in der Art aussehen:

void hochzaehlen(int* einersekunde, int* zehnersekunde)
{            
                *einersekunde++;
                if(*einersekunde==11)
                {
                    *einersekunde=1;
                    *zehnersekunde++;
                    if(*zehnersekunde==10)
                    {
                        *einersekunde=1;
                        *zehnersekunde=1;
                    }
                }  
               
}

[Teo D. (teoderix)]

Lukas K. schrieb:
> da ich ja eigentlich nichts zurückgeben muss oder?

So schon, du veränderst so nur die Lokalen-Variablen, in der Funktion. 
Hat also keinerlei Auswirkungen nach außen!
Stichworte: "Call by Value" (das was du machst) und "Call by Reference"

http://www.lerneprogrammieren.com/blog/praxis/call-value-und-call-reference-bei-der-parameteruebergabe

[Volker S. (vloki)]

Wenn du die Variablen schon übergibst, dann musst die Adresse der 
Variablen (by reference) übergeben und nicht eine Kopie ihres Wertes. 
Die Funktion sieht dann wohl so aus

void hochzaehlen(unsigned char *esek, unsigned char *zsek)
{            
                *esek++;

Aufgerufen wird sie mit hochzaehlen(&einersekunde, ...

BTW:
Der Name der Funktion ist nicht sehr aussagekräftig. Irgendwas wird 
hochgezählt,. wichtig wäre was wir hochgezählt.

[Volker S. (vloki)]

Ach ja, int ist keine gute Wahl. Verschwendet Speicherplatz und 
Rechenzeit.
Zählen tust du auch merkwürdig ;-)
Oder zeigst du etwa bei 1 eine Null an und bei 10 eine 9?

[Lukas K. (totemsteiner)]

wow, vielen dank für die schnellen Antworten.

Probiere es direkt mal aus.

lg

[Lukas K. (totemsteiner)]

Volker S. schrieb:

> Zählen tust du auch merkwürdig ;-)
> Oder zeigst du etwa bei 1 eine Null an und bei 10 eine 9?

so ist es :)

[Teo D. (teoderix)]

Lukas K. schrieb:
> Probiere es direkt mal aus.

Besser wäre es, zu Lesen und zu Lernen!

[Lukas K. (totemsteiner)]

Teo D. schrieb:
> Lukas K. schrieb:
>> Probiere es direkt mal aus.
>
> Besser wäre es, zu Lesen und zu Lernen!

probieren geht über studieren ;)

[Teo D. (teoderix)]

Lukas K. schrieb:
> probieren geht über studieren ;)

Das hat dich aber ganz schön im stich gelassen.
Siehe: Beitrag "mehrere Rückgabewerte, PIC, zwei 7 Segment Anzeigen"
:P


PS: Las solch Sprüche lieber. Kommt hier selten gut an, trotz 
Zwinker-Smiley.

[Volker S. (vloki)]

Teo D. schrieb:
> PS: Las solch Sprüche lieber. Kommt hier selten gut an, trotz
> Zwinker-Smiley.

Hier vielleicht schon. Mir geht das Studieren ohne Probieren meist 
ziemlich schnell auf den Geist. Reines Raten und eine fertige Lösung 
(an)nehmen ohne zu irgendwas verstanden zu haben allerdings noch 
schneller ;-)

[Peter D. (peda)]

Es gibt keinen Grund, überhaupt für jedes Digit separate Variablen zu 
verschwenden.
Ein char kann bequem 0..99 zählen und ein int 0..9999.
Und nur für die Ausgabe zerlegt man es:

void out_number( char val )
{
  display_tens( val / 10 );
  display_ones( val % 10 );
}

[Volker S. (vloki)]

Peter D. schrieb:
> Es gibt keinen Grund, überhaupt für jedes Digit separate Variablen zu
> verschwenden.

Jetzt mal nicht zu schnell, sonst sind wir ja gleich bei der BCD 
Codierung von zwei Stellen in den zwei Nibbles eines Bytes ... ;-)

[Teo D. (teoderix)]

Peter D. schrieb:
> Es gibt keinen Grund, überhaupt für jedes Digit separate Variablen zu
> verschwenden.

Deine Variante frisst aber relativ viel Speicher. Könnte also doch 
Gründe geben.
(die vom TO hab ich mir nicht angesehen)