Jak elegancko przekazać liczbe do wskaźnika?

[WoodPaker]

Witajcie drodzy użytkownicy forum microgeek.eu. Mam mały problem dotyczący przekazywania argumentu do funkcji.
Funkcja jest zdefiniowana w następujący sposób:

Kod: Zaznacz cały

int wiringPiSPIDataRW (int channel, unsigned char *data, int len) ;

Jednakże ja chciałbym wysłać jeden bajt, jedną 8 bit liczbę. A robie to wszystko w innej funkcji bo przy okazji SPI muszę pomigać kilkoma innymi pin'ami. Robie to w ten sposób:

Kod: Zaznacz cały

void SendCMD(INT8U index)
{
    digitalWrite(_DC,LOW);
    digitalWrite(_CS,LOW);
    INT8U   *send=&index;
    wiringPiSPIDataRW(CHANELL, send, 1) ;
    digitalWrite(_CS,HIGH);
}


Moje pytanie brzmi. Czy można przekazać tę liczbę w sposób bardziej elegancki? Jak wpiszę

Kod: Zaznacz cały

wiringPiSPIDataRW(CHANELL, index, 1);


to mi się rzuca (i prawidłowo), że próbuję przekazać liczbę bez zamiany do wskaźnika. Ktoś? Coś?

[Antystatyczny]

A spróbuj tak:

Kod: Zaznacz cały

void SendCMD(INT8U index)
{
    digitalWrite(_DC,LOW);
    digitalWrite(_CS,LOW);
    INT8U   *send;
    *send = index;
    wiringPiSPIDataRW(CHANELL, send, 1) ;
    digitalWrite(_CS,HIGH);
}

ewentualnie tak:

Kod: Zaznacz cały

void SendCMD(INT8U *index)
{
    digitalWrite(_DC,LOW);
    digitalWrite(_CS,LOW);
    wiringPiSPIDataRW(CHANELL, index, 1) ;
    digitalWrite(_CS,HIGH);
}

[PROTON]

Wystarczy:

wiringPiSPIDataRW(CHANELL, &data, 1) ;

[Antystatyczny]

No tak, tak jest najprościej, ale sądziłem, że chodzi również o pozbycie się konieczności wpisywania parametru len

[WoodPaker]

Nie, nie. Parametr len zostaje bo funkcja wiringPiSPIDataRW jest zdefiniowana w osobnej bibliotece, do której nie mam (i nie chce mieć) dostępu. Chodziło mi raczej jak wpisać liczbę do tej funkcji wiedząc, że *data jest wskaźnikiem. Zatem czy to co teraz napisze jest poprawnie?

Kod: Zaznacz cały

void SendCMD(INT8U index)
{
    digitalWrite(_DC,LOW);
    digitalWrite(_CS,LOW);
    wiringPiSPIDataRW(CHANELL,&index, 1) ;
    digitalWrite(_CS,HIGH);
}

int main()
{
    printf("Wsylam...");
    SendCMD(233);
    printf("Wyslalem...");
    return 0;
}


[Antystatyczny]

void SendCMD(INT8U index) przyjmuje jeden parametr, a Ty przekazujesz trzy. Czegoś tu nie rozumiem

[WoodPaker]

Rozpedzilem sie... Juz poprawiłem

[Antystatyczny]

Tak, teraz powinno być dobrze

[WoodPaker]

Dziękuję Ci dobry człowieku. A powiedz mi. Pierwsza wersja też jest poprawnie tylko dłużej, prawda? (się muszę upewnić, że nie zgłupiałem do końca od czasu kiedy pisałem ostatni program)

[Antystatyczny]

Tak, pierwsza wersja niepotrzebnie ma wskaźnik oraz przypisanie do niego. To jest zbędne

[PROTON]

Spokojnie, kompilator i tak zoptymalizuje.

[WoodPaker]

OK. To jeden problem z głowy. Ale jako, że funkcja wiringPiSPIDataRW(CHANELL,&index, 1) nadpisuje bufor tym co otrzymuje od SPI to w takim razie kod

Kod: Zaznacz cały

INT8U SendCMD(INT8U index)
{
    digitalWrite(_DC,LOW);
    digitalWrite(_CS,LOW);
    INT8U *data=&index
    wiringPiSPIDataRW(CHANELL,data, 1) ;
    digitalWrite(_CS,HIGH);
    return data;
}

int main()
{
    printf("Wsylam...\n");
    INT8U data=SendCMD(233);
    printf("Otrzymalem %d\n",data);
    return 0;
}


Jest poprawny?

[WoodPaker]

INT80 jest zadeklarowany jako unsigned char (to tak dla potomnych)

[Antystatyczny]

Nie. Moim zdaniem powinieneś mieć return *data;

[WoodPaker]

OK, dziękuję. Tego mi było trzeba. Niech wam Bozia wynagrodzi w Malinkach

[Antystatyczny]

Możesz również zwrócić index. Return index;