Anweisung wird in switch() nicht ausgeführt

[moks (Gast)]

Hallo zusammen. und zwar führt meine switch Funktion nicht alle 
Anweisungen aus.
Wie im code geschrieben, soll sich bei xänderung das lcd gelöscht und 
neu beschrieben werden.
mein Problem ist, der µC überschreibt einfach das Wort, ohne vorher zu 
löschen.

Woran kann das liegen.?
PS: das ClearScreen() funktioniert sonst auch immer. nur hier wird es 
übersprungen

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <stdio.h>
#include <stdbool.h>
#include "TWI_Master.h"
#include "mydefs.h"
#include "taster.h"
#include "functions.h"
#include "ks108.h"
#include "mydefs.h"
#include "fontsoft.h"
#include "draw.h"
#include "bitmap.h"
#include "abfragen.h"

volatile uint8_t x      = 0;
volatile uint8_t y      = 1;
volatile uint8_t pfeilpos  = 0;
volatile uint8_t z      = 1;
volatile char    lcd_puffer[128][8];  
void taster(void){
  if (get_key_short(1<<left)){
    x--;
  }
  if (get_key_short(1<<right)){
    x++;
  }
}

void homescreen(void){
  LCDSoftText("Spannungen",7,z1,BLACK);
  LCDSoftText("Bettleuchte",7,z2,BLACK);
}  
int main(void){
  
  lcd_init();
  init_io();
  init_timer();
  sei();
  
  
  
  while(1){
    taster();
    switch(x){
      case 0: ClearScreen; homescreen(); break;
      case 1: ClearScreen; LCDSoftText("test",0,0,BLACK); break;
    
    }
  }  
}

[Peter II (Gast)]

moks schrieb:
> Woran kann das liegen.?
> PS: das ClearScreen() funktioniert sonst auch immer. nur hier wird es
> übersprungen

ist ja kein wunder. Und wenn man sich die mühe macht, nicht alles auf 
ein Zeile zu schreiben, dann würden einem der Fehler auch auffallen.

ClearScreen; 
homescreen();

[B. S. (bestucki)]

moks schrieb:
> case 0: ClearScreen; homescreen(); break;

Klammern hinter dem ClearScreen vergessen. Dann wird nicht die Funktion 
aufgerufen, sondern nur dessen Adresse ermittelt und das Ergebnis 
verworfen. Schalte mal -Wall und -Wextra ein. Dann sollte dich der 
Compiler mit sowas wie "statement with no effect" warnen.

[moks (Gast)]

be stucki schrieb:
> Klammern hinter dem ClearScreen vergessen.

hab ich garnicht gemerkt ...
das war auch der Grund

aber:

be stucki schrieb:
> Schalte mal -Wall und -Wextra ein.

das hab ich. aber es kommt trotzdem keine warnung

[B. S. (bestucki)]

moks schrieb:
> be stucki schrieb:
>> Schalte mal -Wall und -Wextra ein.
>
> das hab ich. aber es kommt trotzdem keine warnung

Welchen Compiler benutzt du? GCC wirft eine Warnung:

void foo(void){
}

int main(void){
  foo;
  return 0;
}

[Warning] statement with no effect [-Wunused-value]

-Wall schaltet die Option -Wunused-value ein.
https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#Warning-Options

[moks (Gast)]

muss man das verstehen.?

es war ein haken bei All warnings (-wall)

hab ihn nochmal rausgenommen, wieder reingemacht, gespeichert...

jetzt gehts


danke euch. Problem wurde gelöst ;)