Question

我创建了一个名为ProfileHelper的类，我似乎无法使我的get / set访问器正确;相反，我得到和设置红线。这是我试图使用的代码：

public static String UserName(string columnName)
{ 
    get
    {
        using (SqlConnection cn = new SqlConnection(SiteConfig.ConnectionString))
        {
            string sSql = ("SELECT UserName , LoweredUserName FROM  aspnet_Users  WHERE UserId = @UserId");

            using (SqlCommand cm = new SqlCommand(sSql, cn))
            {
                cm.Parameters.AddWithValue("@UserId", Membership.GetUser().ProviderUserKey.ToString());
                cn.Open();
                using (SqlDataReader rd = cm.ExecuteReader())
                {
                    while (rd.Read())
                    {
                        return columnName;
                    }
                    rd.Close();
                }
                cn.Close();
            }         
        }            
        return columnName;
    }
    set
    {
        using (SqlConnection cn = new SqlConnection(SiteConfig.ConnectionString))
        {
            string sSql = ("UPDATE [aspnet_ Users] SET UserName = @UserName, LoweredUserName = @LoweredUserName WHERE UserId = @UserId");

            using (SqlCommand cm = new SqlCommand(sSql, cn))
            {
                cm.Parameters.AddWithValue("@UserId", Membership.GetUser ().ProviderUserKey.ToString());
                cn.Open();
                cm.ExecuteNonQuery();
                cn.Close();
            }    
        } 
    }
}

Answer 1

这是一种方法，而不是财产。只有属性具有get和set访问器。选一个。

public static String UserName(string columnName) // this implies a method
     { 
      get // this implies a property
      {

Answer 2

语法基本上是：

private string userName;
public string UserName // no parameter here!
{
  get { return this.userName; }
  set { this.userName = value; } // value is a keyword
}

或者，auto-property为您执行支持字段：

public string UserName { get; set; } // equivalent
public string MyExternallyReadOnly { get; private set; } // setter is private

你应该不在属性中使用SQL连接。对财产的访问应该快速可靠。考虑用方法替换它，以明确这实际上是一个具有外部依赖性的更长期的行为（更有可能失败）：

public string GetUserName() { }
public void UpdateUserName() {}

Answer 3

您应该制作一对方法，GetUserName(string columnName)和SetUserName(string columnName, string value)。这是可能需要一段时间的首选方法，例如因为您正在建立数据库连接，并且可以轻松获得参数。

public static String UserName(string columnName)是方法的签名。 public static String UserName将成为财产。属性有get / set访问器，方法没有。你需要改变它。属性不允许参数，indexed properties除外，它不能是静态的（例如像myObj[someColumnName]一样使用，声明为public string this[string columnName]）。如果您希望访问var something = ProfileHelper.UserName;这样的属性，只需使用属性。

获取/设置访问者正确的范围

3 个答案: