Einsteiger braucht hilfe bei den 1. Programmen

Erst mal hallo an alle, da ich hier neu bin und schonmal danke im voraus,

Ich wollte wissen, ob der folgende code richtig ist, da ich grade mit c angefangen habe und das mein erstes programm ist.

#include "asuro.h"

int main(void)
{ char befehl[8];
unsigned char ktaster;
Init();
SerRead(befehl,8,0);
MotorDIR(FWD,FWD);
MotorSpeed(255,255);

while(1);{

ktaster=PollSwitch();
if (ktaster>7)
{MotorSpeed(180,60);}
else
{MotorSpeed(60,180);}
}
return 0;}

int main(void)
{ unsigned int odata[2];
init();
OdometrieData(odata);
if (odata<1023)
{SerWrite("Wir fahren",9);}
else
{SerWrite("Wir fahren nicht",14);}
while(1);
return 0;}

was mir so auf die schnelle einfällt
(1) nimm bitte den code button wenn du code postest
(2) du hast zweimal die main funktion

was soll dein programm eigentlich genau machen?

zu 1) werd ich mir merken und zu 2) werde ich ändern und er soll auf befehl "fahr los" los fahren dann wenn er irgendwo gegesstößt die richtung ändern und danach die odo leds checken und wenn die merken das er fährt "wir fahren" senden und wenn nciht dann "wir fahren nicht" senden.

Code:

---

OdometrieData(odata);
if (odata<1023)
{SerWrite("Wir fahren",9);}
else
{SerWrite("Wir fahren nicht",14);}

---

das wird ztu 100% nicht funktionieren.

er wird dir immer ausgeben, dass er fährt.
mt der funktion OdometrieData() wird NICHT DIE DREHZAHL GEMESSEN.
die drehzahl musst du dir erst selbst ausrechnen. gibts hier n paar threads dazu.

mfg EDH

Danke für die Info

eventuell wäre da jetzt sogar ein einsatzweck für diese multitasking.
in einem thread wird der motor gesteuert
im anderen thread "wir fahren" bzw "wir fahren nicht" ausgegeben

oha wir wollen mal lieber nichts überstürzen^^

Sieht das besser aus? Ich hab den code jezt nochmal überarbeitet

Code:

---

#include "asuro.h"

int main(void)
{        char befehl[8];
        unsigned char ktaster;
        Init();
        SerRead(befehl,8,0);
        MotorDIR(FWD,FWD);
        MotorSpeed(255,255);
       
        while(1);{
       
                ktaster=PollSwitch();
                if (ktaster>7)
                {MotorSpeed(180,60);}
        else
                {MotorSpeed(60,180);}
        }
        return 0;


        void abfrage(void) {
        unsigned int odata[2];
        init();
        OdometrieData(odata);
        if (odata<1023)
                        {SerWrite("Wir fahren",9);}
        else
                        {SerWrite("Wir fahren nicht",14);}
        while(1);
        return 0;}
}

---

mhh...
also genaugenommen sieht der code immer noch ziemlich scheps aus

aber ich glaube nicht das es jetzt was bringt, wenn ich erklär was falsch ist, oder den berichtigten code poste.


hast du die asruo anleitung GENAUESTENS studiert?
da steht ja nicht wirlich viel über c drinn. aber das was man am anfang für den asuro braucht ist vorhanden.
vieleicht solltest du lieber erst selber programme schrieben, wenn du alle beispiele auswendig kannst.

Naja ich hab die anleitung durchgelesen ich kann sie nich auswendig aber ich meine dass ich sie verstanden habe. ABer es kann auch sien dass ich nur denek das ich sie verstanden habe, aie allerdings gar nicht wirklich verstanden habe^^ Aber ich fänd es nett wenn du mir vllt den richtigen code posten würdest (wenn es keine Umstände macht), dann kann ich besser sehen was ich falsch mache.

