Types problem

[westony]

void Prints(char **menu)
{
   while(**menu)
   {

        printf("%s  ", *(*(menu)+i));

    }
}

giving me an error in printf:
Unhandled exception at 0x583814cf (msvcr100d.dll) 0xC0000005: Access violation reading location 0x00000066.

problem comes from **menu I am sure but I dont know how to fix it ? What is the type if **menu ???

[Adak]

A **variable is a pointer to a pointer to variable, and I doubt that is what you want to print your menu.

If menu is a string (char's WITH an end of string marker on the end of them: '\0'), then you can print that string with just printf("%s ", menu);

And Welcome to the forum, Westony!

[westony]

Tnx for the welcome but i have

char *menu[] = {"data", "comes", "here"};

i know how to print it pointer by pointer but i wanna try to print it symbol by symbol

i for get one i++ but this does not matter still giving me the error and and still **menu what TYPE OF DATA is it ???

[Adak]

Tnx for the welcome but i have

char *menu[] = {"data", "comes", "here"};

i know how to print it pointer by pointer but i wanna try to print it symbol by symbol

i for get one i++ but this does not matter still giving me the error and and still **menu what TYPE OF DATA is it ???

Try:

for(i=0;i<3;i++)
  printf("\n%s", menu[i]);

Or:

for(i=0;i<SIZE of menu; i++)
  printf("%c", menu[i]

But you'll have to tell SIZE of menu, how many char's you want to print. Once you go into another function, it needs that info.



Just a call: 
myFunction(menu);

and then the function, itself:
void myFunction(char *menu)

**menu is a pointer to a pointer to your menu.

[westony]

ok but isn't this printf("%c", menu[i]) join to print them pointer by pointer data comes here!!!



i wanna print it symbol my symbol d a t a c o m e s h e r e ????

[laserlight]

So, menu points to the first element of an array of pointers to char, where each pointer to char points to the first character of a string. You want to print these strings.

Since there is no way to determine the number of menu items given just this menu pointer, you should pass the number of items as an argument, and thus be able to write say:

void printMenu(char **menu, size_t num_items)
{
    size_t i;
    for (i = 0; i < num_items; ++i)
    {
        printf("%s ", menu[i]);
    }
}

Incidentally, you probably should have written instead:

const char *menu[] = {"data", "comes", "here"};

since string literals are not to be modified.

[Adak]

I wonder if this is what he was looking for?

#include <stdio.h>
#include <string.h>

int main() {
  int i, j, n; 
  const char *menu[]={"data", "comes", "here"};


  printf("\n\n");

int main() {
  int i, j, len, n; 
  char *menu[]={"data", "comes", "here"};


  printf("\n\n");
  for(i=0;i<sizeof(menu)/sizeof(menu[0]);i++) {
    len=strlen(menu[i]);
    for(j=0;j<len;j++) {
      printf("%c ", menu[i][j]);
    }
  }
  printf("\n\n\t\t\t     press enter when ready");

  i = getchar(); ++i;
  return 0;
}

Output is: d a t a c o m e s h e r e

Edit: Thanks, Laserlight! All fixed!

[westony]

If I read right there is no way to do it by my way ???
Because there is no way to determine the number of menu

and can you help me please understand the meaning of this size_t because in could not understood it in MSDN

[Adak]

We've shown you two ways to print it out. You said you knew another way with pointers.

Seems to cover the subject, no?

No, not quite that way, but you can print it out with pointers, instead of using an index, of course.

[laserlight]

Because there is no way to determine the number of menu

Of course, given the array, you can indeed determine the number of menu items, whether by counting manually or by using sizeof(menu) / sizeof(menu[0]). The problem is that in the function, you only have a pointer to a pointer, thus using sizeof will not work to find the number of menu items. Hence, you pass the number of items as an argument.

and can you help me please understand the meaning of this size_t because in could not understood it in MSDN

It is just an unsigned integer type commonly used for values that represent sizes in the C standard library. It is also the type of the result of the sizeof operator.

Adak's example looks like it might be what you have in mind, but of course you should fix the presumably deliberate bug and move the nested loops into a function as you desire. In the process of fixing that bug you would then be passing the size as an argument. You also might want to cache the result of strlen instead of calling it on every iteration.

[Adak]

You mean the int instead of size_t? (I confess, I seldom use size_t for anything).

I ran the program - ran fine. I was in a hurry though. Just an hour before the Tour de France live coverage starts.

[laserlight]

You mean the int instead of size_t?

No, the use of sizeof(menu) instead of sizeof(menu) / sizeof(menu[0]).

[westony]

I wonder if this is what he was looking for?

#include <stdio.h>
#include <string.h>

int main() {
  int i, j, n; 
  const char *menu[]={"data", "comes", "here"};


  printf("\n\n");
  for(i=0;i<sizeof(menu);i++)
    for(j=0;j<strlen(menu[i]);j++) {
      printf("%c ", menu[i][j]);
    }

  printf("\n\n\t\t\t     press enter when ready");

  i = getchar();
  return 0;
}

Output is: d a t a c o m e s h e r e

my VS shows me again that error :@ ;
Unhandled exception at 0x5805d540 (msvcr100d.dll) in TestingZone.exe: 0xC0000005: Access violation reading location 0xcccccccc.

My syntax is the same but with pointers!!!

[laserlight]

westony: read my comments about Adak's (now-fixed) bug.

[Adak]

That was just my flub - thanks, and it's fixed now.

Edit: Sorry, Westony! Get the updated version of that post!