in reply to Re^2: Advise on sql table mapping.
in thread Advise on sql table mapping.

The print statements will also try out to print stuff when no path can be found. You shouldn't print out stuff within find_joins, you should print out the results of the top-level call to it. Leaving the warn statement active would have helped you see the flow of control better.

Why did you remove the use strict; line? Adding it again points to a potential error.

Replies are listed 'Best First'.
Re^4: Advise on sql table mapping.
by dbmathis (Scribe) on May 19, 2008 at 20:56 UTC

    I had removed strict and forgot to add it back when I was calling @results from outside the loop.

    Just basically studying your code. Thanks btw. This has really helped me greatly.

    After all this is over, all that will really have mattered is how we treated each other.
[reply]

      I'm not sure what you mean by "calling @results from outside the loop", but if you mean what I think you mean, you might benefit from Coping With Scoping. @results is not visible anymore once find_joins has finished running. Use the return value(s) of find_joins, like I did.

[reply]
[d/l]
[select]

        Actually @results did exist because I had also removed: my @results;

[reply]
[d/l]

In Section Seekers of Perl Wisdom