Fehlermeldung void/unsigned < C/C++ < Programmiersprachen < Praxis < Informatik < Vorhilfe
|
Status: |
(Frage) beantwortet | Datum: | 17:11 Do 26.03.2015 | Autor: | Auringer |
Servus,
dank dieses Forums wurden viele Fragen/Probleme beantwortet.
Doch bei diesem Problemfall finde ich oder auch meine Kommilitonen keine Lösung:
Mein Programm:
#include <stdio.h>
#include <stdlib.h>
void PrintBitLine(unsigned short us)
{
int i;
for(i=0; i<16; i++)
{void PrintBitLine
if(us & (0x8000>>i))
printf("1");
else
printf("0");
}
}
int main()
{
unsigned short us0 =1; // Hier wird das Programm noch erweitert
-> printf("d", PrintBitLine(unsigned short us0)); // ebenso hier
return 0;
}
bei dem Pfeil "->" gibt es folgende Fehlermeldung:
error: expected expression before 'unsigned'
Hoffe ihr könnt mir helfen.
Ich habe diese Frage in keinem Forum auf anderen Internetseiten gestellt.
|
|
|
|
> Servus,
> dank dieses Forums wurden viele Fragen/Probleme
> beantwortet.
> Doch bei diesem Problemfall finde ich oder auch meine
> Kommilitonen keine Lösung:
>
> Mein Programm:
>
> #include <stdio.h>
> #include <stdlib.h>
>
> void PrintBitLine(unsigned short us)
> {
> int i;
>
> for(i=0; i<16; i++)
> {void PrintBitLine
> if(us & (0x8000>>i))
> printf("1");
> else
> printf("0");
> }
>
> }
>
> int main()
> {
>
> unsigned short us0 =1; // Hier wird das Programm noch
> erweitert
>
> -> printf("d", PrintBitLine(unsigned short us0)); //
> ebenso hier
>
>
> return 0;
> }
>
>
> bei dem Pfeil "->" gibt es folgende Fehlermeldung:
> error: expected expression before 'unsigned'
>
> Hoffe ihr könnt mir helfen.
> Ich habe diese Frage in keinem Forum auf anderen
> Internetseiten gestellt.
Was genau soll denn dein Programm überhaupt tun? Also wie lautet die Aufgabenstellung?
Du legst dir zunächst die 'void'-Funktion an.
Innerhalb der for Schleife legst du erneut eine 'void' Funktion selben namens an.
Was genau möchtest du mit dem Rechtsshift >> bewirken?
in 'printf' möchtest du einen double ausgeben.
Deine Funktion ist aber 'void', wird dir also keinen Rückgabewert liefern. Du kannst die Funktion höchstens für sich in 'main' aufrufen. Dann wird deine 0 bzw. 1-er Folge geplottet.
Also ein paar mehr Infos wären nicht schlecht.
Valerie
|
|
|
|
|
Hi!
Das void PrintBitLine inmitten der Funktion ist schon seltsam. Vermutlich stört das den Compiler nicht, denn das PrintBitLine an der Stelle ist ein Pointer auf die Funktion selbst, der nicht weiter genutzt wird. Wie auch immer, das ist unnützer, und verwirrender Ballast.
Die Zeile mit dem Bit-shiften ist doch eigentlich recht klar: Da wird ein einzelnes Bit von vorn nach hinten durchgeschoben und mit der Zahl bitweise verundet. Damit wird eben getestet, ob ein einzelnes Bit gesetzt ist, oder nicht.
Das Problem ist die Zeile
> printf("d", PrintBitLine(unsigned short us0));
Das ist ne einzige Katastrophe!
1.
der erste Parameter von printf() ist das, was ausgegeben wird. Steht darin ein Prozentzeichen mit noch ein paar Zeichen dahinter, werden noch weitere Parameter erwartet, die dort eingefügt werden. Für Floats benutzt man %f, für Doubles %lf, und für ganze Zahlen %d. Hier steht nur ein d, das würde auch nur ein d auf die Konsole ausgeben.
Beispiel:
printf("pi = %.2f", 3.14159) -> Ausgabe "pi = 3.14"
printf("heute ist es %d Grad wärmer", 4) -> Ausgabe "heute ist es 4 Grad wärmer"
2.
Die Funktion PrintBitLine() gibt nichts zurück, was ausgegeben werden könnte. Das ist auch nicht nötig, sie gibt die Bitfolge ja bereits selbst aus. Hier macht die printf-Anweisung also gar keinen Sinn. Eigentlich müßte man statt dieser Zeile NUR PrintBitLine() aufrufen.
3.
Die Funktion PrintBitLine bekommt ein unsigned char als Parameter, deshalb steht unsigned char auch in der Implementierung der Funktion. Beim Aufruf macht das keinen Sinn, die Variable us0 weiß schließlich, was sie ist. Es heißt hier ganz einfach:
PrintBitLine(us0);
Nebenbei: eine FOR-Schleife kann sehr viel mehr als nur hoch oder runter zählen. Hier kann man sie benutzen, um den Code kompakter zu machen. Es macht den Code zwar schwer lesbar, aber auch sehr viel schneller. Wie es funktioniert, überlasse ich mal den eigenen Überlegungen:
for(unsigned short i=0x8000; i; i>>=1){
if(us & i){
printf("1");
}
else{
printf("0");
}
}
|
|
|
|