F1-Drehzahlanzeige selber programmieren

Auf keinen Fall. Das ist dann ein Missverständnis.

Es geht um folgendes: Z.B. soll ich anfangen ein Flussdiagramm zu 
erstellen. Oder das erste ist der Aufbau des Programms. Dabei sollen 
folgende Dinge beachtet werden.

Ich würde gerne von Anfang an wissen, wie man so etwas richtig angeht 
und nicht wild anfangen und zu keinem Ziel kommen, da ich mich dauernd 
in irgendwelchen Ideen verrenne.

Gruß Danielli

Mitlesa schrieb:
> Daniel C. schrieb:
>> Auf keinen Fall. Das ist dann ein Missverständnis.
>
> Das widerspricht aber in grossen Zügen dieser Aussage:
>
> Daniel C. schrieb:
>> Ich bin fast Programmier-Neuling und brauche eure Hilfe.
>
> ... denn Programmieren lernt man nicht von heute auf morgen.

Wieso sollte ich mich wiedersprechen?
"Ich bin Programmier-Neuling". Ja das bin ich. Bis auf ein paar Ein- und 
Ausgänge kann ich noch nicht sonderlich viel Programmieren und weiß bei 
meinem Projekt auch noch nicht wo man da anfange kann.

> Daniel C. schrieb:
>> Ich bin fast Programmier-Neuling und brauche eure Hilfe.
>
> ... denn Programmieren lernt man nicht von heute auf morgen.

Habe ich mit keinem Wort gesagt, oder?


Ich möchte Programmieren lernen, es aber direkt auf meine Schaltung 
anwenden, um direkt Hard- und Software zu vereinen und meinem Ziel - dem 
fertigen Progamm- näher zu kommen.

Ich werde sicherlich nicht für ein paar Anwendungen extra Lernboard für 
zig Euro kaufen, um dort das gleiche oder ähnliche Aufgaben zu machen, 
die ich auf meinem Chip und Board auch machen kann ;)


Ich möchte, wenn es um Code geht später meinen einstellen, der von euch 
Fehlerkorrigiert wird, wenn etwas nicht stimmt. Aber was ich brauche ist 
halt jemand der mir sagt, wie man dieses komplexe Programm am cleversten 
aufbaut.

Nicht böse nehmen ;)

Gruß Daniel

Ich rate dir: Fang klein an.
Versuche erst eine LED zum Leuchten zu bringen, dann mehrere, dann 
blinken.
Danach kannst du einen Sensor einlesen und den Status mit LEDs anzeigen.
Hier gibt es viele Tutorials für die simplen Dinge. Die solltest du auch 
durchlesen und selbst programmieren.

Ein Flussdiagramm kannst du machen wenn du weißt was du brauchst.
Was nützt dir das Diagramm, wenn du nicht weißt, wie sich z.B. die Timer 
verhalten. Oder wie du die LEDs ansprichst.

Und: Es gibt nicht DIE Lösung für dein Problem. Jeder wird es anders 
machen. So lange aber das Ergebnis passt, ist das OK.

Bernd N schrieb:
> Als Anfänger weisst du sicherlich schon wie man eine LED einschaltet ?

Ja, das bekomme ich hin. Das habe ich schnell verstanden.

> Anschließend machst du dich an die "Bandanzeige", Leuchtband. Bekommst
> du das schon hin ?

Wenn es darum geht, die nacheinander ein- bzw. auszuschalten: Ja.


Michael L. schrieb:
> Danach kannst du einen Sensor einlesen und den Status mit LEDs anzeigen.

Habe den LDR mal aus Spaß eingelesen und verarbeitet. Klappt auch 
einiger Maßen.

> Und: Es gibt nicht DIE Lösung für dein Problem. Jeder wird es anders
> machen. So lange aber das Ergebnis passt, ist das OK.

Das ist mir klar. Man bildet sich auch selbst eine Meinung zu dem 
Gesagten.



Mit dem "klein Anfangen" habt ihr beide recht. Das ist wahr. Ich werde 
aber nicht durch diese Tutorials schlau. Ich brauche irgendwie etwas 
angepasstere Hilfe. Super wäre z.B., wenn man zusammen sich Gedanken 
über den Programmaufbau machen kann und ob und in wieweit man alles 
bahandelt.

Oder halt irgendwas immer größer machen und erweitern. Hier merkt man 
schon an den Aussagen: Keine Ahnung :D

Z.B. würde ich gerne für Das Drehzahlband (DZB) und die Segmentanzeige 
eine Art Stapel anlegen. Aber da habe ich ebenso keine Ahnung von.
Ob das Sinn macht weiß ich natürlich auch nicht.

Bernd N schrieb:
> Dann stell doch mal den Code oder die Codeschnipsel die
> funktionieren
> hier ein und dann werden wir dir schon helfen das zu strukturieren.
>
> Wie sieht es mit der 7 Segment Anzeige aus ? Code auch schon fertig ?
>
> Wie gesagt, Schrit für Schritt und dann alles zusammenbauen. Da wird man
> dir schon helfen.


Ich habe viele Ideen probeweise umgesetzt, um die beste Lösung für mich 
zu finden. Deswegen sieht das so wüst aus.
Alles andere was ich habe ist in Arduino, aber das möchte ich nicht mehr 
nutzen.

1

#include <avr/io.h>

2

#include <util/delay.h>

3

#define F_CPU 16000000UL

4

5

/*

6

//Startbild-Zeiten(ms)

7

  #define T_SB_STATUS   50

8

  #define T_SB_FUELLEN  50

9

  #define T_SB_WARNUNG  50

10

  #define T_SB_LEEREN   50

11

  #define T_SB_ALLE     50

12

  #define T_SB_ZEICHEN  50

13

  #define T_SB_START    50*/

14

15

//Ausgänge

16

  #define NSL PB0        //Ausgang fuer Nebelschlussleuchte (PB0)

17

  #define DZB PB1        //Ausgang fuer Drehzahlband (PB1)

18

  #define SEG PB3        //Ausgang fuer Segment Display (PB3)

19

20

//Status-LEDs

21

  #define S_LED_1 PB4

22

  #define S_LED_2 PB5

23

  #define S_LED_3 PB6

24

  #define S_LED_4 PB7

25

26

//Gaenge

27

  #define DN PA4        //D/N

28

  #define NR PA5        //N/R

29

30

//Bremsen

31

  #define B PA7        //Bremse

32

  #define HB PA6        //Handbremse

33

34

