cien

[Textarbeit (Gast)]

ich hab mir grad mal einen code "zusammengeschustert" für ein Lauflicht, 
klappt auch alles, ist aber eher un schön gemacht.
irgendwelche Tipps für die Funktion move_text um diese effizienter zu 
machen???

#include <avr/io.h>
#include <stdlib.h>
#include <util/delay.h>
#include <string.h>

#include <lcd.h>

#define length 30
char sLauftext[length+1];

char* move_text(char* test)
{
 char sTmp[2];
 char sLauftextTmp[length+1];
 uint8_t i;
 
 strcpy(sLauftext,test);
 strcpy(sLauftextTmp,sLauftext);

 sTmp[0]=sLauftext[0];
 sTmp[1]='\0';
 for (i=1;i<length+1;i++)
 {
  sLauftextTmp[i-1]=sLauftext[i];  
 }
 strcat(sLauftextTmp,sTmp);
 strcpy(sLauftext,sLauftextTmp);

 return sLauftext;
}


void wait (uint16_t zeit) //zeit: wartezeit in ms (0 bis 65536)
{
   uint16_t i;
   for(i=0;i<zeit;i++) _delay_ms(1); //da für _delay_ms max Takt-abhängige Werte erlaubt
} 

int main (void) 
{  
  char sLauftext[length+1];

  lcd_init(LCD_DISP_ON);
  lcd_clrscr();
  strcpy(sLauftext,"Dies ist der Lauftext Test   ");
  lcd_puts(sLauftext);
  wait(1000);

while(1)
{ 
  lcd_clrscr();
  strcpy(sLauftext,move_text(sLauftext)); 
  lcd_puts(sLauftext);
  wait(200);
}

return 0;           
}

[Xenu (Gast)]

char* move_text(char* text)
{
  char tmp = text[length - 1];

  int n;
  for(int n = length; n > 1; n--)
  {
    text[n-1] = text[n-2];
  }

  text[0] = tmp;

  return text;
}

[Xenu (Gast)]

Das "int" nach "for" muss noch weg.

[Textarbeit (Gast)]

ist ja schon mal kompakter, aber der text sollte schon nach links 
laufen, damit man auch vom anfang anfängt zu lesen und nicht am ende 
startet

[Xenu (Gast)]

Wieso programmierst Du es dann nicht einfach um?

[Textarbeit (Gast)]

da hab ich aber schon wieder das problem mit dem char welches 
zwischengespeichert wird
der Text wird jetzt zwar nach links durchgeschoben, aber irgendwie 
klappt das mit dem Anfügen des ersten Buchstaben nicht (deswegen hatt 
ichs so umständlich über string operationen)

char* move_text(char* text)
{
  char tmp = text[0];

  uint8_t i;
  for( i = 0; i<length; i++)
  {
    text[i] = text[i+1];
  }
  text[length] = tmp;

  return text;
}

wo ist der denkfehler?

[cien (Gast)]

ich merk grad ich hab im ersten post aus versehen Lauflicht geschrieben, 
soll natürlich Lauftext heißen

[cien (Gast)]

das Problem ist einfach, dass der Text zwar ordendlich nach links 
durchgereicht wird, aber der Text eben bis zum Ende durchgereicht wird 
und dann wieder von vorne startet und nicht wie in meiner 
"zusammengeschusterten" version der text vorne abgeschnitten und hinten 
wieder angefügt wird um einen flüssigen Übergang zu schaffen
@admin: wäre es möglich den Namen des threads in Textarbeit zu ändern, 
hatte da zwei zeilen vertauscht :(

[cien (Gast)]

ok, es lag an "length", da dies ein fester wert war und nicht der 
tatsächlichen Länge des strings entsprach

char* move_text(char* text, uint8_t len)
{
  char tmp = text[0];  
  uint8_t i;

  for( i = 0; i<len; i++)
  {
    text[i] = text[i+1];
  }
  text[len-1] = tmp;
  return text;
}

habs einfach so gelöst, dass ich einmalig im main-Programm dei Länge 
abfrage und diese einfach der Funktion mitgebe
und siehe da, es klappt