it is not printing the right values

[prashant.khade1623]

main()
{

int *ptr;
int *p;
p=ptr;
int i;
ptr = malloc (10);

for (i =0;i<10;i++)
ptr[i++] = 2;

for (i =0;i<10;i++)
printf("%d\n",p[i]);
}
the output is something like this

0
4096
132617
-1075507061
4
12643904
3
100
0
2

[Chris McDonald]

"pr****************@gmail.com" <pr****************@gmail.comwrites:

>main()
{

>int *ptr;
int *p;
p=ptr;
int i;
ptr = malloc (10);

>for (i =0;i<10;i++)
ptr[i++] = 2;

>for (i =0;i<10;i++)
printf("%d\n",p[i]);
}

>the output is something like this

>0
4096
132617
-1075507061
4
12643904
3
100
0
2

Unsurprising.
Track the values of the variable 'i'.

--
Chris.

[WANG Cong]

pr****************@gmail.com wrote:

main()
{

int *ptr;
int *p;
p=ptr;
int i;
ptr = malloc (10);

<snip>

It's wrong here. 'ptr' gets a new value via calling malloc while 'p' still
uses its old one, i.e. an uninitialized value.

--
Hi, I'm a .signature virus, please copy/paste me to help me spread
all over the world.

[prashant.khade1623]

On Apr 10, 2:07*pm, WANG Cong <[e-mail]:

[e-mail] wrote:

main()
{

int *ptr;
int *p;
p=ptr;
int i;
ptr = malloc (10);

<snip>

It's wrong here. 'ptr' gets a new value via calling malloc while 'p' still
uses its old one, i.e. an uninitialized value.

--
Hi, I'm a .signature virus, please copy/paste me to help me spread
all over the world.

I have changed the place where p = ptr. But still I am getting wrong
answer

#include<stdio.h>
main()
{

int *ptr;
int *p;
int i;
ptr = malloc (10);

p=ptr;
for (i =0;i<10;i++)
ptr[i++] = 2;

for (i =0;i<10;i++)
printf("%d\n",p[i]);

}
~
~
2
0
2
135153
2
0
2
0
2
0

[Richard]

"pr****************@gmail.com" <pr****************@gmail.comwrites:

On Apr 10, 2:07Â*pm, WANG Cong <[e-mail]:

>[e-mail] wrote:

main()
{

int *ptr;
int *p;
p=ptr;
int i;
ptr = malloc (10);

<snip>

It's wrong here. 'ptr' gets a new value via calling malloc while 'p' still
uses its old one, i.e. an uninitialized value.

--
Hi, I'm a .signature virus, please copy/paste me to help me spread
all over the world.

I have changed the place where p = ptr. But still I am getting wrong
answer

#include<stdio.h>
main()
{

int *ptr;
int *p;
int i;
ptr = malloc (10);

You probably meant something like this:

ptr = malloc(sizeof(i)*10);

>
p=ptr;
for (i =0;i<10;i++)
ptr[i++] = 2;

did you mean this instead?

for (i =0;i<10;i++)
ptr[i] = 2;

[Ben Bacarisse]

"pr****************@gmail.com" <pr****************@gmail.comwrites:

main()

int main(void) is much better.

{

int *ptr;
int *p;
p=ptr;
int i;
ptr = malloc (10);

No one has yet said that malloc does not know what you are
allocating. You can't just say 10, because that allocates 10 bytes.
You need malloc(10 * sizeof *ptr) which means, rather literally,
"allocate space big enough for 10 things of the type ptr points to".

for (i =0;i<10;i++)
ptr[i++] = 2;

for (i =0;i<10;i++)
printf("%d\n",p[i]);

As already pointed out, p is not properly set. It does not point at
an array whose elements you can print like this.

}

--
Ben.

[prashant.khade1623]

On Apr 10, 3:36*pm, Ben Bacarisse <[e-mail]:

"[e-mail]" <[e-mail]:

main()

int main(void) is much better.

{

int *ptr;
int *p;
p=ptr;
int i;
ptr = malloc (10);

No one has yet said that malloc does not know what you are
allocating. *You can't just say 10, because that allocates 10 bytes.
You need malloc(10 * sizeof *ptr) which means, rather literally,
"allocate space big enough for 10 things of the type ptr points to".

for (i =0;i<10;i++)
ptr[i++] = 2;

for (i =0;i<10;i++)
printf("%d\n",p[i]);

As already pointed out, p is not properly set. *It does not point at
an array whose elements you can print like this.

}

--
Ben.

Thanks...

I got you....

thanks for the help

[Lew Pitcher]

pr****************@gmail.com wrote:
[snip]

I have changed the place where p = ptr. But still I am getting wrong
answer

#include<stdio.h>
main()
{

int *ptr;
int *p;
int i;
ptr = malloc (10);

p=ptr;
for (i =0;i<10;i++)
ptr[i++] = 2;

Please note that you increment i /twice/ each time you loop through this
for() loop

[1] i starts off at 0

[2] i is tested to ensure that it is less than zero (failure exits the loop)

[3] ptr[i] is set to 2 and i is incremented by 1

[4] i is incremented by 1, and the loop is repeated at the exit test ([2])

The effect of all this is that you only set even number index values of
ptr[] to 2, leaving all the odd number index values of ptr[] set to their
original (uninitialized) values

for (i =0;i<10;i++)
printf("%d\n",p[i]);

This loop cycles through /all/ entries in the array, including the ones left
uninitialized by the previous loop

}
~
~
2
0
2
135153
2
0
2
0
2
0

--
Lew Pitcher

Master Codewright & JOAT-in-training | Registered Linux User #112576
http://pitcher.digitalfreehold.ca/ | GPG public key available by request
---------- Slackware - Because I know what I'm doing. ------

[Lew Pitcher]

Lew Pitcher wrote:

pr****************@gmail.com wrote:
[snip]

>I have changed the place where p = ptr. But still I am getting wrong
answer

#include<stdio.h>
main()
{

int *ptr;
int *p;
int i;
ptr = malloc (10);

p=ptr;
for (i =0;i<10;i++)
ptr[i++] = 2;

Please note that you increment i /twice/ each time you loop through this
for() loop

[1] i starts off at 0

[2] i is tested to ensure that it is less than zero (failure exits the
[loop)

**Correction**
[2] i is tested to ensure that it is less than /ten/ (failure exits the
[loop)

>
[3] ptr[i] is set to 2 and i is incremented by 1

[4] i is incremented by 1, and the loop is repeated at the exit test ([2])

The effect of all this is that you only set even number index values of
ptr[] to 2, leaving all the odd number index values of ptr[] set to their
original (uninitialized) values

>for (i =0;i<10;i++)
printf("%d\n",p[i]);

This loop cycles through /all/ entries in the array, including the ones
left uninitialized by the previous loop

>}
~
~
2
0
2
135153
2
0
2
0
2
0

--
Lew Pitcher

Master Codewright & JOAT-in-training | Registered Linux User #112576
http://pitcher.digitalfreehold.ca/ | GPG public key available by request
---------- Slackware - Because I know what I'm doing. ------

[Martin Ambuhl]

pr****************@gmail.com asked about a program with undefined
behavior and was concerned about its (predictably) unpredictable behavior:

I have provided a replacement below. Each of the changes is for a
reason. Please read it and think about why that might be so.

main()
{

int *ptr;
int *p;
p=ptr;
int i;
ptr = malloc (10);

for (i =0;i<10;i++)
ptr[i++] = 2;

for (i =0;i<10;i++)
printf("%d\n",p[i]);
}
the output is something like this

[...]

#include <stdio.h>
#include <stdlib.h>

int main(void)
{

int *ptr;
int *p;
#if 0
p = ptr; /* mha: for pre-C99 compilers (most),
this cannot precede the declaration
of i. Nor does ptr have a defined
value at this point. These errors
are fixed by moving this down a
line. */
#endif
int i;
ptr = malloc(10); /* mha: I have not added the error
check for this line. You should
consider doing so */
p = ptr; /* mha: new home of this line */

for (i = 0; i < 10; i++)
#if 0
ptr[i++] = 2; /* mha: it is possible that you mean
this, but unlikely. You are
incrementing i twice, once in the
line above and in this line.
Consider the replacement below. */
#endif
ptr[i] = 2; /* mha: the replacement */

for (i = 0; i < 10; i++)
printf("%d\n", p[i]);
free(ptr); /* mha: added. Always free() any
memory dynamically allocated. */
return 0;
}

[Kenneth Brody]

"pr****************@gmail.com" wrote:
[...]

I have changed the place where p = ptr. But still I am getting wrong
answer

#include<stdio.h>
main()
{

int *ptr;
int *p;
int i;
ptr = malloc (10);

Here, you allocate 10 chars, not 10 ints. You want:

ptr = malloc(10 * sizeof(*ptr));

p=ptr;
for (i =0;i<10;i++)
ptr[i++] = 2;

How many times is "i++" executed each time through the loop?

for (i =0;i<10;i++)
printf("%d\n",p[i]);

}

[...]

--
+-------------------------+--------------------+-----------------------+
| Kenneth J. Brody | www.hvcomputer.com | #include |
| kenbrody/at\spamcop.net | www.fptech.com | <std_disclaimer.h|
+-------------------------+--------------------+-----------------------+
Don't e-mail me at: <mailto:Th*************@gmail.com>