VB.Net中ReadOnly属性/函数的最佳实践是什么？

Question

我不太新到VB.Net和语法，但我不是专家。我正在重写以前用C＃编写的VB中的东西，而当我这样做时，我遇到了一个“困境”。我可以有一个只读属性：

Public ReadOnly Property MaximumIndenture() 
    Get 
     Try 
      'If the connection is closed, open it.' 
      If _dbConnection.State = ConnectionState.Closed Then 
       _dbConnection.Open() 
      End If 

      Dim dictFigureIndenture As Dictionary(Of String, Integer) = New Dictionary(Of String, Integer) 
      Using dbReader As IDataReader = ExecuteReader("SELECT PART_FIGURE, MAX(PART_INDENTURE) AS 'MAX_INDENT' FROM PARTS GROUP BY PART_FIGURE") 
       While dbReader.Read() 
        dictFigureIndenture.Add(dbReader("PART_FIGURE").ToString(), Convert.ToInt32(dbReader("MAX_INDENT").ToString())) 
       End While 
      End Using 

      'If the connection is open, do what you need to and/or close it.' 
      If _dbConnection.State = ConnectionState.Open Then 
       _dbConnection.Close() 
      End If 

      Return dictFigureIndenture 
     Catch ex As Exception 
      'An exception was thrown. Show it to the user so they can report it.' 
      MessageBox.Show(ex.Message) 

      'If the connection is open, do what you need to and/or close it.' 
      If _dbConnection.State = ConnectionState.Open Then 
       _dbConnection.Close() 
      End If 

      Return Nothing 
     End Try 
    End Get 
End Property 

我能有做同样的事情的函数：

Public Function MaximumIndenture() As Integer 
    Try 
     'If the connection is closed, open it.' 
     If _dbConnection.State = ConnectionState.Closed Then 
      _dbConnection.Open() 
     End If 

     Dim dictFigureIndenture As Dictionary(Of String, Integer) = New Dictionary(Of String, Integer) 
     Using dbReader As IDataReader = ExecuteReader("SELECT PART_FIGURE, MAX(PART_INDENTURE) AS 'MAX_INDENT' FROM PARTS GROUP BY PART_FIGURE") 
      While dbReader.Read() 
       dictFigureIndenture.Add(dbReader("PART_FIGURE").ToString(), Convert.ToInt32(dbReader("MAX_INDENT").ToString())) 
      End While 
     End Using 

     'If the connection is open, do what you need to and/or close it.' 
     If _dbConnection.State = ConnectionState.Open Then 
      _dbConnection.Close() 
     End If 

     Return dictFigureIndenture 
    Catch ex As Exception 
     'An exception was thrown. Show it to the user so they can report it.' 
     MessageBox.Show(ex.Message) 

     'If the connection is open, do what you need to and/or close it.' 
     If _dbConnection.State = ConnectionState.Open Then 
      _dbConnection.Close() 
     End If 

     Return Nothing 
    End Try 
End Function 

他们基本上都做同样的事情，但我不知道这将是在社区更受欢迎。最后，我想写很多开源代码供大家使用（大部分可能已经写过），但我绝对不希望有人认为我所做的是最佳做法。任何人都可以给我一个外行的描述，哪个更好用，为什么？对不起，如果这是一个重复的帖子给别人的，只是好奇。谢谢。

Answer 1

这绝对应该是一种方法，而不是一个属性。

属性通常包含轻量级操作，例如获取私有变量的值并可能执行一些简单的计算或转换。从数据库中提取数据的东西肯定不符合通常预期的属性。

Answer 2

我不能答应你，这是最好的做法，但我会说你觉得你更自然，在现实世界中更有意义。我会说MaximumIndenture会比一个函数做出更好的属性，因为在我看来，它更多的是对象属性或属于它的名词。我会使用一个函数来做更多的操作或动词，从而得到一些结果。那些是我的2美分。希望能帮助到你！

Answer 3

使用的方法，而不是一个性质， 如果操作比字段访问 会慢 数量级。特别是访问网络的操作 或文件系统应该可能是方法。

您的代码访问数据库：所以它应该是一个方法。

从。 NET Framework Design Guidelines第二版第135页