int main (void) {

35

36

  DDRB = 0b11111011;      //Ausgang B0-B1, B3-B7

37

  DDRC = 0b11111111;      //Ausgang C

38

  DDRD = 0b11111111;      //Ausgang D

39

  PORTC = 0b00000000;      //Zustand Low 

40

  PORTD = 0b00000000;      //Zustand Low

41

42

  //PORTB |= (1 << NSL) | (1 << DZB) | (0 << SEG);  //Transistoren: 1 = Aus / 0 = An

43

44

  PORTB |=  (1 << PB0);    //" |= " = Aus / " &= ~ " = An

45

  PORTB |=  (1 << PB1);

46

  PORTB &= ~(1 << PB3);

47

48

  uint8_t PortCD = 0;

49

  uint8_t S_LED = 0;

50

51

  while(1) {            

52

    /*

53

    PORTB |= ((1 << PB4) | (0 << S_LED_2) | (0 << S_LED_3) | (0 << S_LED_4));

54

    _delay_ms(1000);

55

    PORTB |= ((1 << PB4) | (1 << S_LED_2) | (0 << S_LED_3) | (0 << S_LED_4));

56

    _delay_ms(1000);      

57

    PORTB |= ((1 << PB4) | (1 << S_LED_2) | (1 << S_LED_3) | (0 << S_LED_4));      

58

    _delay_ms(1000);

59

    PORTB |= ((1 << PB4) | (1 << S_LED_2) | (1 << S_LED_3) | (1 << S_LED_4));

60

    _delay_ms(1000);

61

    PORTB |= ((0 << PB4) | (1 << S_LED_2) | (1 << S_LED_3) | (1 << S_LED_4));

62

    _delay_ms(1000);

63

    PORTB |= ((1 << PB4) | (0 << S_LED_2) | (1 << S_LED_3) | (1 << S_LED_4));

64

    _delay_ms(1000);

65

    PORTB |= ((1 << PB4) | (1 << S_LED_2) | (0 << S_LED_3) | (1 << S_LED_4));

66

    _delay_ms(1000);

67

    PORTB |= ((1 << PB4) | (1 << S_LED_2) | (1 << S_LED_3) | (0 << S_LED_4));

68

    _delay_ms(1000);

69

    PORTB |= ((0 << PB4) | (0 << S_LED_2) | (1 << S_LED_3) | (1 << S_LED_4));

70

    _delay_ms(1000);*/

71

    /*

72

    PORTB &= ~(1 << S_LED_1);    //" |= " = Aus / " &= ~ " = An

73

    PORTB |=  (1 << S_LED_2);

74

    PORTB |=  (1 << S_LED_3);

75

    PORTB |=  (1 << S_LED_4);

76

    _delay_ms(1000);

77

    PORTB &= ~(1 << S_LED_1);

78

    PORTB &= ~(1 << S_LED_2);

79

    PORTB |=  (1 << S_LED_3);

80

    PORTB |=  (1 << S_LED_4);

81

    _delay_ms(1000);

82

    PORTB &= ~(1 << S_LED_1);

83

    PORTB &= ~(1 << S_LED_2);

84

    PORTB &= ~(1 << S_LED_3);

85

    PORTB |=  (1 << S_LED_4);

86

    _delay_ms(1000);

87

    PORTB &= ~(1 << S_LED_1);

88

    PORTB &= ~(1 << S_LED_2);

89

    PORTB &= ~(1 << S_LED_3);

90

    PORTB &= ~(1 << S_LED_4);

91

    _delay_ms(1000);

92

    PORTB |=  (1 << S_LED_1);

93

    PORTB &= ~(1 << S_LED_2);

94

    PORTB &= ~(1 << S_LED_3);

95

    PORTB &= ~(1 << S_LED_4);

96

    _delay_ms(1000);

97

    PORTB &= ~(1 << S_LED_1);

98

    PORTB |=  (1 << S_LED_2);

99

    PORTB &= ~(1 << S_LED_3);

100

    PORTB &= ~(1 << S_LED_4);

101

    _delay_ms(1000);

102

    PORTB &= ~(1 << S_LED_1);

103

    PORTB &= ~(1 << S_LED_2);

104

    PORTB &= ~(1 << S_LED_3);

105

    PORTB |=  (1 << S_LED_4);

106

    _delay_ms(1000);

107

    PORTB |=  (1 << S_LED_1);

108

    PORTB &= ~(1 << S_LED_2);

109

    PORTB &= ~(1 << S_LED_3);

110

    PORTB |=  (1 << S_LED_4);

111

    _delay_ms(1000);

112

    PORTB |=  (1 << S_LED_1);

113

    PORTB |=  (1 << S_LED_2);

114

    PORTB &= ~(1 << S_LED_3);

115

    PORTB &= ~(1 << S_LED_4);

116

    _delay_ms(1000);*/

117

    PORTB |=  (1 << S_LED_1);

118

    PORTB |=  (1 << S_LED_2);

119

    PORTB |=  (1 << S_LED_3);

120

    PORTB |=  (1 << S_LED_4);

121

    _delay_ms(1000);

122

123

    /*S_LED = 1;

124

    _delay_ms(1000);

125

    S_LED = 2;

126

    _delay_ms(1000);

127

    S_LED = 3;

128

    _delay_ms(1000);

129

    S_LED = 4;

130

    _delay_ms(1000);

131

    S_LED = 5;

132

    _delay_ms(1000);*/

133

134

    if (PortCD >= 16) {

135

      PortCD = 0;

136

    }

137

    else {

138

      PortCD++;

139

    }

140

141

    switch (PortCD) {

142

      case 1:    PORTC = 0b00000001; PORTD = 0b00000000;

143

      break;

144

      case 2:    PORTC = 0b00000011; PORTD = 0b00000000;

145

      break;

146

      case 3:    PORTC = 0b00000111; PORTD = 0b00000000;

147

      break;

148

      case 4:    PORTC = 0b00001111; PORTD = 0b00000000;

149

      break;

150

      case 5:    PORTC = 0b00011111; PORTD = 0b00000000;

151

      break;

152

      case 6:     PORTC = 0b00111111; PORTD = 0b00000000;

153

      break;

154

      case 7:     PORTC = 0b01111111; PORTD = 0b00000000;

155

      break;

156

      case 8:     PORTC = 0b11111111; PORTD = 0b00000000;

157

      break;

158

      case 9:     PORTC = 0b11111111; PORTD = 0b00000001;

159

      break;

160

      case 10:    PORTC = 0b11111111; PORTD = 0b00000011;

161

      break;

162

      case 11:    PORTC = 0b11111111; PORTD = 0b00000111;

163

      break;

164

      case 12:    PORTC = 0b11111111; PORTD = 0b00001111;

165

      break;

166

      case 13:    PORTC = 0b11111111; PORTD = 0b00011111;

167

      break;

168

      case 14:    PORTC = 0b11111111; PORTD = 0b00111111;

169

      break;

170

      case 15:    PORTC = 0b11111111; PORTD = 0b01111111;

171

      break;

172

      case 16:    PORTC = 0b11111111; PORTD = 0b11111111;

173

      break;

174

      default:    PORTC = 0b00000000; PORTD = 0b00000000;

175

      break;

176

    }

177

    /*

178

    switch (S_LED) {

179

      case 1:     PORTB &= ~(1 << S_LED_1); PORTB |=  (1 << S_LED_2); PORTB |=  (1 << S_LED_3); PORTB |=  (1 << S_LED_4);

180

      break;

181

      case 2:     PORTB &= ~(1 << S_LED_1); PORTB &= ~(1 << S_LED_2); PORTB |=  (1 << S_LED_3); PORTB |=  (1 << S_LED_4);

182

      break;

183

      case 3:     PORTB &= ~(1 << S_LED_1); PORTB &= ~(1 << S_LED_2); PORTB &= ~(1 << S_LED_3); PORTB |=  (1 << S_LED_4);

184

      break;

185

      case 4:     PORTB &= ~(1 << S_LED_1); PORTB &= ~(1 << S_LED_2); PORTB &= ~(1 << S_LED_3); PORTB &= ~(1 << S_LED_4);

186

      break;

187

      case 5:     PORTB |=  (1 << S_LED_1); PORTB &= ~(1 << S_LED_2); PORTB &= ~(1 << S_LED_3); PORTB &= ~(1 << S_LED_4);

188

      break;

189

      case 6:       PORTC = 0b00111111; PORTD = 0b00000000;

190

      break;

191

      case 7:       PORTC = 0b01111111; PORTD = 0b00000000;

192

      break;

193

      case 8:       PORTC = 0b11111111; PORTD = 0b00000000;

194

      break;

195

      case 9:       PORTC = 0b11111111; PORTD = 0b00000001;

196

      break;

197

      case 10:       PORTC = 0b11111111; PORTD = 0b00000011;

198

      break;

199

      case 11:       PORTC = 0b11111111; PORTD = 0b00000111;

200

      break;

201

      case 12:       PORTC = 0b11111111; PORTD = 0b00001111;

202

      break;

203

      case 13:       PORTC = 0b11111111; PORTD = 0b00011111;

204

      break;

205

      case 14:       PORTC = 0b11111111; PORTD = 0b00111111;

206

      break;

207

      case 15:       PORTC = 0b11111111; PORTD = 0b01111111;

208

      break;

209

      case 16:       PORTC = 0b11111111; PORTD = 0b11111111;

210

      break;

211

      default:    PORTB |=  (1 << S_LED_1);

212

            PORTB |=  (1 << S_LED_2);

213

            PORTB |=  (1 << S_LED_3);

214

            PORTB |=  (1 << S_LED_4);

215

      break;

216

    }*/

217

218

  }

219

220

  return 0;

221

222

}

