31
submitted 11 months ago by milon@lemm.ee to c/programming@programming.dev

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");
}
all 30 comments
sorted by: hot top controversial new old
[-] dneaves@lemmy.world 13 points 11 months ago

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?

There's the new line after the for loop to make sure that the next recursion starts on a fresh line. Otherwise the next recursion would print on the same line, right where it left off, and you'd just have a line of "#"'s. The for loop itself is just for printing "#"'s.

Why is n incremented and not i as stated with i++?

I think this is a confusion with the recursion. Look at the line with draw(n - 1); this happens before any printing of hashes happens, and only continues after its done. And it calls itself as long as it's not less than or equal to 0. To psuedo-code the order of operations here:

draw 3 {
    draw 2 {
        draw 1 {
            draw 0 {};
            print "#" * 1;
        };
        print "#" * 2;
    };
    print "#" *3;
};

so n is never incremented as you think, it just calls decremented versions of the draw function before the current one finishes. The i's are purely involved in the for loop, which only prints hashes. Does that make sense?

[-] milon@lemm.ee 2 points 11 months ago

Yes - I finally caught that part about n as it's just moving in reverse so it gets decremented. Now I'm not sure about i. In the debugger when the program gets to the for loop both n and i are equal to 1. The n I understand but i?

[-] dneaves@lemmy.world 3 points 11 months ago

i starts at 0, checks if i is less than n (the first time it will be, no matter what), prints a "#", then increments i by 1 and loops

[-] mac@programming.dev 8 points 11 months ago* (last edited 11 months ago)

This code has a recursive call (function calls itself) within the function so that has to be taken into account when tracing it

This would make the function execute multiple times so the for loop would end up executing multiple times.

Lets say main calls draw with a height value of 10 (draw(10)). First it sees that n is greater than 0 so it keeps going. Then it calls the draw function with a value of 10 - 1 aka 9. Now its executing in the draw(9) function. Greater than 0 so continues and calls draw(8). etc. all the way down to draw(0) where it sees that n is equal to 0 so returns out of the function due to the return statement.

Now that draw(0) finished executing draw(1) can keep going and goes to the for loop. Here it prints 1 # and then prints a new line (and then returns since it hit the end of the function). Now that draw(1) is done draw(2) can keep going and prints 2 #'s and then prints a new line (and then returns). This keeps going all the way up to the initial draw call, draw(10) which prints 10 #'s and then a new line, returns, and then the main function keeps going but theres nothing after that so it returns and the execution ends.

The effect from coming back after the recursive calls makes it seem like n is increasing but its just different calls to the same function. i is taken into account for but printing the amount of #'s since thats whats within that loop

[-] milon@lemm.ee 2 points 11 months ago

Why does the for loop return when it hits the end of the function? Isn't the recursive portion already completed in draw(n - 1)? The rest of it is just normal non-recursive code if I understand it correctly.

[-] jrbaconcheese@yall.theatl.social 14 points 11 months ago* (last edited 11 months ago)

No, the moment that draw(9) is called, draw(10) goes on pause while draw(9) runs, which pauses when it calls draw(8) … which repeats (or recurses) until draw(0) gets called. Then it returns which returns to draw(1). The draw(1) un-pauses and does the #\n bit and returns to draw(2), which un-pauses and does ##\n and so forth until draw(10) does ##########\n

[-] milon@lemm.ee 1 points 11 months ago

Gotcha. Thanks for the explanation.

[-] mac@programming.dev 5 points 11 months ago* (last edited 11 months ago)

When the draw function calls itself it yields control to that new function its calling. When that function ends it takes back control and continues doing what it was doing.

This means all of the for loops in all of the functions will execute. Draw(1) will do the for loop and then return as it hits the end of the function (standard behaviour when you reach the end of a function even if theres no return statement). Then draw(2) will do the for loop as it gets back control now that draw(1) is done and then return, etc. all the way up

All parts of a function are recursive, theres no such thing as a non recursive part

[-] Excrubulent@slrpnk.net 3 points 11 months ago* (last edited 11 months ago)

The for loop doesn't return, it's just that when control flows to the end of the function, returning is the standard behaviour. It has nothing left to do, so control is returned to the calling code.

The recursive portion is begun with draw(n - 1), but then you have a new for loop, because the same function has been called again. That's what recursion is. Nothing in this function is non-recursive. It calls itself, so all of the code it contains could be happening from the initial call, or within a recursive call. When draw(3) is called, you will get 3 for loops. You will actually get 4 draw calls, but the last one will be draw(0) which returns immediately.

It's confusing slightly because it works in reverse to what you'd expect. The operational part of the code - the part that does the drawing - only completes after the recursion is finished. That's why it does draw(1) first, to make the pyramid right way up.

I don't know that I've ever done recursion like this. It seems deliberately fancy and somewhat confusing for a new learner.

Imagine if you put the for loop before the recursive call. What would happen? You would draw three first, then decrease, so you would have an inverted pyramid. That would be easier to understand, but it would also not make a pyramid.

[-] supernicepojo@lemmy.world 2 points 11 months ago

This is all a really great example of how The Stack works. As the loop recurses it continually adds to the program stack in memory and then plays the next “item” in the stack. There is a specific limit to recursion as well based on this principle.

[-] xmunk@sh.itjust.works 6 points 11 months ago* (last edited 11 months ago)

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.

[-] xmunk@sh.itjust.works 7 points 11 months ago

An example using a nested loop

$n = 3;

for ($i = 0; $i <= $n; $i++) {
    for ($j = 0; $j < $i; $j++) {
        echo '#';
    }
    echo "\n";
}

