Thread: recursion help

  1. #1
    Registered User
    Join Date
    Aug 2004

    recursion help

    i am obviously missing a big point on recursion.
    this is an excercise program that prints to the screen one char at a time. the return value is the number of chars entered.
    all works fine and well until i decide to add the printf at the end of the function, which prints the the chars in the previous functions. the question is, why does the return value change once the printf is enabled?
    (written with borland c builder)

    #include <conio.h>
    #include <stdio.h>
    int func(int);
    void main(void)
    int d=func(0);

    func(int d)
    {int x;
    if (x!=' ') func(d+1);
    return d;


  2. #2
    Registered User hk_mp5kpdw's Avatar
    Join Date
    Jan 2002
    Northern Virginia/Washington DC Metropolitan Area
    func(int d)
        int x;
        if (x!=' ') func(d+1);
        else return d;
    You should have gotten a warning from your compiler along the lines of "warning: 'func' not all control paths return a value;" As you can see, if you execute the recursive call, then the whole else part is not going to be executed and you will not be returning a value. You should probably change this line:
    if (x!=' ') return func(d+1);
    Of course then the extra printf isn't going to help you at all since it will never be executed.

    [edit]I tried things both ways (original and extra return like I had) and it didn't seem to affect the results. I did not try out that extra printf however, just seeing if the extra return would change things... it did not. But, it is best to pay some attention to your compilers warnings.[/edit]
    Last edited by hk_mp5kpdw; 08-16-2004 at 05:44 AM.
    "Owners of dogs will have noticed that, if you provide them with food and water and shelter and affection, they will think you are god. Whereas owners of cats are compelled to realize that, if you provide them with food and water and shelter and affection, they draw the conclusion that they are gods."
    -Christopher Hitchens

  3. #3
    Registered User
    Join Date
    Mar 2003
    First off, please use code tags.

    Second, your code only works by fluke. func() _must_ return a value, but your if...else...
    statement does not necessarily return a value. the line d=func(0) actually sets d to the
    last value that was returned by a function - since func(0) does not actually return a value
    without the added printf() function the last value returned was actually the right answer -
    with the added printf() the last value returned came from the printf() which will have
    returned 1. Its usually better only to return at the end of a function, rather than having
    multiple exit points (others will disagree I'm sure - there are situations where mulitple
    returns from a function make the code clearer - but generally speaking...). So here is a
    (in my opinion) better implementation of the function:
    int func (int d)
      int x,r;
      if (x!=' ') {
        r = func(d+1);
      else {
        r = d;
      return r;

Popular pages Recent additions subscribe to a feed

Similar Threads

  1. Template Recursion Pickle
    By SevenThunders in forum C++ Programming
    Replies: 20
    Last Post: 02-05-2009, 09:45 PM
  2. Recursion... why?
    By swgh in forum C++ Programming
    Replies: 4
    Last Post: 06-09-2008, 09:37 AM
  3. a simple recursion question
    By tetra in forum C++ Programming
    Replies: 6
    Last Post: 10-27-2002, 10:56 AM
  4. To Recur(sion) or to Iterate?That is the question
    By jasrajva in forum C Programming
    Replies: 4
    Last Post: 11-07-2001, 09:24 AM
  5. selection sorting using going-down and going-up recursion
    By Unregistered in forum C Programming
    Replies: 1
    Last Post: 11-02-2001, 02:29 PM