Getting an exception..,

[Pranav]

#include "stdafx.h"
#include<conio.h>
#include<stdio.h>
#include<string.h>
int main()
{
char *str1 = "United";
char *str2 = "Front";
char *str3;

str3 = strcpy(str1, str2); //AT THIS COMMAND EXCEPTION OCCURS

printf("%s", str3);

getch();

return 0;
}
Not working under MS VS 6.0 and also in Bloodshed Dev C++.

[Andrew Poelstra]

On 2008-08-12, Pranav <pr*******@gmail.comwrote:

>
char *str1 = "United";
char *str2 = "Front";
char *str3;

str3 = strcpy(str1, str2); //AT THIS COMMAND EXCEPTION OCCURS

str2 points to a string literal, which is immutable.

Your code has a number of other issues, but this is the cause of
your main complaint.
--
Andrew Poelstra ap*******@wpsoftware.com
To email me, use the above email addresss with .com set to .net

[Barry Schwarz]

On Tue, 12 Aug 2008 04:34:22 GMT, Andrew Poelstra <no****@nospam.com>
wrote:

>On 2008-08-12, Pranav <pr*******@gmail.comwrote:

>>
char *str1 = "United";
char *str2 = "Front";
char *str3;

str3 = strcpy(str1, str2); //AT THIS COMMAND EXCEPTION OCCURS

str2 points to a string literal, which is immutable.

Of course you meant str1 since that is the target of the copy.

>
Your code has a number of other issues, but this is the cause of
your main complaint.

--
Remove del for email

[Andrew Poelstra]

On 2008-08-12, Barry Schwarz <sc******@dqel.comwrote:

On Tue, 12 Aug 2008 04:34:22 GMT, Andrew Poelstra <no****@nospam.com>
wrote:

>>On 2008-08-12, Pranav <pr*******@gmail.comwrote:

>>>
char *str1 = "United";
char *str2 = "Front";
char *str3;

str3 = strcpy(str1, str2); //AT THIS COMMAND EXCEPTION OCCURS

str2 points to a string literal, which is immutable.

Of course you meant str1 since that is the target of the copy.

Yes, I did. Thank you.

--
Andrew Poelstra ap*******@wpsoftware.com
To email me, use the above email addresss with .com set to .net

[CBFalconer]

Pranav wrote:

>
#include "stdafx.h"
#include<conio.h>

The above do not normally exist in standard C. That makes this an
off-topic post. Just omit those includes. BTW, use a space
between "include" and "<".

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

int main()
{
char *str1 = "United";
char *str2 = "Front";

These are non-modifiable, i.e. read-only, strings. Use:

char str1[] = "United";

to get an initialized but modifiable string.

char *str3;

str3 = strcpy(str1, str2); //AT THIS COMMAND EXCEPTION OCCURS

and you are trying to modify str1. Not allowed.

>
printf("%s", str3);
getch();
return 0;
}

Not working under MS VS 6.0 and also in Bloodshed Dev C++.

Not of interest on this newsgroup, but explains where you got the
bad habits.

--
[mail]: Chuck F (cbfalconer at maineline dot net)
[page]: <http://cbfalconer.home.att.net>
Try the download section.

[Martin Ambuhl]

Pranav wrote:

#include "stdafx.h"

^^^^^^^^^^
This is not a standard header.

#include<conio.h>

^^^^^^^^^
This is not a standard header. And a little white space won't kill you.
Hitting the spacebar once will not strain your thumb.

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

int main()
{
char *str1 = "United";
char *str2 = "Front";
char *str3;

str3 = strcpy(str1, str2); //AT THIS COMMAND EXCEPTION OCCURS

^^^^^^^^^^^^
This is not a command. It is a function call with
its returned value assigned to a variable.
^^^^^^^^^^^^^^^^^^^^
str1 points to a string constant, not to a modifiable
array. If you get a fatal error, that's wonderful. You
have been lucky. And if your implementation calls that an
exception, that's OK, but the word "exception" has no meaning
for C.

printf("%s", str3);
getch();

^^^^^^
This is not a standard function. There are times when non-standard
functionality is necessary, but this isn't one of them: getchar()
would do as well (but a message to the user first is a good idea.
Forcing user to guess what the hell is going on is bad practice).

>
return 0;
}

Not working under MS VS 6.0 and also in Bloodshed Dev C++.

Of course not.

[Pranav]

On Aug 12, 11:37 am, Martin Ambuhl <[e-mail]:

Pranav wrote:

#include "stdafx.h"

^^^^^^^^^^
This is not a standard header.

#include<conio.h>

^^^^^^^^^
This is not a standard header. And a little white space won't kill you.
Hitting the spacebar once will not strain your thumb.

#include<stdio.h>
#include<string.h>
int main()
{
char *str1 = "United";
char *str2 = "Front";
char *str3;
str3 = strcpy(str1, str2); //AT THIS COMMAND EXCEPTION OCCURS

^^^^^^^^^^^^
This is not a command. It is a function call with
its returned value assigned to a variable.
^^^^^^^^^^^^^^^^^^^^
str1 points to a string constant, not to a modifiable
array. If you get a fatal error, that's wonderful. You
have been lucky. And if your implementation calls that an
exception, that's OK, but the word "exception" has no meaning
for C. printf("%s", str3);

getch();

^^^^^^
This is not a standard function. There are times when non-standard
functionality is necessary, but this isn't one of them: getchar()
would do as well (but a message to the user first is a good idea.
Forcing user to guess what the hell is going on is bad practice).

return 0;
}
Not working under MS VS 6.0 and also in Bloodshed Dev C++.

