adfemg
  1. adfemg
  2. ADFEMG-95

[ADFng1-04022] - View object bind variable names

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Labels:
      None

      Description

      hi

      Please consider naming guideline [ADFng1-04022] in the "ADF Naming and Project Layout Guidelines v1.00" [1].

      [ADFng1-04022] - View object bind variable names - The bind variables defined within a view object's query on those associated with a view criteria should be treated as variable instances when it comes to naming guidelines. Namely they should use camelcase with the first letter in lower case.
      In turn bind variable names should be prefixed with the letter "p" for parameter.
      For example pStatus or pDateOfBirth.

      Using a bind variable name with a first letter in lower case results in some weird looking bind variable accessor methods (related to naming guideline [ADFng1-01005]) :

      	public String getpFirstName()
      	{
      		return (String)getNamedWhereClauseParam("pFirstName");
      	}
      
      	public void setpFirstName(String value)
      	{
      		setNamedWhereClauseParam("pFirstName", value);
      	}
      

      While starting with an upper case letter and using a suffix like "BVar" would serve a similar purpose and would result in bind variable accessor methods like :

      	public String getFirstNameBVar()
      	{
      		return (String)getNamedWhereClauseParam("FirstNameBVar");
      	}
      
      	public void setFirstNameBVar(String value)
      	{
      		setNamedWhereClauseParam("FirstNameBVar", value);
      	}
      

      many thanks
      Jan Vervecken

        Issue Links

          Activity

          Hide
          chriscmuir added a comment -

          Point duly noted but not one I'm going to change the document for. Closing issue as a result.

          Show
          chriscmuir added a comment - Point duly noted but not one I'm going to change the document for. Closing issue as a result.

            People

            • Assignee:
              Unassigned
              Reporter:
              Jan Vervecken
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: