I used the debugger to examine this code but not understanding a couple areas.

  1. Why does the for loop repeat after it exits to print a new line? If it exits the loop, shouldn't it be done with it?
  2. Why is n incremented and not i as stated with i++?

int main(void)
{
    int height = get_int("Height: ");

    draw(height);
}

void draw(int n)
{
    if (n <= 0)
    {
        return;
    }

    draw(n - 1);

    for (int i = 0; i < n; i++)
    {
        printf("#");
    }
    printf("\n");
}
you are viewing a single comment's thread
view the rest of the comments
[–] 6 points 2 years ago* (2 children)

So you're drawing a triangle of # that starts skinny in the top left and grows to the right with a height and width of n?

As an example, Height 3 would be:

#
##
###

I'm not following your questions, but the code seems pretty reasonable... though the use of recursion feels pretty unnecessary when you could easily just use a nested for loop.

  • source
  • hideshow 4 child comments
  • [–] [S] 2 points 2 years ago (1 child)

    It's supposed to be a pyramid but not my code. It's an example of a recursive function from a CS50 lecture and I'm just trying to understand how the code works line by line.

  • source
  • parent
  • hideshow 2 child comments
  • [–] 2 points 2 years ago (1 child)

    So is my example of Height 3 in line with your expectations?

  • source
  • parent
  • hideshow 2 child comments
  • [–] [S] 1 point 2 years ago (1 child)
  • [–] 2 points 2 years ago (1 child)

    I wrote an equivalent version just using nested loops - reading it might help you understand why the recursion works the way it does.

  • source
  • parent
  • hideshow 2 child comments
  • [–] [S] 1 point 2 years ago (1 child)

    Thanks. I did see that. I have a general understanding of how recursion works I think where the function calls itself again and again but I don't get why the code (for loop) below the draw(n - 1) is recursive.

  • source
  • parent
  • hideshow 2 child comments
  • [–] 5 points 2 years ago

    The code below the draw(n - 1) isn't recursive... the call to draw(n - 1) is the recursion.

    Sometimes, it can be helpful to invert recursion. Think about what draw(0) would be and write it down... then compute draw(1) using the value you previously computed for draw(0).

  • source
  • parent