470,849 Members | 1,358 Online
Bytes | Developer Community
New Post

Home Posts Topics Members FAQ

Post your question to a community of 470,849 developers. It's quick & easy.

exercise 1-20 K&R

write a program that replaces strings of blanks by the minimum number
of blanks to space to the next tab stop.

#include <stdio.h>
#define TABSIZE 8
int main()
{
int counter, c;

while ((c = getchar()) != EOF) {
if (c == '\t') {
for (counter = 1; counter <= TABSIZE; ++counter)
putchar(' ');
}
else
putchar(c);
}
return 0;
}

what do you think about this??

Jun 10 '06 #1
2 2038
Aenima1891 wrote:
write a program that replaces strings of blanks by the minimum number
of blanks to space to the next tab stop.

#include <stdio.h>
#define TABSIZE 8
int main()
int main (void) is preferred.
{
int counter, c;

while ((c = getchar()) != EOF) {
if (c == '\t') {
for (counter = 1; counter <= TABSIZE; ++counter)
putchar(' ');
This doesn't accomplish the goal of the exercise which is to replace
the tab with the number of spaces needed for the *next* tab stop, that
is, the next multiple of TABSIZE characters. Your version always
prints TABSIZE spaces which will only be the right thing to do when the
tab character is encountered immemdiately after a tab stop.
}
else
putchar(c);
}
return 0;
}

what do you think about this??


Robert Gamble

Jun 10 '06 #2
Aenima1891 schrieb:
write a program that replaces strings of blanks by the minimum number
of blanks to space to the next tab stop.

#include <stdio.h>
#define TABSIZE 8
int main()
int main (void)
{
int counter, c;
Note: It often helps understanding to separate variable declarations
by "purpose" and not only by type. counter is used just as that,
whereas c is intended to hold a character[*].
This also makes for easier maintenance if you find out that the
_guaranteed_ range of int is not sufficient for your purposes and
you have to switch to long, unsigned long or size_t as counter type.
int counter, anotherCounter;
int c, anotherTempForCharacters;
int resultOfSomeNoncounterCalculation;

Some company coding standards even forbid declaring more than one
variable per line if the declared variables are not loop variables.

while ((c = getchar()) != EOF) {
if (c == '\t') {
for (counter = 1; counter <= TABSIZE; ++counter)
putchar(' ');
You misunderstood the purpose of the task.
Every _line_ is to be formatted independently of the other. For every
line, you start anew with tab stops. This means that you need a
character count for the current line, not an overall character count.

Tab stops are at the same character number (modulo TABSIZE) throughout
the line, i.e. you insert space characters until your counter modulo
TABSIZE equals that number.

Example: For "#define TABSIZE 4",
"\ta\tab\tabc\tabcd\tabcde\t\n" has to become either
" a ab abc abcd abcde \n"
^ ^ ^ ^ ^ ^ ^ ^ ^
where the second line indicates the tab stop positions for clarity.

Or, if you admit the possibility of inserting 0 spaces to fit the
description, your result would be
"a ab abc abcdabcde \n"
^ ^ ^ ^ ^ ^ ^
This is merely the difference between a while() and a do--while()
loop (figure out which one for which case yourself) for you.
}
else
putchar(c);
}
return 0;
}

what do you think about this??

[*] To be more precise: A character cast to unsigned char or EOF.
This is what you typically pass to putchar(), [f]putc() and the
is....() functions and get from getchar(), etc.
Cheers
Michael
--
E-Mail: Mine is an /at/ gmx /dot/ de address.
Jun 10 '06 #3

This discussion thread is closed

Replies have been disabled for this discussion.

Similar topics

1 post views Thread by S Green | last post: by
15 posts views Thread by Xah Lee | last post: by
5 posts views Thread by Charles | last post: by
10 posts views Thread by Simon Mansfield | last post: by
6 posts views Thread by sathyashrayan | last post: by
5 posts views Thread by Richard Gromstein | last post: by
26 posts views Thread by arnuld | last post: by
reply views Thread by liukaiyuan | last post: by
By using this site, you agree to our Privacy Policy and Terms of Use.