Embedded Documentation Suggestion - Inline param comments

T.J. Crowder tj at crowdersoftware.com
Tue Feb 6 07:20:26 PST 2007


Hi,

Just heard about D and have been reading up.  Wow.  I'm coming from a C, 
Java, and C# background and D looks extremely well thought-out in many ways.

A suggestion on the embedded documentation front:  In addition to the 
current way of documenting params (in a Params section in the block comment 
above the method), allow them to be documented inline.  That way, we don't 
have to repeat the param name, which is in keeping with the guideline of not 
telling the compiler things it already knows and D's excellent emphasis on 
helping people avoid doing stupid things (like changing the param name and 
forgetting to update the documentation block's copy).

E.g., the following:

/**
 * Counts the occurrences of a character in a string.
 * Params:
 *      str = The string
 *      c   = The character
 * Returns:
 *      The number of times the character appears in the string
 */
int countOccurrences(char[] str, char c)
{
    ...
}

could also be written:

/**
 * Counts the occurrences of a character in a string.
 * Returns: The number of times the character appears in the string.
 */
int
countOccurrences(
    char[] str,     /// The string to search.
    char c,         /// The character to count.
    )
{
    ...
}

This is consistent with documenting other declarations.  I like to put the 
params on their own line anyway, and it would conserve a lot of vertical 
space for people like me (for those who normally put all the params on one 
line, there's no significant delta) while reducing duplication.

Just for the sake of completeness:  I don't have a good suggestion for doing 
something similar with the return value.  The logical extension would be 
allowing the return comment inline between the return type declaration and 
the method name:

/**
 * Counts the occurrences of a character in a string.
 */
int                 /// The number of times the character appears in the 
string.
countOccurrences(
    char[] str,     /// The string to search.
    char c,         /// The character to count.
    )
{
    ...
}

...but unlike the param names there's not really a strong reason for doing 
so.

Apologies if this has been suggested before; searching for "param 
documentation" and the like didn't turn it up in the first few pages...

FWIW,
--
T.J. Crowder
tj at crowdersoftware.com 





More information about the Digitalmars-d mailing list