In the code below, I have to first take the string, and remove the newline value by setting input[length-1]=0; Is there a simpler way to do this?
Code:void input(){ int continu=1; char input[256]; int inlength; while (continu==1){ printf("\n\nEnter Command <h=help>: "); fgets(input, 25, stdin); inlength=strlen(input); input[inlength-1]=0; if (strcmp(input, "x")==0){continu=0;} } }
This method is not safe: what if the last character is not a newline? Or even worse, what if no input at all is read?
Better is to find the newline with strchr() and, if it exists, set it to 0. As to your question, I suppose you could write a wrapper around fgets() that always removes a newline if it exists. fgets() itself cannot be told to do this.
Ok, thanks, I changed it to the following. Is there any advantage to using '\0' instead of just plain 0?
-Adam RinkleffCode:char input[BUFSIZ]; char *p; ... printf("\n\nEnter Command <h=help>: "); fgets(input, sizeof(input), stdin); if ((p = strchr(input, '\n')) != NULL) *p = '\0';
The '\0' is not zero but actually an escape character. Wiki NUL character.
Originally Posted by anduril462
Warning: Some or all of my posted code may be non-standard and as such should not be used and in no case looked at.
Readability since it hints that you expect to be assigning to a char instead of some other numeric or pointer type.
By the way, you should improve your indentation, e.g.,
Code:printf("\n\nEnter Command <h=help>: "); fgets(input, sizeof(input), stdin); if ((p = strchr(input, '\n')) != NULL) *p = '\0';That depends on your point of view: since '\0' has a value of zero, and in fact is of type int, one could argue that it is zero.
Look up a C++ Reference and learn How To Ask Questions The Smart Way
I guess I do have a differing point of view here but I do see what you are saying.
Warning: Some or all of my posted code may be non-standard and as such should not be used and in no case looked at.
