DaniWeb Forum Index > Software Development > C

C RSS

Array of pointers to char...messed up!

•

Join Date: Oct 2009

Posts: 63

Reputation: neithan is an unknown quantity at this point

Solved Threads: 2

neithan

Offline

Junior Poster in Training

Array of pointers to char...messed up!

28 Days Ago

I want to finaly learn what's up with pointers and arrays, so i made up this program:

Help with Code Tags

(Toggle Plain Text)

#include <stdio.h>

int main()
{
    int cantidad_frases = 0;
    printf("Cuantas frases quieres escribir?: ");
    scanf("%d", &cantidad_frases);
    char *frases[cantidad_frases];
    for(unsigned i = 0; i < cantidad_frases; i++)
    {
        for(; *frases[i]; frases[i]++)
        {
            *frases[i] = getchar();
        }
    }
    printf("\nReproduciendo las frases:\n\n");
    for(unsigned i = 0; i < cantidad_frases; i++)
    {
        printf("Frase %d:", i + 1);
        for(; *frases[i]; frases[i]++)
        {
            printf("%c", *frases[i]);
        }
        printf("\n");
    }
    return 0;
}

I had no errors nor warnings at all. But program crashes after entering the cantidad_frases value and hitting enter. The problem is in the 11th line.

I really didn't know how to "refer" each letter of the words/string stored in frases[i].
I hope i made my question clear enough so you can help me. Any advise or (please) any specific place where they explain the whole thing about pointers and arrays is more than welcome!

•

Join Date: Jan 2008

Posts: 344

Reputation: gerard4143 is on a distinguished road

Solved Threads: 44

gerard4143 gerard4143 is offline

Offline

Posting Whiz

28 Days Ago

This line should probably be:

char *frases[cantidad_frases];

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
char *frases = (char*)malloc(cantidad_frases);
char *frases = (char*)malloc(cantidad_frases);

So that you have allocated the memory...

Last edited by gerard4143; 28 Days Ago at 8:34 pm.

•

Join Date: Sep 2009

Posts: 332

Reputation: dkalita will become famous soon enough

Solved Threads: 46

dkalita

Offline

Posting Whiz

28 Days Ago

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
for(unsigned i = 0; i < cantidad_frases; i++)
    {
        for(; *frases[i]; frases[i]++)
        {
            *frases[i] = getchar();
        }
    }
for(unsigned i = 0; i < cantidad_frases; i++)
    {
        for(; *frases[i]; frases[i]++)
        {
            *frases[i] = getchar();
        }
    }

U have not assigned memory to frases[i].

Do

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
#define MAX_STR 40 // whatever u want
frases[i] = malloc(MAX_STR);
#define MAX_STR 40 // whatever u want
frases[i] = malloc(MAX_STR);

U are alos doing another mistake:

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
        for(; *frases[i]; frases[i]++)
        for(; *frases[i]; frases[i]++)

here frases[i] is getting incremented. Hence the pointer advances and when later u want the inputed string u wont have it because the pointer has advanced to the end hence it contain nothing. U should do this with some temp pointer as

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
 
for(unsigned i = 0; i < cantidad_frases; i++)
    {
        frases[i] = malloc(MAX_STR);
        char *tmp = frases[i];
        for(; *tmp; tmp++)//AGAIN HERE U SHOULD MODIFY YOUR LOGIC BECAUSE frases[i] initially contains nothing. I am leaving it to you.
        {
            *tmp = getchar();
        }
    }
for(unsigned i = 0; i < cantidad_frases; i++)
    {
        frases[i] = malloc(MAX_STR);
        char *tmp = frases[i];
        for(; *tmp; tmp++)//AGAIN HERE U SHOULD MODIFY YOUR LOGIC BECAUSE frases[i] initially contains nothing. I am leaving it to you.
        {
            *tmp = getchar();
        }
    }

Last edited by dkalita; 28 Days Ago at 3:15 am.

•

Join Date: May 2006

Posts: 3,114

Reputation: WaltP has much to be proud of

Solved Threads: 281

WaltP

Offline

Posting Sensei

28 Days Ago

And in C for(unsigned i = 0; i < cantidad_frases; i++) is illegal.
i must be defined before it is used, not in the for statement.

The 3 Laws of the Procrastination Society:
1) Never do today that which can be put off until tomorrow
2) Tomorrow never comes