also was schon mal nicht geht ist while(1);{...

an dieser stelle wird er hängen bleiben. while(1); ist eine endlosschleife. besser gehts so:

while(1){

Hier rein kommen die Anweisungen die immer wieder wiederholt werden sollen

}

sälbstverständlich werden diese anweisungen dann bis ewig wiederholt. es hilft also auch nix, eine zweite while(1) schleife unten drunter zu schreiben. du musst alle anweisungen da rein packen.

fang mal an mit einem programm, das nur auf die taster reagiert und meinetwegen die leds anschaltet. dann nimmst du als zweite schwoerigkeitsstufe die motoren dazu... und immer so weiter. das ist der beste (und eigentlich auch einzige) weg das zu lernen: schritt für schritt.

meld dich wenn du noch fragen hast!

ok danke mach ich

Was hällst du erstmal davon?

Code:

---

#include "asuro.h"

int main(void)
{       
        unsigned char ktaster;
       
        init();
        StatusLED(GREEN)
        MotorDIR(FWD;FWD);
        MotorSpeed(200,200);
       
        while(1){
        ktaster=Pollswitch();
        Pollswitch();
        if (ktaster>0)
        {StatusLED(RED);}
        else
        {StatusLED(GREEN);}
               
        }
        return 0;
}

---

[/code]

sollte ich die "Pollswitch();" Abfrage vllt noch öfters machen?

(1) Init() statt init()
(2) MotorDir() statt MotorDIR()

wenn du die PollSwitch Abfrage öfter machst, ist das risiko eines falschen wertes kleiner. 3-4 mal helfen eigentlich, 10x geht auf nummer sicher(in einer schleife), braucht aber auch mehr zeit.

es wäre einfacher, wenn du in der if abfrage einfach die varible switched nimmst
die wird von nem interrupt gesetzt.

allerdings musst du daran denken, sie nach dem was der asuro machen soll wen der schalter gedrückt wurde, wieder auf 0 zu setzen.

beispiel:

if(switched=1){

hier kommt das hin was er machen soll

switched=0;

}

ahem nur ein kleiner hinweis.
es muss heise
if(switched==1) {
}

alternativ kann man auch einfach
if(switched) {
}

schreiben

Ähm ich weiß nicht ob ich das jezt richtig verstanden habe aber ich hoffe dass das so richtig ist.

Code:

---

#include "asuro.h"

int main(void)
{       
        unsigned char ktaster;
       
        Init();
        StatusLED(GREEN)
        MotorDir(FWD;FWD);
        MotorSpeed(200,200);
       
        while(1){
        ktaster=Pollswitch();
        Pollswitch();
        if(switched==1){
        {StatusLED(RED);}
        else
        {StatusLED(GREEN);}
               
                switched=o
        }
        return 0;
}

---

den aufruf von pollswith kannste streichen. auserdem hängt der zweite eh irgendiwe so in der lugt un bewerikt gar nix
bei dir steht switched=o
es muss heisen switched=0

auf was bezieht sich den "switched" nur auf die Taster oder reagiert die funktion einfach auf alles was sich verändert?

Sieht das so besser aus?

Code:

---

#include "asuro.h"

int main(void)
{       
       
       
        Init();
        StatusLED(GREEN)
        MotorDir(FWD;FWD);
        MotorSpeed(200,200);
       
        while(1){
       
        if(switched==1){
        {StatusLED(RED);}
        else
        {StatusLED(GREEN);}
               
                switched=0
        }
        return 0;
}

---

switched bezieht sihc nur auf die taster

(1) Schreib lieber MotorDir(FWD,FWD) statt MotorDir(FWD;FWD)
(2) du hast bei der if abfrage zwei geschweifte klammern
(3) nach StatusLED(GREEN) kommt noch ein strichpunkt

ok, aber welche geschweiften klammerrn sind den zuviel?

Code:

---

if(switched==1){
  {StatusLED(RED);}
  else

---

di eklammer wo direkt vor StatusLED steht ist zuviel

Also müsste es heißen:

Code:

---

#include "asuro.h"

int main(void)
{ 
 
 
  Init();
  StatusLED(GREEN)
  MotorDir(FWD;FWD);
  MotorSpeed(200,200);
 
  while(1){
 
  if(switched==1){
  StatusLED(RED);}
  else
  {StatusLED(GREEN);}
     
      switched=0
 
  return 0;
}

---

Kommt der Block in der while Schleife denn nicht in Klammern?

ist er doch... o0

while(1){

da is ne klammer.

allerdings hast du das semikolon und die schliessende klammer hinter switched=0 vergessen.

Hallo Sloti,

ich denke, Dir mangelt es noch massiv an C-Kenntnissen.
Dein Code sieht grauenhaft aus...
So wird das nichts Gescheites werden.
Les Dir am besten erstmal einen der vielen C-Kurse im Netz durch, fang an mit dem Verstehen von einfachen "Hallo-Welt"-Programmen usw..

Hat zwar am Anfang nichts mit Robotern und AVRs zu tun, aber es hilft Dir am besten weiter. Bevor Du mit solchem Code Deinen Controller totflash'st, bis mal was Lauffähiges rauskommt, hast Du auf einem Standard-PC einen leichteren Einstieg in C.

By the way: Dein Betreff zu diesem Thread is ja auch echt gut gewählt.


Gruß

Michl

Ach naja... wenigstens die Grundidee ist ja vorhanden, und jeder versuch ist besser als der vorherige... =) wir müssen uns eigentlich nur noch um die syntax kümmern.

ausserdem: wenn du soweit wärst, du würdest den asuro doch auch nicht in die ecke stellen, um erst mal nen c-kurs zu machen oder? =)

der titel ist allerdings einer der genialsten, das is schon richtig -.-

Ihr habt ja recht der titel war scheiße aber wenn ich ihn speziel auf etwas beziehen würde und ich dann eine andere Frage hätte würde der Titel nicht mehr passen. (Konjunktiv ist ne bescheuerte Erfindung)
und wegen der Klammer ich werd sie da hinschreiben.

ps:@damaltor herzlichen Glückwunsch zum 300 Beitrag^^

mfg.
Sloti

oh... o0 wow... =)

du kannst den titel übrigens ändern *anpiek*

editiere den ersten post und gib ihm einen vernünftigen titel, das hilft doch sehr...

Zitat:

---

ausserdem: wenn du soweit wärst, du würdest den asuro doch auch nicht in die ecke stellen, um erst mal nen c-kurs zu machen oder? =)

---

Ja, würde ich,
oder fährst Du auch Auto ohne die Fahrschule davor besucht zu haben ?
Wenn man die User hier nach Qualität der Beiträge bewerten würde, hättest Du einen "Roboter Noob" anstatt "Roboter Experte" unter Deinem Nick stehen.

Eben.

Ich denke, es sollte hier um Robotik gehen.

Die Grundlagen der Programmierung sowie auch Elektronik-Grundkenntnisse sollte man schon mitbringen. Sonst kommt als nächstes die Frage, was ein Spannungsteiler ist.

Ich finde es ja durchaus gut, wenn ein 14jähriger sich für das alles interessiert. Aber auf die Hau-Ruck-Methode funktioniert das sicher nicht.


Gruß

Michl

Ich denke, dass eure Einwände berechtigt sind, aber um Missverständnissen vorzubeugen:
1. Ich habe noch gar keine Asuro. Ich bin blos darauf gekommen, da wir in der Schule einen Kurs für Robotik haben und dort bald einen (bzw. 15) Asuros bauen.

2. Ich bin kein Mensch der etwas überstürzt anfängt allerdings habe ich mein erstes Program, welches hier ganz am Anfang steht, geschrieben, nachdem ich die Anleitung durch hatte und gemerkt habe, dass Programieren gar nicht so schwer ist. Aber langsam sehe ich ein, dass ich noch kaum Ahnung habe wie das geht. Aber mein Leben ist (hoffentlich) noch lang und ich habe aller Zeit der Welt C zu lernen.

was haltet ihr von dem neuen titel?^^

ist jedenfalls passender als der erste ;)

na dann ist ja gut .-)