https://3v4l.org/e0IJm

[-] milon@lemm.ee 2 points 11 months ago

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.

[-] xmunk@sh.itjust.works 2 points 11 months ago

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

[-] milon@lemm.ee 1 points 11 months ago
[-] xmunk@sh.itjust.works 2 points 11 months ago

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

[-] milon@lemm.ee 0 points 11 months ago

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.

[-] xmunk@sh.itjust.works 4 points 11 months 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).

[-] anton@lemmy.blahaj.zone 5 points 11 months ago

Recursion is often unintuitive for beginners, to understand this code we should simply it a bit


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

    draw(height);
}

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

    draw(n - 1);
    printf("%d",  n);
    
    printf("\n");
}

Inputting 3 should now give us a output like

1
2
3 

Try to understand that case first and than muddle it up with the loop.

If you examine it in a debugger look at the Stack trace. If you use gdb its bt in a visual debugger it's probably the list of function names next to the variables

[-] mvirts@lemmy.world 2 points 11 months ago

Try moving draw(n-1) after the for loop and see how that changes the output.

[-] Merwyn@sh.itjust.works 2 points 11 months ago* (last edited 11 months ago)

You are looking at a recursive method, as you can see with the line draw(n-1) inside the draw(n) method. You can search for "recursive function" on internet for a better understanding.

Basically, the method draw is called a first time n = a user input, but then this method call itself with n-1 until it reach 0. So you can think as if function draw(6) will call draw(5) and wait for it to return before continuing, draw(5) call draw(4), ect until draw(0) that return immediately.

So then the order of execution will be draw(1) that print " #\n" and return, then draw(2) will proceed to print "##\n" and return, then draw(3), ect until draw(n).

[-] milon@lemm.ee 1 points 11 months ago

Right. I was aware it was recursion as stated in the title of my post. I had two questions specific to where the for loop returns after printing #.

[-] Merwyn@sh.itjust.works 1 points 11 months ago* (last edited 11 months ago)

Yes, as I wrote when the method draw(n=1) finish the for loop that print one "#", this call of the method draw return. Then the process start again from the after the line draw(n-1) of the method draw(n=2), which execute the for loop to print "##" and return. Then again you come back to after the line draw(n-1) of inside the method draw(n=3), ect.

You should keep in mind that everytime a draw(n-1) is called, the current method is "paused" until this call return.

[-] milon@lemm.ee 1 points 11 months ago

I see. I guess my understanding was that the recursion was over after the recursive call, but it's actually for all the code in draw().

[-] Merwyn@sh.itjust.works 2 points 11 months ago* (last edited 11 months ago)

Yes, to better understand this you have to understand the "flow" of the program. Meaning the order at which the instructions are executed and not written.

Here you have the flow of the program starting from n =3 until the recursion reach draw(0), note that none of the for loop have been executed yet. At this point it reach the first "return" instruction and go finish the call to draw(0).

Then the flow go back to where it previously was: inside the draw(1) call just after the line calling draw(0). And it start executing the next lines of the draw(1): the for loop.

Then it reach the second "return" and proceed again until the whole program is over.

[-] milon@lemm.ee 1 points 11 months ago

Yes, that helps. Thanks. I see now how n goes from 1 to 2 to 3...etc. Now not so sure how i = 1 when the for loop starts.

[-] abhibeckert@lemmy.world 1 points 11 months ago* (last edited 11 months ago)

Recursion is a little easier to understand if you use goto instead of functions. Functions are a high level concept in the C language (and most other languages) but it gets compiled down to (essentially) the older goto style of programming.

Most modern languages don't even have goto support, since functions are generally more reliable, however as a programmer you should be aware what's going on under the hood. Here's your code rewritten to use goto (I also generally rewrote the whole thing to be a bit easier to grok):

int main(void)
{
    int height = get_int("Height: ");
    int row = 1;
    int col = 0;

draw:
    if (row > height)
    {
        goto end;
    }

    if (col < row)
    {
        printf("#");
        col++;
        goto draw;
    }
    
    // Move to the next row
    printf("\n");
    row++;
    col = 0;
    goto draw;

end:
    return 0;
}

[-] beejjorgensen@lemmy.sdf.org 1 points 11 months ago

Another approach to thinking about it is that draw() does two things. 1) it draws the line that's 1 shorter than itself, then 2) it draws itself.

The for loop happens after it draws the line that's 1 shorter than itself.

[-] abhibeckert@lemmy.world -1 points 11 months ago* (last edited 11 months ago)

Recursion is much easier to understand if you use goto instead of functions. Functions are a high level concept in the C language (and most other languages) but it gets compiled down to (essentially) the older goto style of programming which is much easier to understand.

goto 42 will move execution to line 42 of the code. goto x will move to the line of code labeled x.

Most modern languages don't even have goto support, since functions are cleaner and tend to produce more easily maintained code, however as a programmer you should be aware what's going on under the hood. Here's your code rewritten to use goto:

int main(void)
{
    int height = get_int("Height: ");
    int row = 1; // Starting row
    int col = 0; // Starting column

draw:
    if (row > height) // Exit condition
    {
        goto end;
    }

    if (col < row)
    {
        printf("#");
        col++;
        goto draw; // Repeat the same row
    }
    
    // Move to the next row
    printf("\n");
    row++;
    col = 0; // Reset column for the next row
    goto draw;

end:
    return 0;
}

PS: goto is also how for loops, if statements, switch blocks and others work under the hood.

this post was submitted on 16 Dec 2023
31 points (97.0% liked)

Programming

17314 readers
24 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities !webdev@programming.dev



founded 1 year ago
MODERATORS