anagrams

i'm trying to make an anagram program, but i'm stuck on what to do within my setLetters function

Code:

---

#include <stdio.h>
#include <ctype.h>
#define MAX 100
//void initialize(int intarray[]);//initializes strings probably
//char *gets_s(char s[], int num);, terminates a string with \0
void getString(char string1storage[], char string2storage[]);
void setLetters(char string1storage[], int intarray[]);
void main(void)
{//strings need2be initialized with anagram first?
       
        char string1storage[MAX], string2storage[MAX];//stores anagrams
        int intarray[MAX] = {0};
        //initialize(); //maybe call him after getstring?
        getString(string1storage, string2storage);
        setLetters(string1storage, intarray);
}

/*void initialize(intarray[])
{
        intarray[]={0};
}*/

void getString(char string1storage[], char string2storage[])
{       
        printf("Please enter a word or phrase");
        gets_s(string1storage, MAX - 1);
        printf("Please enter a second word or phrase");
        gets_s(string2storage, MAX - 1);
}

void setLetters(char string1storage[], int intarray[])//runs through all characters in string1storage and updates the letter count in intarray
{
        int index;
        char ch;

        index = 0;
        while(string1storage[index])
        {//isalpha returns true if lower/uppercase letter
                if(isalpha(string1storage[index]));//if true
                        string1storage[index] = 0;//"skip or delete whatever inside there", i'm stuck, i'm intending to delete or ignore non-alphabet chars in the array
                index++;
        }
       
        index = 0;
        ch = string1storage[index];
        while(string1storage[index])
        {
                //97-122--> decimals for ACSII are a-z lowercase
                string1storage[index] = tolower(string1storage[index]);
                index++;
        }
       
       
        printf("testing %s\n", string1storage);

        index = 0;
       
        index = (int) (ch - 'a');//do not replace the a,sees letters in string1storage and increments them by 1 in intarray[index]
       
        printf("testingtwo %d\n", index);
}

---

-thanks

Code:

---

                if(isalpha(string1storage[index]));//if true
                        string1storage[index] = 0;//"skip or delete whatever inside there", i'm stuck, i'm intending to delete or ignore non-alphabet chars in the array
                index++;

---

Remove that.


Quzah.

how would i remove/ignore nonalphabet characters if i removed that?

Look carefully and match up the colors. The magenta 'that' refers to the magenta ';' at the end of your if line in the code block.

o ok thanks, can i have more than one \0 in an array?

Sure, but when you treat it as a string, it will always stop at the first \0.

o ok thanks, how would i print an array of integers? when i try to the output is a happy face

Hard to say without knowing what piece of code is producing this error. My guess is you're using %c instead of %d. Read up on the printf documentation.

if i use %d it produces garbage values

Okay...what printf line is producing this problem? Can you give me some sample input so I can reproduce this?

EDIT: And an up to date version of your code.

Code:

---

#include <stdio.h>
#include <ctype.h>
#define MAX 100
#define LETTERS 26
//void initialize(int intarray[]);//initializes strings probably
//char *gets_s(char s[], int num);, terminates a string with \0
void getString(char string1storage[], char string2storage[]);
void setLetters(char string1storage[], int intarray[]);
void checkLetters(char string2storage[], int intarray[]);
void main(void)
{//strings need2be initialized with anagram first?
       
        char string1storage[MAX], string2storage[MAX];//stores anagrams
        int intarray[LETTERS] = {0};
        //initialize(); //maybe call him after getstring?
        getString(string1storage, string2storage);
        setLetters(string1storage, intarray);
        checkLetters(string2storage, intarray);
}

/*void initialize(intarray[])
{
        intarray[]={0};
}*/

void getString(char string1storage[], char string2storage[])
{       
        printf("Please enter a word or phrase");
        gets_s(string1storage, MAX - 1);
        printf("Please enter a second word or phrase");
        gets_s(string2storage, MAX - 1);
}

void setLetters(char string1storage[], int intarray[])//runs through all characters in string1storage and updates the letter count in intarray
{
        int index, indexforintarray;
        //char ch;

        /*index = 0;
        while(string1storage[index])
        {//isalpha returns true if lower/uppercase letter
                if(isalpha(string1storage[index]))//if true
                        string1storage[index] = '\0';//"skip or delete whatever inside there", i'm stuck, i'm intending to delete or ignore non-alphabet chars in the array
        //use null^?
                        index++;
        }
        printf("%s\n", string1storage);*/
        index = 0;
        //printf("%d\n", isalpha(string1storage[0]));
        while(string1storage[index])
        {
                string1storage[index] = tolower(string1storage[index]);
                index++;
        }
       
        indexforintarray = 0;
        index = 0;
        while(string1storage[index])//for(index = 0; index != '\0'; index++)//use on string1storage
        {
                indexforintarray = (int) (string1storage[index] - 'a');
                //intarray[indexforintarray] = string1storage[index];//wtf is this
                if(indexforintarray >= 0 && indexforintarray <= 25)//probably ignores .!123*+ etc., makes isalpha unneeded
                        intarray[indexforintarray]++;
                index++;
        }
        /*printf("testing %s\n", string1storage);
        printf("testing %d\n", intarray);
        printf("testing %s\n", intarray); */
}

void checkLetters(char string2storage[], int intarray[])
{
        int indexforintarray, lettermatch, index;

        index = 0;
        //printf("%d\n", isalpha(string1storage[0]));
        while(string2storage[index])
        {
                string2storage[index] = tolower(string2storage[index]);
                index++;
        }
       
        indexforintarray = 0;
        index = 0;
        while(string2storage[index])//for(index = 0; index != '\0'; index++)//use on string1storage
        {
                indexforintarray = (int) (string2storage[index] - 'a');
                //intarray[indexforintarray] = string1storage[index];//wtf is this
                if(indexforintarray >= 0 && indexforintarray <= 25)//probably ignores .!123*+ etc., makes isalpha unneeded
                        intarray[indexforintarray]--;
                index++;
        }

        lettermatch = 0;
        for(indexforintarray = 0; indexforintarray < 26; indexforintarray++)
        {
                if(intarray[indexforintarray] > 0)//avoids adding negative numbers
                        lettermatch+=intarray[indexforintarray];
        }

        if(lettermatch == 0)
                printf("Your inputs are anagrams");
        else
                printf("Your inputs are not anagrams");

}

---

i dont understand why my setletters output isn't showing in the program
postedit:edited the code twice sorry, works perfectly now, but i want to integrate the initialize function in my program now for usage.

It's int main(void) and return an integer at the end, usually 0.

Code:

---

        if(lettermatch == 0)

---

You need a second = here. One = is for assignment, two == is for comparison.

Quote:

---

i dont understand why my setletters output isn't showing in the program

---

Perhaps because you commented out all the print statements? Your two loops in there can be merged into one that ignores all non-alpha chars:

Code:

---

index = 0;
for each character in string1storage
    if isalpha is true
        convert it to lower
        subtract 'a' from it
        increment the count of that index in intarray

---

checkLetters will do something very similar.