Of course not.

Thank You All For Your Suggestions and Help. I will take care next
time.

[s0suk3]

On Aug 12, 1:37*am, Martin Ambuhl <[e-mail]:

Pranav wrote:

#include "stdafx.h"

* * * * * * ^^^^^^^^^^
This is not a standard header.

#include<conio.h>

* * * * * *^^^^^^^^^
This is not a standard header. And a little white space won't kill you.
* Hitting the spacebar once will not strain your thumb.

Likewise, letting other people use their own styles won't kill you,
either.

#include<stdio.h>
#include<string.h>
int main()
{
* char *str1 = "United";
* char *str2 = "Front";
* char *str3;
* str3 = strcpy(str1, str2); *//AT THIS COMMAND EXCEPTION OCCURS

* * * * * * * * * * * * * * * * * * * ^^^^^^^^^^^^
* * * * * * * * * * * *This is not a command. It is a function call with
* * * * * * * * * * * *its returned value assigned to a variable.

Actually, it's a statement. Or more specifically, an expression
statement.

* * * * * *^^^^^^^^^^^^^^^^^^^^
* * * * * *str1 points to a string constant, not to a modifiable
* * * * * *array.

It's not a string constant, it's a string literal.

If you get a fatal error, that's wonderful.

You got a weird notion of what's "wonderful."

*You have been lucky.

He got a fatal error... how is that "lucky"?

>* And if your implementation calls that an
* * * * * *exception, that's OK,

It's OK if the compiler gave a warning that contained the word
"exception" (as in the general meaning of "exception"). But if it
"threw" an exception (as in a "program exception"), even at compile
time, then that's not OK.

but the word "exception" has no meaning
* * * * * *for C.

* printf("%s", str3);
* getch();

* * *^^^^^^
* * *This is not a standard function. *There are times when non-standard
* * *functionality is necessary, but this isn't one of them: getchar()
* * *would do as well

No. getchar() and getch() don't do the same thing.

(but a message to the user first is a good idea.
* * *Forcing user to guess what the hell is going on is bad practice).

return 0;
}
Not working under MS VS 6.0 and also in Bloodshed Dev C++.

Of course not.

Sebastian

[Default User]

s0****@gmail.com wrote:

On Aug 12, 1:37*am, Martin Ambuhl <[e-mail]:

If you get a fatal error, that's wonderful.

You got a weird notion of what's "wonderful."

It immediately diagnosed the undefined behavior and sent him here for
help. That's wonderful.

*You have been lucky.

He got a fatal error... how is that "lucky"?

Do you think it would have been better to seem to work for a while?
Then later crash after adding a bunch more code? Immediate and
unequivocal responses by the OS to your programming blunders are very
lucky indeed.


Brian

[jameskuyper]

[e-mail] wrote:

On Aug 12, 1:37�am, Martin Ambuhl <[e-mail]:

Pranav wrote:

...

� str3 = strcpy(str1, str2); �//AT THIS COMMAND EXCEPTION OCCURS

� � � � � � � � � � � � � � � � � � � ^^^^^^^^^^^^
� � � � � � � � � � � �This is not a command. It is a function call with
� � � � � � � � � � � �its returned value assigned to a variable.

Actually, it's a statement. Or more specifically, an expression
statement.

You're using "this" to refer to the entire line of code, which is an
expression statement. Martin was using "this" to refer to the function
call expression contained in that expression statement, so his
statement was just as accurate as yours. Since the "exception" almost
certainly occurred while the function call was executing, that
function call was probably also what Pranav was incorrectly
describing as a a command when he used the word "this".

...

If you get a fatal error, that's wonderful.

You got a weird notion of what's "wonderful."

�You have been lucky.

He got a fatal error... how is that "lucky"?

Fatal errors are much better than undetected bugs. They gave him an
opportunity to detect and correct his error before it caused more
serious damage.

� And if your implementation calls that an
� � � � � �exception, that's OK,

It's OK if the compiler gave a warning that contained the word
"exception" (as in the general meaning of "exception"). But if it
"threw" an exception (as in a "program exception"), even at compile
time, then that's not OK.

The behavior of this code is undefined; as far at the standard is
concerned, that would be perfectly acceptable. Anything else is just a
QoI issue.

� getch();

� � �^^^^^^
� � �This is not a standard function. �There are times when non-standard
� � �functionality is necessary, but this isn'tone of them: getchar()
� � �would do as well

No. getchar() and getch() don't do the same thing.

That's what he was referring to when he wrote about "non-standard
functionality". There are things you can do using getch(), that you
can't do using standard C code, but none of those things is necessary
to this function.

[jameskuyper]

[e-mail] wrote:

[e-mail] wrote:

On Aug 12, 1:37�am, Martin Ambuhl <[e-mail]:

Pranav wrote:

...

� str3 = strcpy(str1, str2); �//AT THIS COMMAND EXCEPTION OCCURS
>
� � � � � � �� � � � � � � � � � � � ^^^^^^^^^^^^
� � � � � � �� � � � �This is not a command. Itis a function call with
� � � � � � �� � � � �its returned value assigned to a variable.

Actually, it's a statement. Or more specifically, an expression
statement.

You're using "this" to refer to the entire line of code, which is an

Sorry, that should be "it", not "this".

expression statement. Martin was using "this" to refer to the function
call expression contained in that expression statement, so his
statement was just as accurate as yours. Since the "exception" almost
certainly occurred while the function call was executing, that
function call was probably also what Pranav was incorrectly
describing as a a command when he used the word "this".