Programmierfehler? C Anfänger Atmega8

> In dem Programm im Anhang ist vermutlich ein Fehler.

Anhang nicht vorhanden.

1

  while(1)

2

    {

3

    if( PINB & 0x02 ) {

4

      while(1)

5

      {

6

        if( !( PINB & 0x02 ) ) {

7

          lcd_clear();

8

          while(1)

9

...

Du hast mehrere Endlosschleifen ineinander, ohne dass im Sourcecode 
irgendwo auch nur ein break steht. Das bedeutet, dass irgendwann die 
innerste Schleife endlos ausgeführt wird, ohne dass die Codeausführung 
jemals wieder zu einer der äußeren Schleifen zurückkehrt.
Ist das Absicht?

> - Das Display flimmert, ich denke die Schleife läuft immer durch und das
> Display wird immer neu beschrieben.

Richtig, wird es.

> Wie verhindere ich das?

Du könntest zusätzlich auch noch den alten Inhalt von page in einer 
Variable festhalten, und das Display nur neu beschreiben, wenn die 
beiden sich unterscheiden.

> das am anfang geht vielleicht anders besser, aber ich weiß nicht wie.

1

while (! PINB & 0x02 );

2

// hier noch ein delay wegen des Prellens

3

while (PINB & 0x02);

4

5

while (1) {

6

7

    ...

Wozu sollen denn diese Konstrukte dienen?

1

                zrck = 1;

2

                while( PINB & 0x01 ) {

3

                  // nix machen

4

                }

5

                zrck = 0;

> die dann auf den 4 LEDs binär angezeigt wird.
> ...
> - Die Dezimalzahlen werden nicht richtig angezeigt

> PORTC = page;

Kann es vielleicht sein, dass die LEDs gegen VCC angeschlossen sind?
Dann musst du page für die Ausgabe invertieren:

PORTC = ~page & 0x0f;

> lcd_data(page);

Wozu dient das?
Falls du den Inhalt von page auf das Display schreiben willst, wird 
das so nicht gehen. Du musst den Integerwert erst in einen String 
wandeln und dann auf das Display schreiben. Steht sicher auch irgendwo 
in dem LCD-Tutorial.

PS: Ach, das meintest du wohl mit "Die Dezimalzahlen werden nicht 
richtig angezeigt". ;-)

Ich selber:
> while (! PINB & 0x02 );

Ups, natürlich: while (!(PINB & 0x02));