•

Join Date: Oct 2009

Posts: 63

Reputation: neithan is an unknown quantity at this point

Solved Threads: 2

neithan

Offline

Junior Poster in Training

27 Days Ago

•

Originally Posted by gerard4143

This line should probably be:

char *frases[cantidad_frases];
Help with Code Tags

C Syntax (Toggle Plain Text)
char *frases = (char*)malloc(cantidad_frases);
char *frases = (char*)malloc(cantidad_frases);
So that you have allocated the memory...

•

Originally Posted by dkalita

Help with Code Tags

C Syntax (Toggle Plain Text)
 
for(unsigned i = 0; i < cantidad_frases; i++)
    {
        frases[i] = malloc(MAX_STR);
        char *tmp = frases[i];
        for(; *tmp; tmp++)//AGAIN HERE U SHOULD MODIFY YOUR LOGIC BECAUSE frases[i] initially contains nothing. I am leaving it to you.
        {
            *tmp = getchar();
        }
    }
for(unsigned i = 0; i < cantidad_frases; i++) { frases[i] = malloc(MAX_STR); char *tmp = frases[i]; for(; *tmp; tmp++)//AGAIN HERE U SHOULD MODIFY YOUR LOGIC BECAUSE frases[i] initially contains nothing. I am leaving it to you. { *tmp = getchar(); } }

char *tmp = frases[i]; <--- i'm confused here. Where does tmp point at?

•

Originally Posted by WaltP

And in C for(unsigned i = 0; i < cantidad_frases; i++) is illegal.
i must be defined before it is used, not in the for statement.

I'm told in C99 is perfectly accepted?

OK so i see my fault was all about memory allocation, so im going to learn about that cause i know nothing.
However, if you had this *frases[].. let's say it cointains:
*frases[0] -> "tutorials"
*frases[1] -> "dani web"
*frases[2] -> "it discussion"

How would you refer to the w in web at frases[1]?

Thank you

•

Join Date: Dec 2006

Posts: 2,029

Reputation: Aia has much to be proud of

Solved Threads: 176

Aia

Offline

Postaholic

27 Days Ago

•

How would you refer to the w in web at frases[1]?

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
frases[1][5];
frases[1][5];

or if you want to complicate it

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
*(*(frases + 1)+ 5)
*(*(frases + 1)+ 5)

Last edited by Aia; 27 Days Ago at 7:48 pm.

•

Join Date: Oct 2009

Posts: 63

Reputation: neithan is an unknown quantity at this point

Solved Threads: 2

neithan

Offline

Junior Poster in Training

26 Days Ago

•

Originally Posted by Aia

Help with Code Tags

C Syntax (Toggle Plain Text)
frases[1][5];
frases[1][5];
or if you want to complicate it
Help with Code Tags

C Syntax (Toggle Plain Text)
*(*(frases + 1)+ 5)
*(*(frases + 1)+ 5)

Awesome. The complicated version...heheh yes.
frases is pointing at 0, so +1 makes it look at the second phrase. And then the * dereference symbol, would return what? The first word in the first phrase? --> *( ? ) + 5 <---- ? should be a memory adress so it can be added with 5 and then dereference returning the "w" letter. But how come just doing *(frases + 1) returns not the value but a memory address?

Instead of mallocation before user's input, isn't the way to allocate at the same time user inputs so you reserve the exact memory needed? Like in run time.

Thank you for your time, i'm learning a lot. I'm gonna focus learning memory, malloc and pointers stuff more.

•

Join Date: Oct 2009

Posts: 63

Reputation: neithan is an unknown quantity at this point

Solved Threads: 2

neithan

Offline

Junior Poster in Training

25 Days Ago

I made it! It works perfectly thanks to your advices and further studying hehe. Any comments / corrections?

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
    int cantidad_frases = 0;
    printf("Cuantas frases quieres escribir?: ");
    scanf("%d", &cantidad_frases);                  
    getchar();                                      
    char *frases[cantidad_frases];
    for(unsigned i = 0; i < cantidad_frases; i++)
    {
        frases[i] = (char *) malloc(sizeof(char[20]));
        char *tmp = frases[i];
        printf("Escribe la frase %d: ", i+1);       
        while((*tmp = getchar()))
        {
            if(*tmp == 10)
            {
                *tmp = '\0';
                break;
            }
            tmp++;
        }
    }
    for (unsigned i = 0; i < cantidad_frases; i++)
    {
        puts(frases[i]);
    }
 
    return 0;
}
#include <stdio.h>
#include <stdlib.h>