neuer PIC Freund schrieb im Beitrag #4890873:
> Du kannst fast alle LEDs rausschmeißen, bis auf 2. Eine für
> "langsam", die andere für "schnell".

Welche LEDs sind gemeint?


> Der LM2907 generiert ein Analogsignal. Damit
> gehst du an einen digitalen Eingang. Das ist bestimmt anders gewollt.

Ist nur als Reckeck-Signalwandler gedacht, ja. Das ist etwas anders 
gewollt.
Ich hoffe das geht so trotzdem :D


> Und kick D22 raus.

Warum das?

Stefan U. schrieb im Beitrag #4891075 im Forum "ATmega-Programmierer 
gesucht":
> Sieht aus wie meine Arbeiten :-)

Danke für dein Kompliment :)


> Für den Anfang solltest du mal berichten,
>
> - wie so deine Programmierkenntnisse sind
> - ob du engliche Datenblätter lesen kannst
> - mit welchem Betriebsystem du arbeitest
>
> Hast du schonmal ein AVR Tutorial durchgearbeitet? Mit welchen
> Funktionseinheiten (Ports, ADC, Interrupts, Timer, etc) hast du dich
> vertraut gemacht? Welche hast du nur mal kurz ausprobiert aber nicht
> weiter vertieft?

Meine Programmierkenntnisse beschränken sich auf Pinbennenung, als Ein- 
oder Ausgang deklarieren, sowie auf Low und High setzen.

Ich kann englisch lesen, kann aber mit den vielen Begriffen in einem 
Datenblatt schwer umgehen. Würde es aber irgendwie hinbekommen, wenn ich 
weiß, wo ich suchen muss.

Ich habe Arduino kennen gelert und arbeite jetzt, durch die 
Verständlichkeit mit anderen, jetzt in AtmelStudio in C, bzw. GCC (wenn 
das einen Unterschied macht).

Ich habe Teile des Tutorials durchgearbeitet. Leider ist nicht alles auf 
meinem Board anwendbar, wie Hard-PWM über PWM-Pin. Da ich aber nicht 
extra dafür ein Board kauefen möchte, bleibt es bei lesen und versuchen 
zu verstehen.

Die Funktionseinheiten Ports und ADC habe ich kennen gelernt und kann 
diese anwenden. Interrupts und Timer übersteigen im Moment meine 
Fähigkeiten.


> Es gibt viele Anleitungen, wo man das Programmieren anhand kleiner
> einfacher Schaltungen (meistens auf Steckbrett, manchmal eine
> vorgegebene Platine) erlernt. Du hast hier eine eigene Schaltung.
>
> Nun ist es so, dass meistens eine Person mit viel Zeiteinsatz ein
> Tutorial schteibt, damit es viele nutzen.
>
> In deinem Fall brauchst du wohl Anleitungen von vielen für eine einzige
> persönliche Schaltung. Hier stimmt für alle betroffenen außer Dir das
> Verhältnis von Aufwand zu Nutzen nicht.
>
> Deswegen frage ich Dich, ob du bereit bist, deine Schaltung erstmal
> beiseite zu legen und das Programmieren anhand von vorgegebenen
> Schaltungen und Programmfragmenten zu üben. Du kannst dich ja gezielt
> auf die Teile konzentrieren, die für dein konkretes Ziel hilfreich sind.
> Und nach dem Tutorial stellst du in einem neuen Thread konkrete Fragen
> zur Umsetzung des Gelernten auf DEINE Schaltung. Was hälst du davon?

Das ist richtig. Nur leider möchte ich das Programmieren direkt auf 
meinem Board üben und anwenden. Es ist auch nicht so, dass ich garnichts 
verstehe, oder schwer hinterher komme. Es ist nur einfach so, dass ich 
ein wenig Starthilfe brauche und denke, dass ich das auch so hinbekomme.

D.h. im Endeffekt: Nein. Ich möchte ungerne meine Schaltung jetzt 
beiseite legen und dann für ein halbes Jahr nur an einem Board 
rumbasteln, wofür mir eigentlich viele Teile fehlen. Mein Board kann die 
gleichen Dinge, nur lerne ich direkt für mein ganzes Projekt und nicht 
für einen simulierten Fall. Oder sehe ich das falsch.


> Das waren viele Fragen. Beantworte sie, dann wird es sicher konkrete
> Vorschläge für Anleitungen, Tutorial, Bücher, Baukästen, ... hageln.

Danke für deine ausführliche Fragerunde. Ich hoffe ich konnte alles das 
beantworten, was gefragt war. :)

Gruß Danielli

Bernd N schrieb:
> Soll der Drehzahlbereich auf dem Bargraphen über den Bereich 0 - 10000
> auf 15 LEDs dargestellt werden oder über den Bereich der 5 Gänge ? Bei 5
> Gängen wäre der Bereich ja je 0 - 2000 Umdrehungen pro Gang.

Beides. Entweder oder. ;)

Ich habe dam mal was vorbereitet, bin damit aber selbst überhaupt nicht 
einverstanden. Geht sowas nicht übersichtlicher?
Die Werte hauen teilweise auch noch nicht hin, aber das ist ja erstmal 
garnicht wichtig.

1

//Variablen deklarieren

2

  uint8_t GArt;

3

  uint8_t Gang;

4

  uint16_t Drehzahl;

5

  uint16_t n1;

6

  uint16_t n2;

7

8

void DZB-Anzeige (uint8_t ALED)    //Drehzahl-Einstellungen

9

