Hallo Maddin,

die Idee gefällt mir .

Bei deinem Code läuft es mir allerdings als Embedded Entwickler kalt den Rücken runter.

Ich nehme an du hast früher viel mit Assembler gemacht?

In einer C/C++ Umgebung (auch wenn Arduino da auch nur halbgar ist) verwendet man eigentlich keine Sprungbefehle.

eine Statemachine wird dabei ganz einfach mit einer switch case Anweisung abgefahren, dann kennt man sich auch noch aus, wenn man nach ein paar Wochen wieder rein schaut .

Bevor du es fest verbaust, würde ich dir empfehlen nochmal am Code zu arbeiten.

Grüße Max

Hallo Max,
vielen Dank für die ehrliche Rückmeldung.
Normalerweise arbeite ich in VBA oder Basic++ und da ist einiges anders.
Das ist tatsächlich mein erster Versuch auf ARDUINO.
Ich tue mir noch sehr schwer damit, aber es läuft
Ich bin eben wach geworden und habe gemerkt, das Delay kommt zu spät.
Muss paar Zeilen vorher kommen.
Mit diesen Semikolons am Ende und die geschweiften Klammern 🤪🤪🤪
Da ist VBA für meinen Geschmack etwas strukturierter.
Kennst du eine andere Plattform mit der Mann entwickeln kann?
Früher hatte ich solche Sachen mit der C-Control gemacht.
Aber die gibt es ja leider nicht mehr.
Meine Wickelmaschine läuft noch damit.
Alle anderen Versuche liefen nicht so erfolgreich.
Dann ist mir die Idee mit den Sprungmarken gekommen.
Jetzt läuft es.
Alle Wege führen nach Rom

Aber jetzt kommt es erst mal wieder in den Sitz, es funktioniert ja.
Mehr soll es ja nicht.

Vielen Dank noch einmal 👍👍👍

LG, Maddin

Ehrlich .........

ich verstehe hier nur Bahnhof (schaut aber nach Experte aus)

echte Intensiv-Technik fürn ARSCH .... mein oller Blauer hat keine Sitzheizung.

Meinen Respekt hast Du auf jeden Fall ...... ich denke, Du weist was Du tust.

Früher, ja früher,

da habe ich noch in ASCII-Fortran programmiert und ganz zu Anfang habe ich auch noch die Programme auf Lochkarten gestanzt und zum Plotten beim Rechenzentrum abgegeben .

Lang ist´s her .

Und klasse, dass Dein ... wieder im Winter warm hat .

Grüssle ausm Badischen
Rainer alias Nasenbär

Hallo Maddin,

dass sieht echt spannend aus👍🏻(ich spinn dann auch gleich weiter, was man noch so alles mit der Steuerung machen könnte). Wie gern würde ich mal mehr Ahnung von solchen Sachen haben. Aber wo und wie anfangen🤔.

VG Michael

Ich mach mal in meinem Threat weiter, will den hier nicht kapern

LG, Maddin

Nachdem ich die konstruktive Kritik aufgenommen hab und für Besserung gelobt habe, hier das Ergebnis.
Ohne Sprungmarker, mit Variablen, immer wieder genullt (hier war der Hund begraben).

int statis_heizung = 0; // 0 - aus, 1 - reduziert, 2 - volle Leistung
long zaehler = 0;
const int mosfet = 9;
const int t_ein = 10;
const int t_aus = 11;
const int rot = 12;
const int gruen = 13;


void led_aus (int ausgang){
digital_schreiben (ausgang, LOW);
}
void led_an (int ausgang){
digital_schreiben (ausgang, HIGH);
}


void digital_schreiben (int ausgang, int wert){
digitalWrite(ausgang, wert);
}


void hw_schreiben (int ausgang, int wert){
analogWrite(ausgang, wert);
}


int hw_lesen (int welche_taste){
return (digitalRead (welche_taste));
}


void setup(){
pinMode(mosfet, OUTPUT);
pinMode(gruen, OUTPUT);
pinMode(rot, OUTPUT);
pinMode(t_ein, INPUT);
pinMode(t_aus, INPUT);
pinMode(10, INPUT_PULLUP);
pinMode(11, INPUT_PULLUP);
}


int taste_gedrueckt (int welche_taste){
int statis = hw_lesen (welche_taste); // 0 - aus; 1 - an
if (statis == HIGH){
return 0;
}
while (statis == LOW){
delay(50);
statis = hw_lesen (welche_taste); // 0 - aus; 1 - an
}
return 1;
}


int schalte_heizung(int statis){
switch (statis){
case 0:
hw_schreiben (mosfet, 0);
led_aus(rot);
led_aus(gruen);
break;
case 1:
hw_schreiben (mosfet, 180);
led_aus(rot);
led_an(gruen);
break;
case 2:
hw_schreiben (mosfet, 255);
led_an(rot);
led_aus(gruen);
break;
default:
//fehler anzeigen
break;
}
}


void timer_aufziehen(){
zaehler = 7000000;
}

int ist_timer_abgelaufen(){
return zaehler == 0;
}

int timer_anhalten(){
zaehler = -1;
}

int timer_weiterzaehlen(){
if (zaehler > -1){
zaehler = zaehler - 1;
}
}


void setze_statis(int statis){
switch (statis){
case 0:
schalte_heizung(0);
statis_heizung = 0;
break;
case 1:
schalte_heizung(1);
statis_heizung = 1;
break;
case 2:
schalte_heizung(2);
timer_aufziehen();
statis_heizung = 2;
break;
}
}


void loop(){
int einschalter = 0;
int ausschalter = 0;

switch (statis_heizung){
case 0:
einschalter = taste_gedrueckt(t_ein);
if (einschalter == 1){
setze_statis(2);
}
ausschalter = taste_gedrueckt(t_aus);
if (ausschalter == 1){
setze_statis(1);
}
break;
case 1:
einschalter = taste_gedrueckt(t_ein);
if (einschalter == 1){
setze_statis(2);
}
ausschalter = taste_gedrueckt(t_aus);
if (ausschalter == 1){
setze_statis(0);
}
break;
case 2:
timer_weiterzaehlen();
einschalter = taste_gedrueckt(t_ein);
if (einschalter == 1){
setze_statis(1);
}
ausschalter = taste_gedrueckt(t_aus);
if (ausschalter == 1){
setze_statis(0);
} else{
if (ist_timer_abgelaufen()){
setze_statis(1);
}
}
if (statis_heizung != 2){
timer_anhalten();
}
break;
default:
// Fehler
break;
}
}

Zum Listing hat sich auch noch das Layout geändert.
Der Fehler war in den externen PullDown Widerständen.
Der Arduino hat interne PullUps, die zu nutzen ist eleganter.
Durch die Externen kam es immer wieder zu Fehlinterpretationen des Schalters.

Das Projekt Sitzheizung

Da ist auch das Listing hinterlegt.
Das Layout ist jetzt so:



Läuft einwandfrei.

Hallo Mathias
Da hätte ich im Nachgang noch eine Frage: Nebst den 2 7,5er Sicherungen ist ja noch die 20A Sicherung. Wozu dient die? In meinem Handbuch Ausgabe 11-2018 ist die nicht beschrieben.
Danke für die Hilfe, LG Ulrich