int main()
{
    int cantidad_frases = 0;
    printf("Cuantas frases quieres escribir?: ");
    scanf("%d", &cantidad_frases);                  
    getchar();                                      
    char *frases[cantidad_frases];
    for(unsigned i = 0; i < cantidad_frases; i++)
    {
        frases[i] = (char *) malloc(sizeof(char[20]));
        char *tmp = frases[i];
        printf("Escribe la frase %d: ", i+1);       
        while((*tmp = getchar()))
        {
            if(*tmp == 10)
            {
                *tmp = '\0';
                break;
            }
            tmp++;
        }
    }
    for (unsigned i = 0; i < cantidad_frases; i++)
    {
        puts(frases[i]);
    }

    return 0;
}

I was turning mad when i saw phrase one was skipped. I realized there was a remanent \n in the buffer (by the line feed in the cantidad_frases selection). So i added the getchar() after the scanf to clean that up. Is there any elegant way to do it? EDIT: other than using another input function.

Last edited by neithan; 25 Days Ago at 8:46 pm.

•

Join Date: Dec 2006

Posts: 2,029

Reputation: Aia has much to be proud of

Solved Threads: 176

Aia

Offline

Postaholic

24 Days Ago

•

Any comments / corrections?

In the C89 standard, an array must be set with a constant. Which would not allowed what you just did.

 Help with Code Tags 
 c Syntax (Toggle Plain Text) 
 int cantidad_frases = 0;
    printf("Cuantas frases quieres escribir?: ");
    scanf("%d", &cantidad_frases);                  
    getchar();                                      
    char *frases[cantidad_frases]; /* This is not allowed in C89 */
 int cantidad_frases = 0;
    printf("Cuantas frases quieres escribir?: ");
    scanf("%d", &cantidad_frases);                  
    getchar();                                      
    char *frases[cantidad_frases]; /* This is not allowed in C89 */

Furthermore, there can not be mixing of variable declarations and code. All variable must be declared first in the block. Making char *frases[cantid_frases]; incorrect as well.
If you want dynamically allocated multidimensional arrays you may take a look at pointer to pointer. Example

C89 doesn't accept declaration of variable in the for loop construct.

Help with Code Tags

(Toggle Plain Text)

for(unsigned i = 0; i < cantidad_frases; i++)

Variable i must be declared outside the ().

When you ask for user input, the control of the program is handled to the user. They can enter an integer (in which case an enter key has been added to the stream), they can enter a float, a char or any combination and length of those. Moreover they might just press enter or be generous with blank sequence.
Can scanf() handle that kind of conditions? The answer is no. The elegant option then is to think how would you deal with the input and design your program to accommodate for those possibilities. There's not "silver-bullet function" that will do that for you.

•

Join Date: Oct 2009

Posts: 63

Reputation: neithan is an unknown quantity at this point

Solved Threads: 2

neithan

Offline

Junior Poster in Training

#10

24 Days Ago

•

Originally Posted by Aia

In the C89 standard, an array must be set with a constant. Which would not allowed what you just did.
Furthermore, there can not be mixing of variable declarations and code. All variable must be declared first in the block. Making char *frases[cantid_frases]; incorrect as well.

Ok but, i'm sorry if this sound ignorant which probably will, isn't C89 too old? I look it up and this C99 thing works with those issues, right? Why still thinking of C89? (remember i'm a beginner)

•

Originally Posted by Aia

If you want dynamically allocated multidimensional arrays you may take a look at pointer to pointer. Example

I'm going to check that url and see what i can learn.

•

Originally Posted by Aia

When you ask for user input, the control of the program is handled to the user. They can enter an integer (in which case an enter key has been added to the stream), they can enter a float, a char or any combination and length of those. Moreover they might just press enter or be generous with blank sequence.
Can scanf() handle that kind of conditions? The answer is no. The elegant option then is to think how would you deal with the input and design your program to accommodate for those possibilities. There's not "silver-bullet function" that will do that for you.

That is true...i'll see if i can improve that.

Thank you.