So mein nächster Schritt soll sein dass wenn er gegen die Wand stößt sich umdrehen soll erstmal nicht mehr. Ich weiß, dass es eine turn Funktion gibt aber ich weißt nicht wie sie funktioniertund in der standart asuro Bibliothek auch soeit ich weiß nciht drin ist. ich habs einfach mal so gemacht:

Code:

---

#include "asuro.h"

int main(void)
{       
        unsigned char ktaster;
       
        Init();
        StatusLED(GREEN)
        MotorDir(FWD;FWD);
        MotorSpeed(200,200);
       
        while(1){
       
        if(switched==1){
        {StatusLED(RED);
        MotorDir(FWD,RWD);
        MotorSpeed(100,100);
        Sleep(255);
        MotorDir(BREAK,BREAK);}
        else
        {StatusLED(GREEN);}
               
                switched=0
        }
        return 0;
}

---

Ich denke dass es nicht so funkioniert wie es soll, er soll nämlich um 180° gedreht werden.

also syntax der turn funktion ist so:
Turn(grad,geschwindigkeit)

achtung:
zuvor Encode_Init() aufrufen

zu dem code oben:

switched = 0 in den else block zu schreiben ist schwachsinn

wie soll er eigentlich drehen, wenn immer alle motoren gleichschnell laufen?
[edit] ups da hab ich übersehen das vorher noch mal MotorDir drann steht :oops: [edit]