C String an Terminal ausgeben

[Da H. (darko91)]

Hallo Zusammen,

...BITTE NICHT ABSCHRECKEN VON DER LÄNGE DES BEITRAGS...

Bitte um eure Hilfe zu folgendem Punkt, da ich schon fast am Verzweifeln 
bin:

Ich schreibe manuell einen String, wie unten "Hallo Welt", in die 
Variable c rein. Diese wird in diesem Fall auch richtig im Terminal 
(TeraTerm) ausgegeben.

char c[] = {"Hallo Welt "};

while(1)
{
  int i;
  for(i = 0; i <= sizeof(c); i++)
  {
    UDR0 = c[i]; _delay_ms(200);
  }
}

Nun will ich das Ganze über Bluetooth steuern. LEDs zum leuchten bringen 
funktioniert mal einwandfrei, einen von einem Smartphone gesendeten 
String im Terminal auszugeben bereitet mir aber schon Kopfschmerzen.

Angefangen von einfachen Characters:
Wenn ich diese 1:1 ans Terminal weitergebe, kommen irgendwelche Symbole 
raus. Ich habe mit Hilfe der Ascii Tabelle die Zahlen um 128 Stellen 
nach hinten verschoben.
Nun werden mir zwar Zahlen von 1-9 sowie manche Sonderzeichen richtig 
angezeigt (Ascii Tabelle, Nr. 34 (") bis inkl. 63 (?)), jedoch kommen 
die Buchstaben falsch raus (z.B. Statt "a" kommt ein "!" raus oder statt 
"z" ein ":" - also 64 Stellen nach hinten verschoben).

if(blueToothSerial.available()){
    int i;
    for(i = 0; i <= sizeof(c); i++)
    {
        c[i] = blueToothSerial.read()-128;  // um 128 Stellen nach hinten verschieben
    }
          
    UDR0 = c[i]; _delay_ms(200);
}

Wenn ich noch eine if-Bedingung rein haue, ist nur jede 2. Eingabe 
richtig (sowohl Ziffer, als auch Buchstabe). Woran könnte das liegen? 
Logischer Denkfehler?

if(blueToothSerial.available()){
   int i;
   for(i = 0; i <= sizeof(c); i++)
   {
  
        if(c[i] >= '@' && c[i] <= 'z')              // lt. Ascii Tabelle alles zw. "@" und "z"
           c[i] = blueToothSerial.read()-64;   // um 64 Stellen nach hinten verschieben
        else
           c[i] = blueToothSerial.read()-128;  // um 128 Stellen nach hinten verschieben
        }
          
        UDR0 = c[i]; _delay_ms(200);
}

Wie gebe ich danach einen String aus zB 10 Characters richtig ans 
Terminal?

btw: Die Implementierung vom Bluetooth Modul füge ich hier nicht hinzu, 
der Code wäre viel zu lang. Falls es jedoch nötig ist, bitte Bescheid 
geben.

[TestX (Gast)]

Sizeof ist die falsche funktion. Was du willst heißt strlen ....

[Uwe (de0508)]

Hallo,

wer kommt den auf diese Idee?

UDR0 = c[i]; _delay_ms(200);

Das geht doch gegen alle Regeln der Programmierkunst!

Ich verwende je eine Interrupt gesteuerte Sende- und Empfangsroutine mit 
FiFos.

[Walter S. (avatar)]

TestX schrieb:
> Sizeof ist die falsche funktion. Was du willst heißt strlen ....

und strlen hilft da leider auch nix

[Da H. (darko91)]

Uwe S. schrieb:
> Hallo,
>
> wer kommt den auf diese Idee?
>

UDR0 = c[i]; _delay_ms(200);

> Das geht doch gegen alle Regeln der Programmierkunst!
>
> Ich verwende je eine Interrupt gesteuerte Sende- und Empfangsroutine mit
> FiFos.

Kannst du mir ein Beispiel in C-Code zeigen?

[STK500-Besitzer (Gast)]

Darko Jen schrieb:
> Kannst du mir ein Beispiel in C-Code zeigen?

Du programmierst ja offensichtlich mit Arduino.
Guck dir einfach mal die Serial-Bespiele an!

[Falk B. (falk)]

@Darko Jen (darko91)

>> Ich verwende je eine Interrupt gesteuerte Sende- und Empfangsroutine mit
>> FiFos.

>Kannst du mir ein Beispiel in C-Code zeigen?

Ist für dich im Moment viel zu kompliziert, du musst erst mal die 
Grundlagen lernen. Man kann einen UART problemlos ohne Interrupts 
betreiben, aber ganz sicher NICHT mit DERARTIGEN Wartezeiten!

Ausserdem solltes du dir mal ansehen, wie Strings normalerweise 
verarbeitet werden.

https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Der_UART

[Da H. (darko91)]

Falk Brunner schrieb:
> @Darko Jen (darko91)
>
>>> Ich verwende je eine Interrupt gesteuerte Sende- und Empfangsroutine mit
>>> FiFos.
>
>>Kannst du mir ein Beispiel in C-Code zeigen?
>
> Ist für dich im Moment viel zu kompliziert, du musst erst mal die
> Grundlagen lernen. Man kann einen UART problemlos ohne Interrupts
> betreiben, aber ganz sicher NICHT mit DERARTIGEN Wartezeiten!
>
> Ausserdem solltes du dir mal ansehen, wie Strings normalerweise
> verarbeitet werden.
>
> https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Der_UART

Danke Falk für die Hilfe. Ich werde mir das noch in Ruhe durchlesen, hab 
es ehrlich gesagt bisher nur überflogen.
Es handelt sich hierbei nämlich um eine Abgabe, die sehr dringend ist 
und mir der Kopf wegen verschiedensten Sachen schon raucht. Ich wäre 
sehr dankbar, wenn mir jemand mit dem Code weiterhilft - ich werde es 
mir im Anschluss, wenn ich mehr Zeit habe, bestimmt detailliert ansehen. 
Wegen Zeitdrucks schaffe ich es zur Zeit leider nicht.

Vielen Dank!
D.

[Falk B. (falk)]

@ Darko Jen (darko91)

>mir im Anschluss, wenn ich mehr Zeit habe, bestimmt detailliert ansehen.

Naja, wer's glaubt wird seelig.

Wenn du Hilfe willst, musst du schon ein wenig die Netiquette 
beachten und ein paar wichtige Informationen rüber wachsen lassen.

Welcher Controller? (AVR?)
Welche Umgebung (Arduino?)
Welcher Hardwareaufbau?
WAS soll denn KONKRET gemacht werden?

>Nun will ich das Ganze über Bluetooth steuern. LEDs zum leuchten bringen
>funktioniert mal einwandfrei, einen von einem Smartphone gesendeten
>String im Terminal auszugeben bereitet mir aber schon Kopfschmerzen.

???
Deine Hardware hat einen Bluetooth Empfänger UND eine Terminalanbindung 
per Kabel?

>raus. Ich habe mit Hilfe der Ascii Tabelle die Zahlen um 128 Stellen
>nach hinten verschoben.

Dazu braucht man keine ASCII-Tabelle. Ausserdem ist alles >127 KEIN 
ASCII mehr.

>Nun werden mir zwar Zahlen von 1-9 sowie manche Sonderzeichen richtig
>angezeigt (Ascii Tabelle, Nr. 34 (") bis inkl. 63 (?)), jedoch kommen
>die Buchstaben falsch raus (z.B. Statt "a" kommt ein "!" raus oder statt
>"z" ein ":" - also 64 Stellen nach hinten verschoben).

>    UDR0 = c[i]; _delay_ms(200);

Lass den Käse und verwende die ganz normalen Arduino Sendefunktionen! 
Denn sonst spuckt du denen dazwischen, das mögen die nicht wirklich!

  char c;

  c = blueToothSerial.read();
  serial.print(c);

reicht vollkommen, um ein Zeichen auszugeben.

Einen STRING c zu nennen ist recht ungünstig, c kling wie char und das 
ist ein EINZELNES Zeichen.

>Logischer Denkfehler?

Mangelnde Grundlagen.

>Wie gebe ich danach einen String aus zB 10 Characters richtig ans
>Terminal?

char string[]="Ich bin ein echt toller String";
serial.print(string);

http://www.arduino.cc/en/Serial/Print

>btw: Die Implementierung vom Bluetooth Modul füge ich hier nicht hinzu,
>der Code wäre viel zu lang. Falls es jedoch nötig ist, bitte Bescheid
>geben.

Es reicht, den Namen des Shields zu nennen und ggf. einen Link auf eine 
Seite, wo der beschrieben ist. Siehe Netiquette.

[Da H. (darko91)]

Falk Brunner schrieb:
> @ Darko Jen (darko91)
>
>>mir im Anschluss, wenn ich mehr Zeit habe, bestimmt detailliert ansehen.
>
> Naja, wer's glaubt wird seelig.

Mein Ziel ist es natürlich, zumindest die Grundlagen in C zu 
beherrschen. Durch mehr Hilfe lerne ich mehr dazu

> Wenn du Hilfe willst, musst du schon ein wenig die Netiquette
> beachten und ein paar wichtige Informationen rüber wachsen lassen.
>
> Welcher Controller? (AVR?)
ATMEGA328P
> Welche Umgebung (Arduino?)
Nein, in Atmel Studio direkt (gcc), es wird jedoch ein Arduino Board 
verwendet
> Welcher Hardwareaufbau?
Arduino Uno R3 Board
+ Bluetooth Modul von Seeedstudio (es gibt eh nur 1, die Implementierung 
habe ich von ihrer HP 
(http://www.seeedstudio.com/wiki/index.php?title=Bluetooth_Shield) 
übernommen und umgeändert, sodass Atmel es auch versteht. Habe bereits 
erste Ergebnisse erzielt, z.B. die Ansteuerung von LEDs per Bluetooth.
Die 3. Hardware ist ein LCD Display - auch hier erhielt ich bereits 
erste Ergebnisse.

> WAS soll denn KONKRET gemacht werden?

Es soll ein über Bluetooth gesendeter String von einem Smartphone am LCD 
Display (DOGM162L-a) zu sehen sein

>>Nun will ich das Ganze über Bluetooth steuern. LEDs zum leuchten bringen
>>funktioniert mal einwandfrei, einen von einem Smartphone gesendeten
>>String im Terminal auszugeben bereitet mir aber schon Kopfschmerzen.
>
> ???
> Deine Hardware hat einen Bluetooth Empfänger UND eine Terminalanbindung
> per Kabel?
Bluetooth Shield, Das Terminal dient als Zusatz, damit ich nicht unnötig 
das LCD Display belaste.

>>raus. Ich habe mit Hilfe der Ascii Tabelle die Zahlen um 128 Stellen
>>nach hinten verschoben.
>
> Dazu braucht man keine ASCII-Tabelle. Ausserdem ist alles >127 KEIN
> ASCII mehr.
>
>>Nun werden mir zwar Zahlen von 1-9 sowie manche Sonderzeichen richtig
>>angezeigt (Ascii Tabelle, Nr. 34 (") bis inkl. 63 (?)), jedoch kommen
>>die Buchstaben falsch raus (z.B. Statt "a" kommt ein "!" raus oder statt
>>"z" ein ":" - also 64 Stellen nach hinten verschoben).
>
>>    UDR0 = c[i]; _delay_ms(200);
>
> Lass den Käse und verwende die ganz normalen Arduino Sendefunktionen!
> Denn sonst spuckt du denen dazwischen, das mögen die nicht wirklich!

Würde ich gerne, die Aufgabe ist aber leider Gottes, dies ohne den 
Arduino Befehlen zu schaffen.
>
>
>

>   char c;
> 
>   c = blueToothSerial.read();
>   serial.print(c);
>

>
> reicht vollkommen, um ein Zeichen auszugeben.
>
> Einen STRING c zu nennen ist recht ungünstig, c kling wie char und das
> ist ein EINZELNES Zeichen.
Ich taufe es gerne um :) , ich brauche nur einen richtigen Weg, den 
String eben auszugeben
>
>>Logischer Denkfehler?
>
> Mangelnde Grundlagen.
>
>>Wie gebe ich danach einen String aus zB 10 Characters richtig ans
>>Terminal?
>
> char string[]="Ich bin ein echt toller String";
> serial.print(string);

Das klappt natürlich, ich will den per Bluetooth empfangenen String in 
eine Variable abspeichern und dann erneut ausgeben
>
> http://www.arduino.cc/en/Serial/Print
>
>>btw: Die Implementierung vom Bluetooth Modul füge ich hier nicht hinzu,
>>der Code wäre viel zu lang. Falls es jedoch nötig ist, bitte Bescheid
>>geben.
>
> Es reicht, den Namen des Shields zu nennen und ggf. einen Link auf eine
> Seite, wo der beschrieben ist. Siehe Netiquette.

[Falk B. (falk)]

@Darko Jen (darko91)

>Bluetooth Shield, Das Terminal dient als Zusatz, damit ich nicht unnötig
>das LCD Display belaste.

Denkst du, dass es dadurch verschleißt? ;-)

>Würde ich gerne, die Aufgabe ist aber leider Gottes, dies ohne den
>Arduino Befehlen zu schaffen.

Dann musst du es so wie im Tutorial machen.

>Ich taufe es gerne um :) , ich brauche nur einen richtigen Weg, den
>String eben auszugeben

Siehe oben. Einfacher geht es nicht.

>Das klappt natürlich, ich will den per Bluetooth empfangenen String in
>eine Variable abspeichern und dann erneut ausgeben

Sollte ja nicht soo schwer sein.
Welche Funktionen bietet deine Bluetooth-Lib?

Das Grundprinzip ist einfach.

Man prüft, ob Daten vom Bluetooth verfügbar sind
Wenn ja, holt man eins ab.
Ist es ein normales Zeichen, speichert man es im String, dazu muss man 
sich die aktuelle Speicherposition merken.
Ist es das Zeichen \r, ist der String zu Ende, es wird nicht \r 
gespeichgert sondern 0 (Stringterminator).
Dann kann man den String normal ausgeben.

[D. (Gast)]

> Das Grundprinzip ist einfach.
>
> Man prüft, ob Daten vom Bluetooth verfügbar sind
> Wenn ja, holt man eins ab.
> Ist es ein normales Zeichen, speichert man es im String, dazu muss man
> sich die aktuelle Speicherposition merken.
> Ist es das Zeichen \r, ist der String zu Ende, es wird nicht \r
> gespeichgert sondern 0 (Stringterminator).
> Dann kann man den String normal ausgeben.

Danke für die Info.

kannst du mir noch mitteilen, wie vom empfangenen String aus 
bluetoothSerial.read() einzelne characters bekomme? Ich kriege ja per 
Bluetooth einen ganzen String, der in bluetoothSerial.read() 
eingespeichert wird.
Danke

[Falk B. (falk)]

@ D. (Gast)

>kannst du mir noch mitteilen, wie vom empfangenen String aus
>bluetoothSerial.read() einzelne characters bekomme?

RTFM?

http://www.arduino.cc/en/Serial/Read

Dein bluetoothSerial.read() ist nur ein anderer Name für serial.read()

>Ich kriege ja per
>Bluetooth einen ganzen String, der in bluetoothSerial.read()
>eingespeichert wird.

Nein, die Funktion liefert exakt ein Zeichen pro Aufruf.

[D. (Gast)]

http://www.arduino.cc/en/Serial/Read
>
> Dein bluetoothSerial.read() ist nur ein anderer Name für serial.read()
>
> Ich kriege ja per >Bluetooth einen ganzen String, der in
> bluetoothSerial.read() >eingespeichert wird.
>
> Nein, die Funktion liefert exakt ein Zeichen pro Aufruf.
Sorry. Wie kann ich durch die Funktion jedes Zeichen nach der Reihe 
lesen?

[Falk B. (falk)]

@ D. (Gast)

>> Nein, die Funktion liefert exakt ein Zeichen pro Aufruf.
>Sorry. Wie kann ich durch die Funktion jedes Zeichen nach der Reihe
>lesen?

Hast du das nicht schon ansatzweise programmiert?
Oder nur kopiert, ohne es zu verstehen?

if(blueToothSerial.available()){
    int i;
    for(i = 0; i <= sizeof(c); i++)
    {
        c[i] = blueToothSerial.read()-128;  // um 128 Stellen nach hinten verschieben
    }
          
    UDR0 = c[i]; _delay_ms(200);
}

Da ist zwar noch viel falsch, aber das Grundgerüst stimmt.

if(blueToothSerial.available()){

Sind Daten im Puffer verfügbar?

Wenn ja, dann

c = blueToothSerial.read();

usw.