Nummerneingabe SwitchCase AVR

Christoph schrieb:
> Den Quelltext hab ich als PDF angehängt... hoffe das ist ok.

Nein ist es nicht.

Alle Hinweise beim Posten beachten!

Vielleicht passt das besser...

Mit den PCINT- Tasten zählt die Variable z1... zn durch und wird auf dem 
LCD ausgegeben.
Wenn die Zahl gewählt wurde, soll mit Enter INT 1 die nächste Zahl 
angesprungen werden um wieder mit den PCINT- Tasten  die Zahl wählen zu 
können.

Das Problem ist das Bestätigen und weiter springen zur nächsten Zahl, 
hier durchläuft es alle Abschnitte der ISR INT1 und gibt mir dann die 
Zahlen 10,11,12,13,... auf dem Bildschirm aus.

ENTER habe ich über Interrupt ausgeführt, da ich kein polling möchte, 
bzw. weil diese Nummerneingabe nur ein Teil meines Programms ist und ich 
somit diesen Programmteil einfacher in das weitere Programm einbinden 
könnte.

Wenn ich sowas sehe:

1

void zahl11 (){

2

  if(z11<9

3

    {z11++;}

4

  else

5

   z11=0;

6

  ...

höre ich sofort zu lesen auf.

Sehr geehrter Herr Mueller,

es tut mir äußerst Leid ihre wertvolle Zeit geraubt zu haben und 
scheinbar bin ich auch soweit gegangen, dass Sie sich gezwungen gefühlt 
haben, diese Frage meinerseits zu kommentieren.

Glauben Sie mir bitte, könnte ich perfekt programmieren und würde mir 
Programme mal geschwind aus dem Ärmel zaubern, hätte ich hier keine 
Frage gestellt. Ich versuche die Programme erst selbst zum laufen zu 
bringen und wende mich erst wenn ich wirklich am verzweifeln bin  an die 
Gemeinde hier.

Da ich aber kein Programmier- Gott bin und dennoch versuche etwas hin zu 
bekommen habe ich mich erdreistet hier meinen unprofessionellen 
Quelltext einzustellen. Darum bitte ich Sie es doch einfach wie beim 
Unfall auf der Autobahn zu machen: Handy weg, Augen gerade aus und 
einfach dran vorbei fahren. Hilft denen die Helfen wollen und schützt 
die die Hilfe brauchen.

Herzlichen Dank für Ihr Verständnis!

Mit freundlichen Grüßen Christoph

Christoph schrieb:
> Hilft denen die Helfen wollen und schützt die die Hilfe brauchen.

Verhindert nicht, dass man ohne Führerschein keine Kraftfahrzeuge 
bedienen sollte.

Nick M. schrieb:
> Verhindert nicht, dass man ohne Führerschein keine Kraftfahrzeuge
> bedienen sollte.

Und man soll auch keine Fahrschule (mikrocontoller.net) nutzen um das 
Fahren zu erlernen und einen Führerschein zu erlangen? Schreib halt mal 
objektiv was dir nicht passt
an:
void zahl11 (){
  if(z11<9
    {z11++;}
  else
   z11=0;

Yeet schrieb:
> Schreib halt mal
> objektiv was dir nicht passt

Dass der Code demonstriet, dass der Schreibsler sich absolut keine 
Gedanken gemacht hat, was er so schreibt.
Ich werfe mal den Begriff "array" aufs Spielfeld.

Eine erste Idee wäre ja, den Quellcode als *.c-File anzuhängen wie 
normale Menschen das tun. Ihn in ein PDF zu drucken und das dann 
anzuhängen, das hat schon was.

> Eine erste Idee wäre ja, den Quellcode als *.c-File anzuhängen wie
> normale Menschen das tun.

Damit kann der unnormale Mensch dienen. :D

Christoph schrieb:
> Leider klappt es nicht, so wie gewollt.

Dann schreibe bitte auch hin, was das Programm tun soll und was es 
stattdessen tut. Das erleichtert einiges.

Christoph schrieb:
> hier durchläuft es alle Abschnitte der ISR INT1 und gibt mir dann die
> Zahlen 10,11,12,13,... auf dem Bildschirm aus.

Meine erste Idee, ohne den Quellcode gesehen zu haben, sind prellende 
Tasten.
Du brauchst also zunächst eine saubere Entprellung.

Da ich doch noch kurz auf den Quellcode gesehen habe, rate ich Dir, 
etwas über Arrays und Funktionsparameter zu lesen.

Auch der Delay in der INT-Routine ist sträflich. Wenn Du es doch tust, 
obwohl Du weißt, das man sowas nur in begründeten Ausnahmefällen macht, 
darfst Du dich nicht wundern, dass es nicht geht.

Christoph schrieb:
> Nummerneingabe.pdf

Was ist DAS ?

Tasteneingabe per PinChange Interrupt,
AUF und AB auf einem Taster.

Bitte erst mal Grundlagen lesen
und an Beispielen lernen.

MaWin schrieb:
> AUF und AB auf einem Taster.

Sieht nur nach "AUF" aus. Jedenfalls habe ich weder "--" noch eine 
Prüfung auf "<0" gesehen.

Das ist echt traurig was ihr hier abzieht. Wenn er's besser wüsste hätte 
er nicht gefragt

Tom schrieb:
> Das ist echt traurig was ihr hier abzieht. Wenn er's besser wüsste hätte
> er nicht gefragt

Damit es mancher versteht "was hier abgezogen" wird:
Ich mach mir öfters den Spass Fehler in fremden Code zu finden. Ich mach 
das freiwillig und zu meinem Vergnügen. Wenn ich aber einen Quältext ... 
ähhh ... Quelltext sehe, der nur Kraut & Rüben ist, dann vergeht mir die 
Lust ganz schnell.
Ich denk mir dann, dass der TO sich selbst doch "etwas" mehr Gedanken 
machen soll, bevor er zig Andere damit beschäftigt. Und grad hier ist es 
so, dass es dem TO an Grundlagenwissen fehlt. Letztendlich müsste ihm 
jemand den Senf komplett neu schreiben. Viel Glück dabei!

Ja, ich selbst mach auch dumme Fehler! Ich behaupte sogar, dass meine 
Fehler brilliant sind!

Aber hier geht es nicht um einen Fehler, hier geht es um eine kaputte 
Gesamtstruktur. Und die sollte sich der TO doch bitte selbst erarbeiten, 
paar Hinweise hat er ja schon.

Hallo zusammen,


ich nutze das Xplained mini 328PB.

Mir reicht es völlig, wenn die Zahlen in einer Schleife hoch gezählt 
werden können. Das hochzählen hat mit meinem Quältext (was hab ich 
gelacht) funktioniert, ist wohl aber nicht zielführend.

Bei PD3 handelt es sich um den "ENTER/Bestätigen" Taster.
Zum switchen der Zahlen habe ich einen weiteren Taster an PD5 oder PD6.

Mein Ziel war es die einzelnen Zahlen in ein Array zu schreiben, dieses 
Array an meinen AT- Befehl zu übergeben und diesen anschließend an das 
GSM-Modul zu übermitteln. Das übermitteln von festgelegten Arrays 
(Nummern) funktioniert bereits.
Es geht lediglich um die Eingabe und das schreiben in das Array.

Der gedachte Ablauf wäre:
Auswahl im bereits vorhandenen Menü => Nummer eingeben
Dann die nicht vorhandene Maske um die Nummer einzugeben, welche dann in 
ein Array gespeichert wird.




Noch eine kurze Info in eigenem Interesse. Ich bin kein Depp, der mit 
seiner Zeit nichts anzufangen weiß und sich gedacht hat ich programmiere 
mal was mit dem Wissen es nicht zu schaffen. Genau sowenig ist es meine 
Absicht hier jemanden auf den Sack zu gehen.
Dass mein Quelltext weit weg von professionell ist habe ich verstanden 
und mir auch vorab schon gedacht (Ich reflektiere mein Können durchaus).
Jedoch muss ich ein schulisches Projekt bearbeiten und versuche dies 
nach bestem Wissen und Gewissen zu tun.
Leider wurde mir im Unterricht lediglich beigebracht, wie die 
Grundfunktionen ablaufen, das integrieren von Programmteilen war leider 
kein Inhalt. Somit versuche ich derzeit unter Zeitdruck mit rudimentärem 
Wissen ein laufendes Programm zustande zubringen.
Hierbei möchte ich erneut anmerken, dass ich mich vorab durchaus 
versucht habe zu informieren, sei es hier auf microcontroller.net oder 
mit diversen Büchern. Jedoch ist das Anlegen eines Array etwas anderes, 
als es dann in Programm und Hardware einzubinden.

Ich möchte weißgott kein Mitleid, sondern dachte an konstruktive Hilfe.
Diese Hilfestellung wurde von einigen hier auch absolut gegeben!!!
Aber vor allem Herr Nick Mueller scheinen es zu genießen mich immer 
wieder mit seinen Kommentaren zu erniedrigen. Herr Mueller, bitte ich 
habe es verstanden, danke!

Für die Hilfsbereitschaft möchte ich sehr danken und wünsche einen 
schönen Abend!!!

Christoph schrieb:
> Aber vor allem Herr Nick Mueller scheinen es zu genießen mich immer
> wieder mit seinen Kommentaren zu erniedrigen. Herr Mueller, bitte ich
> habe es verstanden, danke!

Da hätte ich zwei verschiedene Antworten darauf:
Die erste, schmerzhafte: "Heul doch!"

Die längere:
Du musst lernen, mit Kritik umgehen zu können. Insbesondere wenn du 
programmierst. Das fängt mit Selbstkritik an.
Deine Postings sind ordentlich strukturiert und geschrieben (sag ich 
ganz ohne Sarkassmus). Genau so solltest du deine Programme 
strukturieren. Überlegen was du willst und Schritt für Schritt 
systematisch aufteilen und detailieren. C ist auch nur eine Sprache.
Mach es gleich wie im Deutschen, überleg dir was du sagst und wie du 
argumentierst.
Überleg dir was du willst und wie du das in kleinere Happen unterteilst 
die für sich klar und einfach sind.

Und wenn du dir danach den Code selbstkritsch anschaust und sagst "Sieht 
eher schlampig aus", dann ist das ein Signal dein Vorgehen zu 
überdenken. Bei einem Anfänger kommt das oft vor. Bei mir passiert das 
auch gelegentlich und ich setzt mich hin und mach es ordentlich. Wird 
dann auch kürzer und klarer erkennbar was eigentlich passiert.

So, jetzt schau dir nochmal deinen code an und vergleich den mit deinen 
Postings. Ist dein code eine ordentliche Beschreibung?

Und dann, wenn dein code ordentlich strukturiert ist, findest du auch 
Fehler. Du kannst Funktion für Funktion ohne Seiteneffekte testen 
(printf, Debugger) und schnell(er) Fehler finden.

Hallo Zusammen,

danke für die Hilfe, extra Dank geht an Michael M. für die Erläuterung.

Bin die letzten Tage etwas vorwärts gekommen und hab die Nummerneingabe 
nun funktionsfähig.