Attiny85 Zufallswürfel

Dankeschön:))))

Code geändert:

1

#include <avr/io.h>          

2

#ifndef F_CPU

3

#define F_CPU 8000000UL

4

#endif

5

#include <util/delay.h>

6

#include <avr/interrupt.h>

7

#include <avr/sleep.h>

8

int result;

9

int i;

10

unsigned char nums[6]={0x01,0x02,0x03,0x06,0x07,0x0e};

11

int main (void) {           

12

     ADMUX=(1<<REFS2)|(1<<REFS1);

13

  ADCSRA=(1<<ADEN)|(1<<ADSC)|(1<<ADPS2)|(1<<ADPS1);

14

  while( ADCSRA & (1<<ADSC) );

15

  result = ADC;

16

    DDRB  = (0xff)&(~(1<<PB5));                    

17

  GIMSK=(1<<PCIE);

18

  PCMSK=(1<<PCINT4);

19

  set_sleep_mode(SLEEP_MODE_PWR_DOWN);

20

  sei();

21

   while(1) {  

22

         sleep_mode();     

23

   }

24

   return 0;                

25

}

26

ISR(PCINT0_vect){

27

28

  if(PINB & (1<<PB4)){

29

  for(i=0;i<20;i++){

30

    _delay_ms(1);

31

  }

32

  ADMUX = (ADMUX & ~(0x1F));

33

  ADCSRA|=(1<<ADSC);

34

  while( ADCSRA & (1<<ADSC) );

35

  result = ADC%6+1;

36

  PORTB=nums[result];

37

38

  for(i=0;i<1000;i++){

39

    _delay_ms(1);

40

  }

41

  PORTB=0x00;

42

  }

43

}

Irgendwie kommt es aber dazu, dass PB2 alleine eingeschaltet wird. Wie 
kann das sein?

Wie sieht es mit Kommentaren aus?
Bei einem Würfel, bei dem ein Knopf gedrückt wird, würde ich einfach den 
Zähler hochlaufen lassen, solange der Knopf gedrückt wird. Da hast du 
auf jeden Fall Zufallszahlen wenn das mit 10mhz passiert

Aber es erfolgt ja nur eine Zuweisung von Werten aus dem Array und 0x00. 
Da kommt ja gar keine 4 vor!Wie ist so etwas möglich? Was ist unklar? Wo 
sind Kommentare erforderlich?

Klar! Vielen Dank!!

Kommentare nur allgemein, bereits in einer Woche weißt du nicht mehr auf 
welche Einstellungen du den ADC und den PCInt gesetzt hast.
Wie oben geschrieben gibt es den arraywert nums[6] nicht.
Wie ist dein Schalter angeschlossen? Offensichtlich fragst du ab ob er 
high ist. Pulldown extern angeschlossen? Wenn ja, warum nicht den 
internen pullup benutzen? (Am Ende aufpassen dass du den pullup nicht 
ausschaltest)
_delay_ms () nimmt schon eine Weile auch größere zahlen, die Schleife 
ist nicht nötig.
Warum fragst du in der Interruptroutine, der durch pin4 ausgelöst wird, 
ob pin4 gedtückt wurde?
Die variable result ist nicht nötig, es reicht ein:

PORTB= nums[adc%6];