To a certain degree, I share your confusion - ordinarily you wouldn't expect the result of a database query to be stored within an object, however this may be the author's attempt at memoization whereby the results of a previous query are stored for efficiency reasons.

That being said, the code snippet gives no clues as to the rationale behind the existence of the method e.g. why is the result of a get_sites() invocation not implemented within the constructor since it always returns a list of all sites i.e. takes no arg(s) that might constrain the result set in some way - unless it's available purely and simply to refresh the result set.

Of course, as has been said elsewhere, the memoization directly raises it's own problems e.g. caching, and so is not a really sensible way of working; The method should, IMO, merely return the result set from the query, letting the caller determine to what use it should be put - by doing so it would provide the necessary degree of abstraction between the structure of the query on the backend DB and the data returned from the query - a la the GoF Facade Pattern.

A user level that continues to overstate my experience :-))

In reply to Re: OOP Confusion by Bloodnok
in thread OOP Confusion by packetstormer

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":


  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.