Thread: Very proud of this code.

I know it is really nothing to some of you C-gods on the board, but i'm really proud of this code.

Code:

/* find 'ei' without string functions */


#include <stdio.h>
#include <stdlib.h>


int main(void){

    char c,c1;

    while ( (c = getchar()) !='#'){
          if (c == 'e'){
             c1 = getchar();
             if (c1 == 'i'){
                printf("Found ei\n");
                }
            }

    }

     system("pause");
     return (0);


}

I was asked in my book to find all the occurances of 'ei' - without using pointers, arrays or even any string functions. And I have to say I am proud of my code. I am most impressed by the fact that I had to think logically. I am not very good at math and I turned to programming to help me develop my logic skills. I began programming in python which in my opinion is an unbelievable language. however the python code takes abstraction to the next level as is seen here

Code:

#test for ei
word = raw_input("Enter a sentence")
if  "ei" in word:
    print "ei is in sentence"
else:
    print "No\n"

Nothing wrong with python it is a RAD,versatile and flexible language. However there was not much thinking going on. With C everyithing is so basic you have to think so low level, And that is an asset. Attemping this problem even allowed me to dabble around in my IDE and though it didn't work - allowed me to tinker around with functions - which is what i will be learning next

Code:

/* find 'ei' without string functions very bad code cause i never debugged it */


#include <stdio.h>
#include <stdlib.h>
void check4ei(void);


int main(void){

    char ch = getchar();
    int space;

    while ( (ch != '#') ){
          space = 0;
          check4ei()
          if ( ch == ' ' || ch == '\t' || ch =='\n' && space == 0){
             space = 1;
             }

           else{
                continue;
                }

    }

    system("pause");
    return (0);

 }






 /* function declararion */


void check4ei(void){

    char c, c1;
    int eicount = 0;
    int space = 0;

    c = getchar();
    if (c == 'e' && space == 0){
             c1 = getchar();
             if (c1 == 'i' && space == 0){
                 eicount++;
                }

            }

  }

Anyway the program can be refactored and should. It can easily be made to find a given sequence. That is of course once user input has been validated with some C functions.
Anyway thanks to the board for allowing me to rant and for all of your help in general when I have a probelm. I know it sounds gay, but i'm really grateful. I'm learning alot in many ways. It is 12:46 here in Canada, and I'm just so proud of myself.

C can be annoying as F**k sometimes, but when you get it. Its really rewarding.

Code:

#include <stdio.h>
#include <stdlib.h>


int main(void){

    char c;

    do {
          if ((c = getchar()) == 'e'){
             if ((c = getchar()) == 'i'){
                printf("Found ei\n");
                }
            }
    } while ( c !='#');

     system("pause");
     return (0);
}

Still there is problem with the code below,

Code:

#include <stdio.h>
#include <stdlib.h>

int main(void){

    char c;

    do {
          if ((c = getchar()) == 'e'){
             if ((c = getchar()) == 'i'){
                printf("Found ei\n");
                }
            }
    } while ( c !='#');

     system("pause");
     return (0);
}

The above code does not work for the string, "eeieei" ...
So the corrected code could be,

Code:

do
{
   if((c=getchar()) == 'e')
    {
        while(c=='e')
         {
             if((c=getchar())=='i')
             {
                  printf("Found ei..\n");
                  break; // Just the same with or without it 
                             // But just to avoid condition check
             }
         }
    }
}while(c!='#');

Sorry if there is a mistake, don't have a compiler to check the code right now.

I fixed the error. This program can search for any string, not just ei.

Code:

#include <stdio.h>
#include <stdlib.h>

int main(void){
    char c;
    char searchstr[]="ei";
    int length=strlen(searchstr);
    int index=0;
    do {
          c=getchar();
          if(c == searchstr[index]) index++;
          else if(c==searchstr[0]) index=1;
          else index=0;          
          if(index == length) {
              printf("Found %s\n", searchstr);
              index=0;
          }
    } while ( c !='#');

     system("pause");
     return (0);
}

Its educational to see the different way programmers interpret a problem in their code. Thanks for the additional code guys.

Code:

int main(void)
{
    char ch = getchar();    // this is C++
    int space;

    // ch is never changed in this loop -- it will run forever
    while ( (ch != '#') )
    {

So is the comment style following that statement, its only only on some compilers. What i'm getting at for pedantic reasons, is assigning a function to a variable is still allowed it is more a stylistic approach in a tribute to the history of C isn't it? much like

Code:

int a;
scanf(%"i" , &a);

will still capture an integer value insted of the traditional %d formatter used.

Let me know if that is not correct.

3) system("pause") -- change it to cin.get(); But this will require cleaning out the cin buffer before you get to it.

This is what i was referring to when I asked if i could mix C and C++ togather. Using cin.get(). Is that possible?


Dr Zoidberg I'm unable to use string functions as yet in the assignement. So unfortunately, this code wouldn't be allowed

Code:

char searchstr[]="ei";
    int length=strlen(searchstr);

same thing here quzah.

Code:

ungetc( c, stdin );

I haven't even ran across that function yet, though I have a Idea of the codes purpose.