need help with strcpy...

[CbbLe]

Im going through a list of names(namn[m]) and I want to compare them with a few other names (tankbaraNamn[k]).....now my problem is:

if a name, in the array tankbaraNamn[k], can't be found in the array, name[m], I want the program to simply print the name which couldn't be found (tankbaraNamn[k] and say that it's not in the list. My strcpy isn't working though, I get error messages whenever I try to copy a specific part of the array(tankbaraNamn[k] to another string(tempNamn)..I don't know how to solve this...can anyone please help me?

Im using the program Dev-C++.

NOTE: tankbaraNamn[k] and tempNamn are declared:

char tankbaraNamn[5][15], tempNamn[15];

tankbaraNamn[k] has previously in the code scanned the strings which should be located there (and it works as it should)

//start on error--------------------------------

for(k=0;k<poster;k++)
 for(m=0;m<100;m++)
 {
  if((strcmp(tankbaraNamn[k], namn[m])==0))
  {
   hittat=1;
   printf("%s finns p? plats %d\n",tankbaraNamn[k], m);
  }
  else
  {
   strcpy(tempNamn, tankbaraNamn[k]);
   hittat=0;
  }
  if(hittat==0)
   printf("%s finns inte med p? listan\n", tempNamn);
  else
  ;
 }

//end of code------------------------------------

plz help a poor gu:(:(

[kjordan2001]

What's the type of these arrays? string? char[]?

[kjordan2001]

nm, saw your types after reading again. What's the error? Try making them pointers, since I think strcpy wants pointers if it's a compile error.

[subflood]

use strncpy

what error are you getting?

[CbbLe]

can't convert char to int (or maybe it was cannot convert int to char)..

the error is at the same row as strcpy.

[Lycander]

How about instead of making a copy of the name (char*) that is not found, make a note of the index. Like this:

for(k=0;k<poster;k++)
{
	for(m=0;m<100;m++)
	{
  if((strcmp(tankbaraNamn[k], namn[m])==0))
  {
 	 hittat=1;
 	 printf("%s finns p? plats %d\n",tankbaraNamn[k], m);
  }
  else
  {
 	 hittat=0;
 	 printf("%s finns inte med p? listan\n", tankbaraNamn[k]);
  }

	}
}

[CbbLe]

Yea, but then the program will print out the names it doesn't find several times (like 80)...I only want it printed once..which is why I have to strcpy somehow.

someone solve this please :(

[kjordan2001]

Yea, but then the program will print out the names it doesn't find several times (like 80)...I only want it printed once..which is why I have to strcpy somehow.

someone solve this please :(

That seems to be what you're doing anyways...execpt with an extra if statement.

[CbbLe]

"That seems to be what you're doing anyways...execpt with an extra if statement"

ye but the difference is that those words can only be printed once (since they will be found 1 time for sure) but the other words is printed every time the for-loop counts +1 since they will never be found

[kjordan2001]

"That seems to be what you're doing anyways...execpt with an extra if statement"

ye but the difference is that those words can only be printed once (since they will be found 1 time for sure) but the other words is printed every time the for-loop counts +1 since they will never be found

for(k=0;k<poster;k++)
{
count = 0;
for(m=0;m<100;m++)
{
 if((strcmp(tankbaraNamn[k], namn[m])==0))
 {
 ?printf("%s finns p? plats %d\n",tankbaraNamn[k], m);
 ?++count;
 }
 else if (count <= 0)
 {
 ?printf("%s finns inte med p? listan\n", tankbaraNamn[k]);
 }

}
}

Edit: Guess you want it printed out multiple times if it's found multiple times?

Edited July 23, 2004 by kjordan2001

[CbbLe]

no, I want everything printed only once...if the tankbaraNamn[k] is located in the namn[m] the name should only be printed once...if it's not found, the tankbaraNamn[k] should be printed too (once).

[azcodemonkey]

for(k=0;k<poster;k++)
{
count = 0;
for(m=0;m<100;m++)
{
if((strcmp(tankbaraNamn[k], namn[m])==0))
{
 printf("%s finns p? plats %d\n",tankbaraNamn[k], m);
 ++count;
}


} // inner for-loop
   if (count <= 0)
   {
       printf("%s finns inte med p? listan\n", tankbaraNamn[k]);
   }
}// outer for-loop

Wouldn't you want to put the test for count being <= 0 outside of the inside loop if you only wanted printed once? Same should go for printing if found.

[kjordan2001]

no, I want everything printed only once...if the tankbaraNamn[k] is located in the namn[m] the name should only be printed once...if it's not found, the tankbaraNamn[k] should be printed too (once).

Okay, then you should go for the break code I had put before but then changed:

for(k=0;k<poster;k++)
{
count = 0;
for(m=0;m<100;m++)
{
if((strcmp(tankbaraNamn[k], namn[m])==0))
{
 printf("%s finns p? plats %d\n",tankbaraNamn[k], m);
 ++count;
 break;
}
}
if (count <= 0)
{
 printf("%s finns inte med p? listan\n", tankbaraNamn[k]);
}

}

This way you don't waste time looking for something you've already found since you said you only want it printed out once.

[CbbLe]

aaah, weenur..I love you :)

thanks to everyone who helped me with my problem, now it works as it should ^^