Something is wrong with my box program :C

Hello everyone!

So my teacher asked us to make a box in a particular size based on the users input.

Specifically he only wanted the top left and bottom of a box.

But I happen to be really bad with while loops and stuff, but here is my program so far.

Code:

---

/*This programe creates a box based on input from user*/

#include <stdio.h>

int
main(void)
{
int num;
int top;
int newtop;
int oside;
int obot;
int bot;
int side;

        printf("Enter the size of a box: "); /*takes input from user*/
        scanf("%d", &num);
top = 0;
side = 0;
bot = 0;

while (newtop != num) {
newtop = top + 1 ;
printf("*");

}
while (side!= top) {
oside = side + 1;
printf("*\n");

}
while(bot!= top) {
obot = bot + 1;
printf("*");

}

return 0;
}

---

When I excute the program I see nothing but stars, however it compiles with no error.

Any help would be appreciated and thank you in advance !

Your program is using newtop before it's been given a value (thus the stars all over). Global variables are assigned to 0 or '\0', but local variables are not, so you need to assign them to something, before you use them as a comparison in your program.

Thank you for responding!

Here is my program with newside, otop, and obot being assigned something before the while statements.

Code:

---

/*This programe creates a box based on input from user*/

#include <stdio.h>

int
main(void)
{
int num;
int top;
int newtop;
int oside;
int obot;
int bot;
int side;

        printf("Enter the size of a box: "); /*takes input from user*/
        scanf("%d", &num);
top = 0;
side = 0;
bot = 0;
newtop = 0;
oside = 0;
obot = 0;

while (newtop != num) {
newtop = top + 1 ;
printf("*");

}
while (side!= top) {
oside = side + 1;
printf("*\n");

}
while(bot!= top) {
obot = bot + 1;
printf("*");

}

return 0;
}

---

However I still receive the same error of endless astricks :C

Any more insight you can spare?

You need to increment (or decrement) some variable that you are testing, as it loops

Code:

---

while (newtop != num) {
newtop = top + 1 ;
printf("*");
 
}

---

The test is newtop equaling num. Top + 1 is a constant number, since top isn't being incremented (and shouldn't be imo).

Just use newtop++; and you're OK. Keep it simple, always.

Your second while loop has the same type of problem. Fix it also.

Thank you, I tried the ++ before but my noobish self did it wrong. Anyways, it only seems to be executing the first while statement
ex.) linux3[6]% ./a.out
Enter the size of a box: 10
**********
thank you by the way for responding!

Did you fix the second while loop problem I mentioned yet?

Yes i got it to work. thank you very much for your assistance!

I hate to bother once again :C

But if I wanted to a diagnol how would I be able too O:?

Thanks if you do answer this you've already helped so much

Inside the box, or outside it?

Use code tags and show me a bit of what you want.

And no, you don't hate to bother me, that badly! < rofl! >

Currently my program prints something like this

* * *
*
* * *

With a diagnol it would look something like this
* * * *
* *
* * [err for some reason it wont let me space this one out. Activate imagine! haha]
* * * *

Haha your right I don't hate it you probably do =P

You didn't use code tags around your diagram, so I can't tell much from it. The forum software ruins diagrams if you don't use code tags.

So edit your post, and click on the "Go Advanced" button in the bottom right hand corner, and highlight your diagram. Now click on the # icon just right of top dead center, in the header icon section of the advanced editor page.

Then I can see WTF you're diagram is all about.

I believe this is what you want to do, but not really sure:

Code:

---

#include <stdio.h>

#define ROW 20
#define COL 20

int main(void){
  int r,c,diag=ROW-1;

  for(r=0;r<ROW;r++,diag--) {
      for(c=0;c<COL;c++) {
        if(r==0 || r==ROW-1) {
            putchar('*');
        }
        else if(c==0 || c==COL-1 || c==diag)
            putchar('*');
        else
            putchar(' ');   
      }
      putchar('\n');
  }

  putchar('\n');
  return 0;
}

---