kleines C problem

[holaco (Gast)]

hallo leute, ich hab folgendes programm der zählen muss wieviele 
buchstaben ein wort enthält, es funktioniert leider nur wenn ich wörter 
eingebe die gerade eine anzahl von buchstaben hat. kann mir mal jemand 
helfen? danke im voraus.

#include<stdio.h>
void main()
{
  char text1[20];
  int i;

  printf("geben sie ein wort ein:\n");
  gets(text1);

  for(i=0;text1[i];i++)
    i++;
  printf("das wort hat %i buchstaben\n",i);
}

[Stefan Salewski (Gast)]

for(i=0;text1[i];i++)
  i++;

iplussplus doppelt?

[NurEinGast (Gast)]

for(i=0;text1[i];i++)
    i++;

hier wird "i" zweimal incrementiert.

[holaco (Gast)]

mann mann ich habs übersehen, vielen vielen dank.

[Ich (Gast) (Gast)]

> for(i=0;text1[i];i++)
>    i++;
>
> hier wird "i" zweimal incrementiert.

>> mann mann ich habs übersehen, vielen vielen dank.

Das springt doch eigentlich sofort ins Auge.
Was machst du denn bei >10000 Zeilen Code... ;-)

[Johnny (Gast)]

Am besten Du gewöhnst Dir an, die geschweiften Klammern bei Schleifen 
etc. immer zu schreiben. Der Code wird übersichtlicher und man muss 
nicht immer überlegen, ob die nachfolgenden Zeilen ausgeführt werden 
oder nicht. Es sticht einem dann geradezu in die Augen ;-) Wenigstens 
hast Du die zweite Zeile mit dem i++; eingerückt, das ist schon mal ein 
Schritt in die richtige Richtung.

[P. S. (Gast)]

Gross-/Kleinschreibung hilft dem Gehirn auch ungemein. Und 
Variablennamen mit mehr als einem Buchstaben...

[Timmo H. (masterfx)]

Wieso für eine zählervariable mehr als einen Buchstaben nehmen? Ok in 
diesem fall könnte man wirklich count oder so nehmen aber egal. Ich 
hätte es so gemacht:

void main(){
  char text1[20];
  int i=0;

  printf("geben sie ein wort ein:\n");
  gets(text1);

  while(text1[i++]) ;
  printf("das wort hat %i buchstaben\n",i);
}

[Peter (Gast)]

und hoffen das das Wort nicht länger als 20 Zeichen ist.
Da wird wohl bald der nächste Patch fällig sein.

[Volker (Gast)]

... nicht länger als 19. Die /0 muß ja auch noch reinpassen :-)

[Timmo H. (masterfx)]

Stimm für sowas sollte man lieber getch nehmen:

#define COMMAND_BUFFER 20
void main(){
  char debug[COMMAND_BUFFER];  /* Buffer für Debug-Kommando */
  unsigned char   i;
  
  printf("geben sie ein wort ein:\n");
  while(1){
    i = 0;
    do{
       debug[i] = getch();
       putc(debug[i]);      /* Echo */
       if(debug[i] == '\b' && i){  /* Backspace? */
         putc('\b');
         i--;        /* letztes Zeichen löschen */
       }
       else{
         i++;  
       }
       if(i >= COMMAND_BUFFER -1){  /* String zu lang */
         i = 0;
         printf("\nString zu lang!\n");
         continue;  
       }
       /* Schleife bei ENTER verlassen */  
     }while(debug[i-1] != '\r' && debug[i-1] != '\n');
     
     debug[i-1] = '\0';  /* Stringterminierung */
     printf("das Wort hat %i buchstaben\n",i-1);

}