Databases
StackOverflow2013
Postsdbo
POClass Methods: Passing private data fields in private methods
 

Note that there are some explanatory texts on larger screens.

plurals
  1. POClass Methods: Passing private data fields in private methods
    primarykey
    data
    text
    <p>I was wondering if which of the following patterns are considered to be more "correct".</p> <p>The first example sets the value of private data member length by calling a void member function that implicitly takes arguments. The second example sets the value of length by assigning it to the return value of the member function that explicitly takes an argument.</p> <p>It seems that the second method makes the code clearer, since you know when and how the private member is getting set, where the first requires a trace of the code to verify what and how a value is being assigned. The second method also seems like it would allow for better reuse down the road because it could operate in any class/context (since the arguments and return type are explicit).</p> <p>The first method is quicker, and if used throughout the entire class (for private member functions) can save some coding, however I'm not sure if this will bite me down the road ?</p> <p>Thank you for the insight and clearing this up for me.</p> <pre><code>class MyDataType { private int length; private string content; private char[] buffer; public MyDataType(str) { content = str; calculateLength(); buffer = new char[length+1]; for(int i=0; i &lt; length; i++) buffer[i] = content[i]; buffer[i] = NULL; } private void calculateLength() { int i = 0; while(content[i++] != NULL) {} // i know, buffer overflow... length = i; } } </code></pre> <hr> <pre><code>class MyDataType { private int length; private string content; private char[] buffer; public MyDataType(str) { content = str; length = calculateLength(content); buffer = new char[length+1]; for(int i=0; i &lt; length; i++) buffer[i] = content[i]; buffer[i] = NULL; } private int calculateLength(string s) { int i = 0; while(s[i++] != NULL) {} return i; } } </code></pre>
    singulars
    1. PO
      singulars
      1. PTAnswer
    1. This table or related slice is empty.
    1. PTQuestion
    1. USsamsara
    1. USsamsara
    plurals
    1. This table or related slice is empty.
    1. This table or related slice is empty.
    1. This table or related slice is empty.
    1. PO
      singulars
      1. PTAnswer
    2. PO
      singulars
      1. PTAnswer
    3. PO
      singulars
      1. PTAnswer
    1. VO
      singulars
      1. POClass Methods: Passing private data fields in private methods
      1. This table or related slice is empty.
      1. VTClose
    2. VO
      singulars
      1. POClass Methods: Passing private data fields in private methods
      1. This table or related slice is empty.
      1. VTUpMod
    3. VO
      singulars
      1. POClass Methods: Passing private data fields in private methods
      1. This table or related slice is empty.
      1. VTUpMod
    1. COWhy is the second method not static in the second example, ie doesn't seem to use any of the other variables within the class?
      singulars
      1. POClass Methods: Passing private data fields in private methods
      1. USKingCronus
    2. COThis is just an example I whipped up for the sake of argument. So by asking me if its static (which, yes, I suppose it could be), are you implying that the first example is correct ? Just b/c a method could be static, should it be ?
      singulars
      1. POClass Methods: Passing private data fields in private methods
      1. USsamsara
    3. COIn your first example you have the getLength method setting the private member variable and not returning anything. Typically a getX method is expected to return X. Perhaps it should be renamed.
      singulars
      1. POClass Methods: Passing private data fields in private methods
      1. USTom
 

Querying!

 
Guidance
A row detail

Detail views are divided into sections. All the information in the data section comes from columns in the selected row. The other sections display data from other, related rows.

Related data can be related in a to-one or a to-many fashion. Captions of data related in a to-many fashion link to a list view showing a filtered view of the table.

Try moving around until you find a non-empty to-many entry and click on the label to get to one. You can move back to the root by clicking on the database name in the header.

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload