Thread: Char Arrays + Functions + For

Because buf is of type char[][] and word is of type char[].
So when you do buf[i], it becomes char[] and word[i] becomes char.
char[] != char.
Types mismatch.
It's like trying to assign a character to a string... but where in the string do you want to put that character?

Originally Posted by lautarox

Yes, sure, but why I'm having that error? why can't I assign that value to the buffer var? that will just help me a lot

Oh no it won't. That's why I wanted to know what you are trying to do, because I think what you have done so far is pretty much beyond repair for any purpose. As laserlight, etc, have been trying to point out, this is because you are using a 2D array as if it were a 1D array, which implies you don't understand the difference.

A simple drawing to show what you are doing:
buf looks like:
[ ] [ ] [ ] [ ] [ ]
[ ] [ ] [ ] [ ] [ ]
[ ] [ ] [ ] [ ] [ ]
[ ] [ ] [ ] [ ] [ ]
[ ] [ ] [ ] [ ] [ ]

And word looks like:
[ ] [ ] [ ] [ ] [ ]

So now you choose a row in the buf, and you will get this:
[ ] [ ] [ ] [ ] [ ]

And then you choose an element in word, and you simply get one element:
[ ]

But where among the elements above do you wish to assign it? The compiler does not know.
buf[n]: [ ] [ ] [ ] [ ] [ ]

^
|
|

word[n]: [ ]

?

This is why it isn't working.

Originally Posted by lautarox

Well.. but there I'm calling the first array of the 2d array, how can this assignment be done?

What do you mean by "I'm calling the first array of the 2d array"?

If you're looking for how to find anagrams, this may help:
http://netlib.bell-labs.com/cm/cs/pearls/sketch02.html
http://netlib.bell-labs.com/cm/cs/pearls/s02.pdf

Nice to see you are using printf to do some debugging!

(edit: sorry, now I was misreading you)



Anyway, I will keep reading thru it...

Originally Posted by MK27

I presume you mean to move a pointer forward. That would be cant++...

No can do. "cant" is an array, not a pointer.

Sorry, I had to pause for food. But now let's look at your first for loop:

Code:

for(i=0;i<lenght;i++){
		buff[i] = word[i];

So we are copying word into buff one letter at a time...

Code:

		for(j=0;j<lenght;j++){
			if(buff[i] == word[j]){
                                    cant[i]++;
			}

Now, we check to see if this letter is in word, and if so add one to cant[i]. Hmmm...

• you didn't initialize cant to anything, use int cant[6]={0} to make the whole array 0.
• since you copied from word to start with, this will always be true, so why not skip this inner loop and just say cant[i]++ ?

So the result of this is that word is copied into buff, and each element of cant is incremented by one. Let's move on to the second loop (skipping the printfs):

Code:

seclenght = strlen(tolook);
for(i=0;i<lenght;i++){
		for(j=0;j<seclenght;j++){
			if(buff[i] == tolook[j]){
				seccant[i]++;
			}
		}
	}

Now we iterate through buff, checking each letter to see if it's in tolook. If so, the element of seccant (which is undefined again) corresponding to that letter is incremented by one.
Next:

Code:

        j=0;
	int l;         // what is this for and why is it here?
	for(i=0;i<lenght;i++){
			if(cant[i] <= seccant[i]) {
			j++;
		}
	}

First, j is reset to use as a counter in the loop, where we compare cant to seccant. Since (if the two arrays had been initialized to zero, that is) cant={1,1,1,1,1,1} and seccant's elements will be either zero or one, j will equal the number of letters from word that found a match in seccant. Then:

Code:

if(j==lenght) {
		return 1;
	}
	else {
		return 0;
	}

Wow, lottarocks, this might actually work if you had initialized your arrays to zero! But there is a lot of unnecessary stuff going on. How about this:

Code:

int cant[6]={0}, seccant[6]={0}, matches=0;
for(i=0;i<lenght;i++){                    // matches is new to replace j for counting 
		buff[i] = word[i];
                cant[i]++;
}
seclenght = strlen(tolook);
for(i=0;i<lenght;i++){
		for(j=0;j<seclenght;j++){
			if(buff[i] == tolook[j]){
				matches++;
			}
		}
	}
if(matches==lenght) {
		return 1;
	}
	else {
		return 0;
	}

If you look at this a little bit you should be able to see how to streamline things even further.

Originally Posted by lautarox

Cantidad de buff[0] = 100 -> Why am I getting this high numers?
Cantidad de buff[1] = 101
Cantidad de buff[2] = 97
Cantidad de buff[3] = 114

Hey great, these are actually letters:
100=d
101=e
97=a
114=r

google "codigos ascii" (or ascii table) and you will quickly figure it out. It's very important to understand with C and probably all computer languages.