Naming for Namespace, Class, and Variables

subaru_sti · #1 (**permalink**) 07-07-2009, 2:56 AM

Hello.

I'm new to OOP and am racking my brain over what to name stuff... I have read a lot of different articles on standards and when to use nouns and different case, etc... I'm stumped when it comes to a real world example. In the code below, the namespace is called "Navigation" which I think is correct. Then I have a class called "Heading", which by definition is the direction a person/vehicle is truly pointing towards. I think that is also named correctly. In the code below, I have four things I have named: _WhatToName1, WhatToName2, WhatToName3, and WhatToName4.

The value that gets passed in and stored is a double between 0 and 360 -- essentially the degree value from a circle. For WhatToName3 and WhatToName4, I have seen a lot of places that just use "value" as the name. Is that standard?

Code:

Namespace Navigation
    Public Class Heading
        Private _WhatToName1 As Double

        Public Sub New(ByVal WhatToName3 As Double)
            Me.WhatToName2 = WhatToName3
        End Sub

        Public Property WhatToName2() As Double
            Get
                Return _WhatToName1
            End Get
            Set(ByVal WhatToName4 As Double)
                _WhatToName1 = WhatToName4
            End Set
        End Property
    End Class
End Namespace

As I am just learning, I want to make sure that I start naming stuff correctly so that it can be easily reused and understood.

Thanks in advance for your help,

Stephen

Robert_Zenz · #2 (**permalink**) 07-07-2009, 4:10 AM

Hello.

First question, are you developing a FLOSS application, Closed-Source Library (for resale) or a Closed-Source Application?

In the last case, I'd stick with naming convention YOU can live with...in the other two cases it is most important that the interface of the code is intuitive. In the first two cases make sure to use XML-Documentation.

Code:

Navigation.Heading.Direction 'leads me to something what points into a direction

Value is used whenever it is logical that the Object has properties and a underlaying value.

Bobby

JohnH · #3 (**permalink**) 07-07-2009, 6:08 AM

WhatToName2: Direction as suggested is a good property name for this.
WhatToName1: _direction would be my choice.
WhatToName3: direction is the guidelines suggestion in this case.
WhatToName4: value is put here by the property snippet, it is also commonly used in the .Net library.

Guidelines for Names

InertiaM · #4 (**permalink**) 07-07-2009, 8:07 AM

Quote:

Originally Posted by JohnH

WhatToName2: Direction as suggested is a good property name for this.
WhatToName1: _direction would be my choice.
WhatToName3: direction is the guidelines suggestion in this case.
WhatToName4: value is put here by the property snippet, it is also commonly used in the .Net library.

I would agree with JohnH on 2, 1 and 4.

For 3, I prefer to defy convention and use "whatDirection" as I feel "Me.Direction = direction" is messy. Purely my personal preference though.

JohnH · #5 (**permalink**) 07-07-2009, 9:23 AM

Quote:

Originally Posted by InertiaM

I would agree with JohnH on 2, 1 and 4.

For 3, I prefer to defy convention and use "whatDirection" as I feel "Me.Direction = direction" is messy. Purely my personal preference though.

"Me.Direction = direction" may look at bit confusing to a developer, but the key here is what the user sees when using this class. The guideline I had in mind was this from Constructor Design :

Quote:

Do use the same name for constructor parameters and a property, if the constructor parameters are used to simply set the property. The only difference between such parameters and the properties should be casing.

So when consumer uses this constructor it should be obvious that the 'direction' parameter is a way to set the 'Direction' property directly when creating the class instance.

Whether is was "Me.Direction = direction" (setting value through property setter) or "Me._direction = direction" (setting value to private field) is just a matter of reuse of existing code if necessary, for example when setter does validation.

subaru_sti · #6 (**permalink**) 07-07-2009, 9:57 PM

Thanks everyone. This is very helpful and I think I am getting the hang of it. I have another one that is exactly the same but is for Bearing ("The direction of one object in relation to another object"). So that one would be Navigation.Bearing.Direction right?

Last question: I have another class, the same as above, which is called "Speed". It is used to hold meters per second. I am using it for this GPS app but I don't think it should go into the Navigation namespace as it could be related to stuff other than navigation, right? For the property name of this class, would I want to name it something like "MetersPerSecond" or is that a no no?

Stephen

jmcilhinney · #7 (**permalink**) 07-07-2009, 11:02 PM

Quote:

Originally Posted by subaru_sti

Thanks everyone. This is very helpful and I think I am getting the hang of it. I have another one that is exactly the same but is for Bearing ("The direction of one object in relation to another object"). So that one would be Navigation.Bearing.Direction right?

Last question: I have another class, the same as above, which is called "Speed". It is used to hold meters per second. I am using it for this GPS app but I don't think it should go into the Navigation namespace as it could be related to stuff other than navigation, right? For the property name of this class, would I want to name it something like "MetersPerSecond" or is that a no no?

Stephen

I can't see why you would need a class for speed. Surely speed is just a number, so you'd just assign a value to a Double variable or the like. The only reason I can see to have a class for speed would be if you wanted to have a numerical value and a unit as well, e.g. kph or m/s.

subaru_sti · #8 (**permalink**) 07-07-2009, 11:14 PM

I want to have a class so that in the SET, I can validate that the value is greater than 0.

Stephen

subaru_sti · #9 (**permalink**) 07-08-2009, 12:14 AM

Okay. I think this makes sense, to me anyways:

Code:

Namespace GPS
    Public Class Speed
        Private _metersPerSecond As Double

        Public Sub New(ByVal metersPerSecond As Double)
            Me.MetersPerSecond = metersPerSecond
        End Sub

        Public Property MetersPerSecond() As Double
            Get
                Return _metersPerSecond
            End Get
            Set(ByVal value As Double)
                If value < 0 Then Throw New ArgumentOutOfRangeException("Speed cannot be less than 0.")

                _metersPerSecond = value
            End Set
        End Property
    End Class
End Namespace

Stephen

jmcilhinney · #10 (**permalink**) 07-08-2009, 1:10 AM

That still doesn't make sense. Consider this. If you had a Car class would it make sense to say myCar.Speed.MetersPerSecond = 10? That doesn't make sense to me. What does make sense is myCar.Speed = 10. The Car class would have a Speed property of type Double and you would validate the value inside that property. Having a whole separate class for speed is not appropriate.

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)