{

10

  If (GArt = 0) {          //Gang D-R

11

12

    if (Drehzahl >=1000) {

13

      ALED = 1

14

    }

15

    else if (Drehzahl >=1100) {

16

      ALED = 2

17

    }

18

    else if (Drehzahl >=1200) {

19

      ALED = 3

20

    }

21

    else if (Drehzahl >=1500) {

22

      ALED = 4

23

    }

24

    else if (Drehzahl >=2000) {

25

      ALED = 5

26

    }

27

    else if (Drehzahl >=3000) {

28

      ALED = 6

29

    }

30

    else if (Drehzahl >=4000) {

31

      ALED = 7

32

    }

33

    else if (Drehzahl >=5000) {

34

      ALED = 8

35

    }

36

    else if (Drehzahl >=5500) {

37

      ALED = 9

38

    }

39

    else if (Drehzahl >=6000) {

40

      ALED = 10

41

    }

42

    else if (Drehzahl >=6500) {

43

      ALED = 11

44

    }

45

    else if (Drehzahl >=7000) {

46

      ALED = 12

47

    }

48

    else if (Drehzahl >=7500) {

49

      ALED = 13

50

    }

51

    else if (Drehzahl >=8000) {

52

      ALED = 14

53

    }

54

    else if (Drehzahl >=8500) {

55

      ALED = 15

56

    }

57

  }

58

59

  else If (GArt = 1) {          //Gang 1-5

60

61

    if (Gang = 1) {

62

      n1 = 250

63

64

      if (Drehzahl >=1000) {

65

        ALED = 1

66

      }

67

      else if (Drehzahl >=1100) {

68

        ALED = 2

69

      }

70

      else if (Drehzahl >=1250) {

71

        ALED = 3

72

      }

73

      else if (Drehzahl >=(1000+(2*n1))) {

74

        ALED = 4

75

      }

76

      else if (Drehzahl >=(1000+(3*n1))) {

77

        ALED = 5

78

      }

79

      else if (Drehzahl >=(1000+(4*n1))) {

80

        ALED = 6

81

      }

82

      else if (Drehzahl >=(1000+(5*n1))) {

83

        ALED = 7

84

      }

85

      else if (Drehzahl >=(1000+(6*n1))) {

86

        ALED = 8

87

      }

88

      else if (Drehzahl >=(1000+(7*n1))) {

89

        ALED = 9

90

      }

91

      else if (Drehzahl >=(1000+(8*n1))) {

92

        ALED = 10

93

      }

94

      else if (Drehzahl >=(1000+(9*n1))) {

95

        ALED = 11

96

      }

97

      else if (Drehzahl >=(1000+(10*n1))) {

98

        ALED = 12

99

      }

100

      else if (Drehzahl >=(1000+(11*n1))) {

101

        ALED = 13

102

      }

103

      else if (Drehzahl >=(1000+(12*n1))) {

104

        ALED = 14

105

      }

106

      else if (Drehzahl >=(1000+(13*n1))) {

107

        ALED = 15

108

      }

109

    }

110

111

    else if (Gang = 2) {

112

      n2>=4500

113

    }

114

115

    else if (Gang = 3) {

116

      n2=6000

117

    }

118

119

    else if (Gang = 4) {

120

      n2=7500

121

    }

122

123

    else if (Gang = 5) {

124

      n2=9000

125

    }  

126

127

128

    if (Drehzahl >=(n2+00) {

129

      ALED = 1

130

    }

131

    else if (Drehzahl >=(n2+100) {

132

      ALED = 2

133

    }

134

    else if (Drehzahl >=(n2+200) {

135

      ALED = 3

136

    }

137

    else if (Drehzahl >=(n2+300) {

138

      ALED = 4

139

    }

140

    else if (Drehzahl >=(n2+400) {

141

      ALED = 5

142

    }

143

    else if (Drehzahl >=(n2+500) {

144

      ALED = 6

145

    }

146

    else if (Drehzahl >=(n2+600) {

147

      ALED = 7

148

    }

149

    else if (Drehzahl >=(n2+700) {

150

      ALED = 8

151

    }

152

    else if (Drehzahl >=(n2+800) {

153

      ALED = 9

154

    }

155

    else if (Drehzahl >=(n2+900) {

156

      ALED = 10

157

    }

158

    else if (Drehzahl >=(n2+1000) {

159

      ALED = 11

160

    }

161

    else if (Drehzahl >=(n2+1100) {

162

      ALED = 12

163

    }

164

    else if (Drehzahl >=(n2+1200) {

165

      ALED = 13

166

    }

167

    else if (Drehzahl >=(n2+1300) {

168

      ALED = 14

169

    }

170

    else if (Drehzahl >=(n2+1400) {

171

      ALED = 15

172

    }

173

  }

174

}

Daniel C. schrieb:
> If (GArt = 0) {          //Gang D-R
>
>     if (Drehzahl >=1000) {
>       ALED = 1
>     }
>    ...

Das haut von vorne bis hinten nicht.
* Da fehlen überall die abschliessenden ';'-Zeichen
* ALED ist definiert als eine Eingabe-parameter, wird aber in der 
Funktion gesetzt

Stephan schrieb:
> wie wäre es mit einem Zeiger und einer Tabelle wo die Werte für
> ALED und
> n2 drin stehen ?

Kannst du kurz ein Beispiel zeigen?

Eric B. schrieb:
> Daniel C. schrieb:
>> If (GArt = 0) {          //Gang D-R
>>
>>     if (Drehzahl >=1000) {
>>       ALED = 1
>>     }
>>    ...
>
> Das haut von vorne bis hinten nicht.
> * Da fehlen überall die abschliessenden ';'-Zeichen
> * ALED ist definiert als eine Eingabe-parameter, wird aber in der
> Funktion gesetzt

Nicht im ernst, oder? :D
Wie müsste es denn richtig sein?


Axel R. schrieb:
> Wo geht PIN1 vom XCON9 hin?
> Hast Du den Schaltplan gezeichnet?
> Kann man bitte diese Art von Blockschaltbild nochmal in eine Version
> bringen, mit der alle etwas anfangen können?
> Ich mein, muss ja nicht. Ich würde mich freuen.
> StromTuner

Über die Beschriftung PB1 an den Controller-Pin PB1.
PB3 ist leider falsch beschriftet :D

Ist das so schlimm gezeichnet? :P

Axel R. schrieb:
>> Über die Beschriftung PB1 an den Controller-Pin PB1.
> Du siehst selbst nicht durch, scheint es fast...
>
> Links ist ein Q1 (2x) blau eingerahmt zu finden.
> X108-1 geht auf PB1, von dort auf PBT von dort auf PBC und von dort nach
> PB1?
> der Portpin allein schafft es nicht, die LEDs zu treiben.
> MIr jedenfalls, das ist nur ein ganz kleine wenig böse gemeint, ist das
> zu umständlich, mich dort durchzuarbeiten, um Dich gegegenenfalls auf
> Fehler hinweisen zu können.
> Daher bin ich raus.
>
> StromTuner

Sry, für mich war es logisch, da ich den Plan entworfen habe :D

Der Controller-Pin PB1 schaltet über Q1 (PBC) die Basis des Transistors.
Dieser Transistor gibt dann über Emitter nach Collektor 5V (PBT) auf den 
Pin 1, XCON9 (PB1).

Das ist so gezeichnet, damit man bzw. ich direkt erkenne, wo der Pin des 
Controllers eigentlich hingeht. Dass dazwischen ein Transistor liegt, 
der für das Schalten zuständig ist, interessiert mich eigentlich nicht 
mehr :D
Sry, hört sich doof an.


Axel R. schrieb:
> Oder so. man kann das doch bitte thematisch etwas aufbereiten, gerde
> wenn andere so nett sein sollen, sich das anzusehen. menno :\
>
> Vorhin hatte ich mich noch aufgeregt, weil jemand einen anderen genau
> deswegen angefertzt hatte. War dort nicht ganz so schlimm. dafür hatte
> jedes Netz eine andere Farbe :)
>
> Ich lese noch n bissl mit...
>
> StromTuner

Das stimmt natürlich. Ich werde versuchen, in der nächsten Zeit ein paar 
Änderungen vorzunehmen. Bitte etwas Geduld. Aber du hast recht. Der 
Schaltplan ist zweckmäßig, aber nicht gut leserlich.

Daniel C. schrieb:
>> * ALED ist definiert als eine Eingabe-parameter, wird aber in der
>> Funktion gesetzt
>
> Nicht im ernst, oder? :D
> Wie müsste es denn richtig sein?

Für die Rückgabe genau eines Wertes bietet sich immer der normale
Funktions-Rückkehrwert an.  Du deklarierst deine Funktion also
nicht "void" als Ergebnistyp, sondern "uint8_t".  Dann merkst du
dir entweder in einer Variablen den fertigen Wert:

1

uint8_t DZB_Anzeige (void)

2

{

3

  // ...

4

  uint8_t rv; // return value

5

6

  // ...

7

  if (...) {

8

     rv = 42;

9

  }

10

11

  // ...

12

  return rv;

13

}

Alternativ kannst du auch „mitten aus der Funktion springen“.  Das
empfinden manche zwar als schlechten Programmierstil (eine Funktion
sollte nur einen Eintritt- und einen Austrittspunkt haben), aber
es hat zusätzlich den Vorteil, mit den vielen geschachtelten "if"s
aufzuräumen:

1

   if (bedingung_a) return 13;

2

   if (bedingung_b) return 26;

Da durch das "return" der nachfolgende Code an dieser Stelle nicht
mehr interessiert, braucht man keine weiteren "else"-Zweige.

Der Aufrufer muss dann mit dem so zurückgegebenen Wert etwas
anfangen:

1

   var1 = DZB_Anzeige();

2

   // hier den Wert in "var1" weiterverarbeiten

Bernd N schrieb:
> Stimmt aber ich habe den Eindruck er weiss selber nicht wie und daher
> wäre mir eine Beschreibung (hatte ich gefragt) lieber als der Code.
> Jedenfalls ist seine Antwort für mich nicht klar.

Oh entschuldige bitte. Dann haben wir aneinander vorbei geredet.

Als Antwort: Es gibt zwei verschiedene Arten der Drehzahlanzeige.
Einmal für die Gänge D, N und R, evtl auch P wird das DZB 
(Drehzahlband), die 15 LEDs, im kompletten Drehzhalbereich angezeigt. 
Grob gesagt 0-10.000, allerdings in meinem Fall irgendwann 1.000 - ca. 
9.000 bis 9.500.

In der Zweiten Variante wird der Gang D durch die fiktive Gänge 1-5 
ersetzt. Diese wiederrum haben einen eigenen Drehzahlbereich, je nach 
Gang.
Der 1. z.B. von 1.000 - 4.500 und der 2. von 4.500 bis 6.000, u.s.w.

Ist das die Antwort zu der Frage, oder soll ich das komplett 
aufschreiben, wie es im Code ist, nur ohne Code? Wäre kein Problem.

Bernd N schrieb:
>>> a kann man das Array zweidimensional anlegen
> Soweit ist er noch nicht.

Ist glaub' ich auch bisschen viel für den Anfang.  Er sollte wohl
erstmal mit einem eindimensionalen arbeiten und sich auf einen
Gang konzentrieren.

Kommt beim AVR dann noch hinzu, dass man solche Tabellen lieber in
den Flash legen sollte, damit sie keinen RAM schlucken.

Der Anfang wäre wohl sowas:

1

#define NLEDS 15

2

uint8_t aled(int Drehzahl)

3

{

4

  static const __flash int limits[NLEDS] =

5

  {

6

    1000, 1100, 1200, 1500, 2000,

7

    3000, 4000, 5000, 5500, 6000,

8

    6500, 7000, 7500, 8000, 8500

9

  };

10

  uint8_t i;

11

  for (i = 0; i < NLEDS; i++)

12

    if (Drehzahl >= limits[i])

13

       return i + 1;

14

  return 0;  // < 1000

15

}

> Der Anfang wäre wohl sowas:
> #define NLEDS 15
> uint8_t aled(int Drehzahl)
> {
>   static const __flash int limits[NLEDS] =
>   {
>     1000, 1100, 1200, 1500, 2000,
>     3000, 4000, 5000, 5500, 6000,
>     6500, 7000, 7500, 8000, 8500
>   };
>   uint8_t i;
>   for (i = 0; i < NLEDS; i++)
>     if (Drehzahl >= limits[i])
>        return i + 1;
>   return 0;  // < 1000
> }

Würde das bedeuten, dass die Schleife nie unterbrochen wird, weil die 
Drehzahl nie unter 1.000 fällt?
Also ich kann mir ungefähr denken, was da laufen soll.

Das gefällt mir sogar richtig gut. So würde zu mindest für Gang D eine 
große Übersichtlichkeit geschaffen. Danke

Daniel C. schrieb:
>> Das haut von vorne bis hinten nicht.
>> * Da fehlen überall die abschliessenden ';'-Zeichen
>> * ALED ist definiert als eine Eingabe-parameter, wird aber in der
>> Funktion gesetzt
>
> Nicht im ernst, oder? :D
> Wie müsste es denn richtig sein?

Nein, ich werde dir dein Code nicht schreiben, das musst du schon selber 
tun.

Die Funktion könnte z.B. die Drehzahl als Eingabeparameter haben, sich 
dann ausrechnen wieviele LED aufleuchten sollen und zuletzt eine 
Anzeige-funktion aufrufen die dann die tatsächliche LED-Ansteuerung 
macht.

Mein Ratschlag wäre: lass dieses Projekt erstmal liegen und lerne 
C-Grundlagen. Die beherrscht du offensichtlich noch überhaupt nicht.

Daniel C. schrieb:
> Würde das bedeuten, dass die Schleife nie unterbrochen wird, weil die
> Drehzahl nie unter 1.000 fällt?

Nein.

Es wird das Array durchgegangen, bis die Drehzahl größer als einer
der Werte ist.  Ist sie jedoch nicht größer als irgendeiner der
Werte, würde die Schleife zum normalen Ende gelangen.  Das kann
dann eigentlich nur bedeuten, dass die Drehzahl bereits kleiner
als der erste Wert war.

Natürlich könnte man diesen Fall auch gleich am Anfang abfangen:

1

   if (Drehzahl < limits[0])

2

      return 0;

3

   for ...

> So würde zu mindest für Gang D eine große Übersichtlichkeit geschaffen.

Ich würde das an deiner Stelle dann auch für die anderen Gänge in
ähnlicher Weise schreiben und auf die Berechnungen im Stile von
„Drehzahl >=(1000+(7*n1))“ ganz und gar verzichten.  Die verschiedenen
möglichen Werte von „n1“ sind doch letztlich auch wieder vorab bekannt,
sodass man die Tabellen vollständig auscodieren kann.  Wenn du diese
in den Flash legst (wie mit dem Schlüsselwort __flash gezeigt), dann
stört der Speicherverbrauch mehrerer Tabellen auch kaum.

Theor schrieb:
> Jörg: In Deinem Fall ziemlich sich ein Lapsus.

Stimmt. ;-)

Irgendeinen leisen Zweifel hatte ich schon im Hinterkopf, ob das
denn so OK wäre … aber nicht näher drüber nachgedacht.

Daniel soll ja programmieren lernen :-), naja, ihm gleich ein paar
Bugs vorzulegen ist sicher nicht so schön, andererseits hoffe ich
ja, dass er über kurz oder lang dazu kommt, darüber nachzudenken,
was wir ihm denn so als Tipps auf die Reise geben, um das auch alles
zu verstehen.

Bernd N schrieb:
> Den Code habe ich dir geliefert und nun sag mir ob du den Code
> verstanden hast ? es ist wichtig das du jetzt am Verständnis arbeitest.

Hex ist noch nicht in meinem Schädel verankert. Ich arbeite 
Übersichtliher weise, damit ich sehe, was ich da mache mit der 
Binär-Schreibweise.

Daher komme ich ein bisschen mit dem Hex durcheinander.
Aber ich kann mir denken, was gemeint ist.

Muss man dort mit 0b bzw. 0x arbeiten oder kann man sogar auch die 
definierten Pinnamen verwenden?


> Gib der Funktion einen weiteren Parameter mit und mach dann die
> Fallunterscheidungen bei der Berechnung.
> void BargraphAnsteuerung (uint16_t Drehzahl, uint8_t Gang)
> {...

Ich glaube das übersteigt jetzt meine Leistung. Das überfordert mich 
grade doch ein wenig mehr.

Eine Unterscheidung bringt doch auch garnichts, da ich doch nur die 15 
Pins für die LEDs ansteure, oder nicht? Wenn nein, habe ich es nciht 
verstanden

Voller Eifer hast Du schon angefangen Code zu schreiben. Du bist also 
bestens motiviert. Das ist gut, da geht was. Hat aber einen Haken. bei 
einem Projekt mit der von Dir angestrebten Komplexität sollte man 
erstmal ein bischen Papier produzieren.

Im Anhang ein Beispiel wie ich meine Software organisiere, am Beispiel 
der 8 Kanal Lichtsteuerung mit Triacs an welcher ich gerade sitze.
Zustandsdiagramme & Co. sind bei mir Pflicht sonst finde ich 
systematische Fehler nicht. Zu jedem Block gibt es noch eine kurze 
Beschreibung was wann und wie getan wird.

Die von Dir bereits skizzierten Teilaufgaben habe ich auch bei mir. 
Sogar noch feiner aufgeschlüsselt.

Warum das Ganze:
Ich kann einen Block umschreiben, Fehler beheben, neue Funktionen 
hinzufügen und dabei genau sehen welche Abhängigkeiten bestehen.

Mögliche Beispiele bei Dir:

Block Drehzahl einlesen,
Block Geschwindigkeit einlesen,
Block Gangberechnung,
...

Warum das? Weisst Du vorher wieviel Störungen auf welchem Signal sind 
und ob da nicht noch ein Filter rein muss?

viel Erfolg
hauspapa

So müsste es nach meinem Verständnis aussehen.
Jetzt ist meine Frage ob ich das nach dem Prinzip von Jörg Wunsch an die 
Drehzahl anpassen kann, obwohl es mir wesentlich besser gefällt, wenn 
ich für jeden Gang das Drehzahlband anpassen kann.

1

void Drehzahlband (uint8_t AnzahlLeds)

2

{

3

  const uint16_t BarGraph [16] = {

4

    0x00, 0x01, 0x03, 0x07, 0x0F, 0x1F, 0x3F, 0x7F, 0xFF,        //Aus, PC0 - PC7

5

    0x1FF, 0x3FF, 0x7FF, 0xFFF, 0x1FFF, 0x3FFF, 0x7FFF, 0xFFF      //PD0 - PD7

6

  };

7

  PORTC |= BarGraph [AnzahlLeds] & 0xFF;          //PortC An?!

8

  PORTD |= BarGraph [AnzahlLeds] >> 8;          //PortD An?!

9

}

10

11

void Warnung (uint8_t AnzahlLeds)

12

{

13

  const uint16_t BarGraph [2] = {

14

      0b101010101010101, 0b010101010101010

15

  };

16

  PORTC |= BarGraph [AnzahlLeds];            //PortC An?!

17

  PORTD |= BarGraph [AnzahlLeds];            //PortD An?!

18

}

Stefan U. schrieb:
> Ich habe ein
> bisschen den Eindruck, das du dich selbst zu sehr unter Druck stellt und
> irgendwie einen ganzen Wald auf einmal pflanzen willst, anstatt dich mit
> einzelnen Bäumen zu beschäftigen

Das muss ich auch, weil ich bis spätestens Mitte des Jahres fertig sein 
will. Allerdings habe ich zurzeit nicht die Möglichkeit mich voll auf 
die arbeit zu konzentrieren. Ich versuche nebenbei Stück für Stück daran 
zu arbeiten.

Deswegen auch diese Passivität. Es wird besser, allerdings erst ab Mitte 
März. Dann geht es richtig zur Sache. Bis dain versuche ich schon mal 
die ersten einfachen Schritte mit eurer Hilfe aufzubauen.

Danke dafür. :)

Daniel C. schrieb:
> Bis dain versuche ich schon mal die ersten einfachen Schritte mit eurer
> Hilfe aufzubauen.

Probier' einfach viel aus.

Cyborg schrieb:
> Daniel C. schrieb:
>> Aber eine Bitte habe ich: Keine blöden Kommentare.
>
> Wer eine vernünftige Frage stellt, braucht das auch nicht befürchten.
> Ist deine Frage vernünftig? Ein Forum kann Einzelfragen oder Problemen
> mit Details sicher helfen, obwohl die Fachkompetenz der User alles
> andere als Verbindlich ist. Was du da vor hast ist ein Projekt und das
> übersteigt die Möglichkeiten des Forums um einiges. Auch machen die
> Leute hier nicht deinen Job, insbesondere wenn du damit Geld verdienst.

Ich bekomme im Moment von den Leuten, die hier schreiben, genau die 
Hilfe die ich mir erhofft habe und die mich weiter bringt. Also wo ist 
das Problem? Wenn irgendwann keiner mehr Bock hat hier was zu schreiben, 
werde ich das schn merken.

Gut, dass du besser über das Forum bescheid weißt, als die User in dem 
Forum selbst. Es gibt scheinbar Leute die mir helfen und helfen wollen, 
obwohl es die Forums-Möglichkeiten übersteigt, komisch.

Dass ich damit Geld verdienen will habe ich nie gesagt und wird auch nie 
so sein. Einmaliges Projekt, einmliger Prototyp.


> Daniel C. schrieb:
>> Z.B. soll ich
>
> So so, du sollst. Hast wohl jemanden mehr versprochen, als du
> leisten kannst und jetzt sollen wir dir aus der Patsche helfen?
> Na, du traust dir ja was. Google nach einem guten Tutorial und
> arbeite dich ein. Wenn du soweit bist, kannst du hier gezielte
> Fragen zu Problemen stellen oder mach eine Berufsausbildung, was
> andere ja auch machen.

Wer redet denn von "Jemandem"? Es geht um das Forum, mich und mein 
Projekt. Ich habe nicht einmal versprochen, dass ich C lerne, obwohl 
genau das eigentlich auch gesagt werden könnte.

Aus der Patsche hilft das Forum jeden. Denn wenn ich in der Patsche 
sitze, dann sind Fehler im Projekt.

Ic werde mich einarbeitne, keine Sorge.

Und jetzt sage ich dir: Werd nicht frech, Kollege. Komm mir nicht mit 
"Berufausbildung". Nur weil ich Student bin, heißt dass nicht, dass ich 
keine Berufausbildung, keinen Meister und weitere Qualifikationen 
vorweisen kann. Sowas gehört nicht einmal in so ein Forum. Übrigends: 
Das gehört für mich ganz klar zu einem blöden Kommentar.


> Daniel C. schrieb:
>> Habe ich mit keinem Wort gesagt, oder?
>
> Patzig zu werden, kommt für einen, der Hilfe sucht hier sicher nicht
> gut an.

Etwas patzig zu verstehen, erlaubt es nur, wenn man es patzig verstehen 
möchte. Ich schreibe jetzt grade patzig, weil ich mehr als sauer über so 
einen Kommentar hier bin.

> Daniel C. schrieb:
>> Ich möchte Programmieren lernen, es aber direkt auf meine Schaltung
>> anwenden, um direkt Hard- und Software zu vereinen und meinem Ziel - dem
>> fertigen Progamm- näher zu kommen.
>
> Mach das so, wie es vernünftig ist, entweder durch einen Kurs, eine
> Berufsausbildung oder Autodidaktisch. Das Internet bietet da ausreichend
> Möglichkeiten, die du dir selbst aussuchen kannst.
> Hilfe hier, muss verdient werden. Die bekommt oder verlangt man nicht
> mal so. ;-b

Und erneut. Stelle keine Vermutungen an. Besinne dich auf die Fakten und 
klaren Angaben.

Ich werde schon anfangen zu lernen. Gib mir Zeit. Und tu mir einen 
Gefallen. Wenn dir meine Art des Hilfeersuchs nicht passt, dann schaue 
auch nicht rein.

Für jegliche Hilfe, die dennoch von dir hier kommt, sowie auch die in 
deinem Kommentar stehenden Tipps und Anregungen, sind herzlich 
willkommen. Danke dafür.

Daniel C. schrieb:
> void Warnung (uint8_t AnzahlLeds)
> {
>   const uint16_t BarGraph [2] = {
>       0b101010101010101, 0b010101010101010
>   };
>   PORTC |= BarGraph [AnzahlLeds];            //PortC An?!
>   PORTD |= BarGraph [AnzahlLeds];            //PortD An?!
> }

Was passiert wenn AnzahlLeds gleich 4 ist?

Eric B. schrieb:
> Daniel C. schrieb:
>> void Warnung (uint8_t AnzahlLeds)
>> {
>>   const uint16_t BarGraph [2] = {
>>       0b101010101010101, 0b010101010101010
>>   };
>>   PORTC |= BarGraph [AnzahlLeds];            //PortC An?!
>>   PORTD |= BarGraph [AnzahlLeds];            //PortD An?!
>> }
>
> Was passiert wenn AnzahlLeds gleich 4 ist?

Nichts, hätte ich jetzt gesagt.
Weil es auch nur die beiden Zustände gibt. Da muss ich dann drauf 
achten, wenn bevor ich das Programm aufrufe.


Jörg W. schrieb:
> #define NLEDS 15
> uint8_t aled(int Drehzahl)
> {
>   static const __flash int limits[NLEDS] =
>   {
>     1000, 1100, 1200, 1500, 2000,
>     3000, 4000, 5000, 5500, 6000,
>     6500, 7000, 7500, 8000, 8500
>   };
>   uint8_t i;
>   for (i = 0; i < NLEDS; i++)
>     if (Drehzahl >= limits[i])
>        return i + 1;
>   return 0;  // < 1000
> }

Das habe ich gestern nicht so richtig verstanden und versuche es mal 
selbst zu erklären: NLEDS wird als Zahl 15 defniert, damit die 
Reichweite von const gegeben bzw. begrenzt wird. Unten wird dann die 
Drehzahl überprüft und dann über return an aled übergeben.
Ist also aled die variable, die von 0-15 die anzahl anhand der drehzahl 
sagt?

Würde es gehen, wenn ich aled mit dem Aufruf des Drehzahlprogramms 
ersetze, sodass wenn die Drehzahl überprüft wurde und die Anzahl der 
LEDs klar ist, diese in das Drhzahlprogramm geladen wird, damit dieses 
weiß, welche von den LEDs bitte angehen sollen?

Daniel C. schrieb:
>> Was passiert wenn AnzahlLeds gleich 4 ist?
>
> Nichts, hätte ich jetzt gesagt.

Falsch. Im besten Fall zeigen deine LEDs ein unerwartetes Muster, im 
schlimmsten Fall resettet sich dein uC.
Ausserdem kannst du mit

1

PORTX |= muster

nur LEDs einschalten, aber nicht ausschalten.

Nochmal: nimm ein C-Buch und lerne erstmal Grundlagen!

Bernd N schrieb:
> Deine Code Kommentare und Beschreibungen machen mir den Eindruck
> das du
> den Code nicht verstehst, weder meinen noch Jörgs.
>
>> Daniel C. schrieb:
>>> void Warnung (uint8_t AnzahlLeds)
>>> {
>>>   const uint16_t BarGraph [2] = {
>>>       0b101010101010101, 0b010101010101010
>>>   };
>>>   PORTC |= BarGraph [AnzahlLeds];            //PortC An?!
>>>   PORTD |= BarGraph [AnzahlLeds];            //PortD An?!
>>> }
>
> Beschreibe mal genau was hier passiert und zwar Zeile für Zeile.
>
> Warum verwende ich folgende Syntax in meienm Code ?
>
>     PORTC |= BarGraph [AnzahlLeds] & 0xFF;
>     PORTD |= BarGraph [AnzahlLeds] >> 8;
>
> Und bleib mal nur bei diesem Code.


Programm Programmname (Variable mit Größenangabe)

Konstante mit 16er Größe la sVariable Bargraph mit zwei Zuständen
Die beiden Zustände in Binärform, ah, was natürlich keinen Sinn macht, 
da man ja auch PortC und D verteilen muss ...

Da habe ich nicht nachgedacht. :D

Daniel C. schrieb:
> Deswegen auch diese Passivität. Es wird besser, allerdings erst ab Mitte
> März. Dann geht es richtig zur Sache.

Dann schlage ich vor, Du lernst bis dahin die Grundlagen und dann kannst 
Du wiederkommen und hier konkrete Fragen stellen.
Denn die meisten hier kotzt Deine Einstellung ganz schön an (meine 
Vermutung).
Und dann Deine empörte Reaktion, wenn hier dementsprechende Kommentare 
kommen...

Cyborg schrieb:
> Was du da vor hast ist ein Projekt und das übersteigt die Möglichkeiten
> des Forums um einiges.

Er hatte im Parallelthread jemanden gesucht, der ihm das für (wenig)
Geld programmiert.  Wir haben ihn davon überzeugt, dass es sinnvoller
ist, dass er selbst programmieren lernt, und dass wir ihm dabei
helfen werden.

Solange er damit auch gedanklich Fortschritte macht, ist es zwar ein
anspruchsvolles Ziel, aber durchaus machbar.  Andere haben schließlich
auch programmieren gelernt.

Daniel C. schrieb:
> Das habe ich gestern nicht so richtig verstanden und versuche es mal
> selbst zu erklären: NLEDS wird als Zahl 15 defniert, damit die
> Reichweite von const gegeben bzw. begrenzt wird.

So in etwa.

Der Präprozessor ersetzt, noch bevor der eigentliche Compiler den
Text sieht, alles Auftreten von „NLEDS“ („number of LEDs“) durch das,
was im #define angegeben ist, hier also die Zahl „15“.  Da die
Anzahl deiner LEDs ja durch die Hardware vorgegeben ist, ist das
sinnvoll, das auch in der Software zu reflektieren.

> Unten wird dann die
> Drehzahl überprüft und dann über return an aled übergeben.

Wobei ja schon bemerkt worden ist, dass meine Auswertereihenfolge
falsch herum war: wenn bspw. die Drehzahl 3800 ist, dann würde bereits
der erste Vergleich positiv ausgehen (sie ist ja größer als 1000).
Man muss das also genau andersherum aufziehen, entweder die Richtung
des Vergleichs ändern, oder die Anordnung der Werte in der Tabelle.

> Ist also aled die variable, die von 0-15 die anzahl anhand der drehzahl
> sagt?

aled ist keine Variable, sondern aled() muss als Funktion aufgerufen
werden.  Der Wert, den die Funktion zurückgibt, kann dann an eine
Variable zugewiesen werden.

Hier ist es einfach die Anzahl der einzuschaltenden LEDs.  Wenn du
das analog zu den anderen Vorschlägen machen willst, kannst du
natürlich auch gleich die Bits für die einzuschaltenden LEDs
zurückgeben (also 0b0000000000000001 bis 0b0111111111111111).  Dann
musst du aber dieses Bitmuster innerhalb von aled() noch ermitteln.
Entweder legst du dazu eine zweite Tabelle an:

1

   static const __flash uint16_t ledbits[NLEDS] =

2

   {

3

      0b0000000000000001,

4

      0b0000000000000011,

5

      // ...

6

      0b0111111111111111

7

   };

Der Index in diese Tabelle ist dann die Laufvariable der vorigen
for-Schleife, an der die Schleife abgebrochen worden ist.   Der
Abbruch dürfte dabei nicht durch "return" erfolgen, sondern mit
"break" (Sprung zum Ende der Schleife).  Alternativ kannst du
dir auch eine Rechenvorschrift für diese Reihe ausdenken:

1, 3, 7, 15, 31, ... (das sind die Bitmuster)

Vielleicht klingelt's ja, wenn man sich die folgende Reihe direkt
darunter schreibt:

2, 4, 8, 16, 32, ...

;-)

Nun, dein Ziel ist anspruchsvoll.  Du wirst nicht drumherum kommen,
ein paar Tutorials zu C durchzuarbeiten.  Tipp: wenn du die Sprache
auf dem PC statt auf dem Controller lernst, hast du es einfacher.
Den gleichen GCC, den es für den AVR gibt, gibt es auch für den PC.
(Lediglich das Schlüsselwort "__flash" ist AVR-spezifisch.)

Theor schrieb:
> und wenn es noch so knirscht.

Nein, mitmachen und was lernen muss er schon.

Das bedeutet eben auch, dass er endlich mal den Compiler anwirft und
an seiner Hardware ein paar LEDs blinken lässt.

Auch wenn es viel gemurmel gibt, dass würde hier nichts, bin ich selbst 
sehr zuversichtlich. Ich bin ehrgeizig genug mir dir Zeit dafür zu 
nehmen und das zu lernen.

Problem ist im Moment die Zeit. Ich habe den Bereich hier aufgemacht, um 
schon mal ein paar kleine Einstiege zu bekommen. Ich habe aber noch 
nicht wirklich die Zeit mich voll drauf zu konzentrieren, aber das 
kommt.

Bitte habt etwas geduld. Ich mache das hier grade schon mal nebenbei, um 
irgendwie mal vorran zu kommen. Es lief die ganze Zeit schon schleppend.

Nun starte ich hier, mit sehr kleinen Schritten und viel Hilfe von euch 
und auch wenn es so aussieht, als habe ich keine Lust, bin ich sehr wohl 
voller Ehrgeiz.

Ich verfolge mit bestem Wissen und Gewissen alle Posts und bin dran. 
Aber bitte, ich bin grade nur nicht voll dabei.

Die Klausuren, wie schon gesagt stehen an. Und diese haben priorität. 
Ich möchte aber auch nicht ncihts tun, bis Ende März und dann vor leeren 
Tatsachen stehen.

Sobald alle Klausuren rum sind (Mitte / Ende März) werde ich, wie schon 
angedeutet, voll durchstarten und mich durch alles durchlesen, was hier 
schon angesprochen wurde.

Ich versuche nur jetzt schon mal kleine Eindrücke zu bekommen und 
versuche mit dem jetzigen Wissen, was aus kurzen C-Anfängen, sowie ein 
wenig Arduino und VBA besteht, das Beste raus zu machen und schon ein 
wenig am Projekt zu arbeiten.

Tut mir Leid, für die ganze Mühe. Wenn ihr jetzt sagt, dass hat alles 
keinen Sinn, entwerder ganz oder garnicht, dann ist es auch ok. Dann 
warten wir alle zusammen bis März, um dann durchzustarten. Aber ich 
werde da sein.


Ich verstehe schon einiges, auch wenn es manchmal nicht rüber komt, und 
auch wie Bernd mir wieder erklärt hat, habe ich das Prinizp verstanden, 
nur die falschen Begriffe, statt Funktion Programm, verwendet. Aber ich 
bin dran.

Vielen Dank :)

Beitrag "AVR asm - Befehl nur jedes 2. mal ausführen"


Gruß

Jobst

Cyborg schrieb:
> Daniel C. schrieb:
>> Aber ich bin dran.
>
> Wenn dich einer erwischt, ganz sicher.;-b
> Manche brauchen eben etwas länger zum kapieren.

Sehr lustig :D :)

Bernd N schrieb:
> void BargraphAnsteuerung (uint8_t AnzahlLeds)
> {
>     const uint16_t BarGraph [16] = {0x00, 0x01, 0x03, 0x07, 0x0F, 0x1F,
> 0x3F, 0x7F, 0xFF,
>                                     0x1FF, 0x3FF, 0x7FF, 0xFFF, 0x1FFF,
> 0x3FFF, 0x7FFF
>                                    };
>     PORTC |= BarGraph [AnzahlLeds] & 0xFF;
>     PORTD |= BarGraph [AnzahlLeds] >> 8;
> }

Erneuter Versuch der Erklärung:

Void: Spezifizierer (Zeiger ohne konkreten Datentyp)
BargraphAnsteuerung: Name der Funktion
(uint8_t: 8-Bit-Datentyp (0-255)
AnzahlLeds): Parameter

const: Konstante
uint16_t: 16-Bit-Datentyp
BarGraph: Name der Konstante
[16]: 16 verschiedene Parameter
0x00, 0x01, 0x03, 0x07, 0x0F, 0x1F, 0x3F, 0x7F, 0xFF, 0x1FF, 0x3FF, 
0x7FF, 0xFFF, 0x1FFF, 0x3FFF, 0x7FFF: 0b0000000000000000 - 
0b0111111111111111

PORTC: Port C
|=: Oberator (oder)
BarGraph: OperatorName
[AnzahlLeds]: Parameter
&: Operator (und)
0xFF: 0b11111111

PORTD: Port D
|=: Oberator (oder)
BarGraph: OperatorName
[AnzahlLeds]: Parameter
>> 8: 8 Bit nach rechts verschieben


>     PORTC |= BarGraph [AnzahlLeds] & 0xFF;

Port C bekommt als Biteinstellung den Inhalt aus dem Parameter 
"BarGraph" und 0xFF. D.h. wenn Bargraph 0b00001111 ist und mit 
0b11111111 und-Verknüpft wird, ergibt dies 0b00001111.
0 und 1 und-Verknüpft ergibt 0
1 und 1 und-Verknüpft ergibt 1
0 und 0 und-Verknüpft ergibt 0


>     PORTD |= BarGraph [AnzahlLeds] >> 8;

Der Parameter "BarGraph", genau gesagt das Bitmuster, wird um 8 Bit nach 
rechts verschoben. Aus z.B. 0b00001111 11111111 wird dann 0b00001111.
Dieses Muster bekommt Port D als Eingabe


Johannes S. schrieb:
> Atmel hat sich soviel Mühe gegeben und einen schönen Simulator in das
> Atmel Studio eingebaut. Warum benutzt du den nicht einfach mal anstatt
> immer nur zu raten?

Den kenne ich schon, habe den aber noch nie so wirklich in Betracht 
gezogen, danke für die Erinnerung. Gute Idee :)


Theor schrieb:
> Eine gute Quelle, in der C insgesamt beschrieben ist, das Material
> geordnet und in Zusammenhänge gestellt ist, ist das C-Buch hier:
> http://openbook.rheinwerk-verlag.de/c_von_a_bis_z/

Super Sache! Danke für den Link. Das ist ja super erklärt. :)