这是一个正确的类实现和初始化吗?

本文关键字：实现初始化一个 | 更新日期: 2023-09-27 18:35:20

我发现我的编码实践最近在下滑 - 并注意到自己陷入了一些坏习惯 - 通常是由于我缺乏动力（可能是原因，但为我设定的任务的愚蠢性质） - 所以为了给自己一个刺激，我决定自己写，这应该是一个非常简单的，基础类。在这里：

public class Customer
{       
    public string CUSTOMERNAME;
    public List<Site> sites = new List<Site>();
    public Customer()
    {
    }
    public void AddSite(Site location)
    {
        sites.Add(location);
    }
}
public class Site
{
        public string SITENAME;
        public Address SITEADDRESSDETAILS;
        public string SITEPHONENUMBER;
        public Site(string sitename, Address siteaddress, string tel)
        {
            SITENAME = sitename;
            SITEADDRESSDETAILS = siteaddress;
            SITEPHONENUMBER = tel;
        }
}
public class Address 
{
    public List<string> address = new List<string>();
    public Address() {
    }
    public void AddAddressDetail(string line)
    {
        address.Add(line);
    }
}

现在一切似乎都正常了，但我只是忍不住觉得事情可以做得更好。我已经使用以下代码对其进行了测试：

static void Main(string[] args)
{
    Customer customer = new Customer();
    customer.CUSTOMERNAME = "Max Hammer Ltd";
    Address addy = new Address();
    addy.AddAddressDetail("1 Edgerail Close");
    addy.AddAddressDetail("Greenbushes");
    addy.AddAddressDetail("Bluehill");
    addy.AddAddressDetail("Surrey");
    addy.AddAddressDetail("RH0 6LD");
    Site surreyOffice = new Site("Surrey Office", addy, "01737 000000");
    addy = new Address();
    addy.AddAddressDetail("6 Electric Avenue");
    addy.AddAddressDetail("Brixton");
    addy.AddAddressDetail("London");
    addy.AddAddressDetail("SW4 1BX");
    Site brixtonOffice = new Site("Brixton Office", addy, "020 7101 3333");
    customer.AddSite(surreyOffice);
    customer.AddSite(brixtonOffice);
    Console.WriteLine(customer.CUSTOMERNAME);
    int numberOfSutes = customer.sites.Count;
    for (int i = 0; i < numberOfSutes; i++)
    {
        Console.WriteLine(customer.sites[i].SITENAME);
        foreach (string line in customer.sites[i].SITEADDRESSDETAILS.address)
        {
            Console.WriteLine(line);
        }
        Console.WriteLine(customer.sites[i].SITEPHONENUMBER);
    }
    Console.ReadKey();
}

我对我的Main课不满意，我不知道为什么 - 即使它做了我想要的。有什么提示，指点吗？

这是一个正确的类实现和初始化吗?

以下是我的看法：

使用具有适当大小写的属性。
为什么在地址中列出行？！[已删除]。
直接访问列表。

public class Customer
{
    public string Name {get; set;}
    public List<Site> Sites { get; set; }
    public Customer()
    {
        Sites = new List<Site>();
    }
}
public class Site
{
    public string Name { get; set; }
    public string Address { get; set; }
    public string PhoneNumber { get; set; }
    public Site(string sitename, string siteaddress, string tel)
    {
        Name = sitename;
        Adress = siteaddress;
        PhoneNumber = tel;
    }
}
public class Program
{
    static void Main(string[] args)
    {
        Customer customer = new Customer();
        customer.Name = "Max Hammer Ltd";
        string address = string.Join(Environment.NewLine, new []{"1 Edgerail Close", "Greenbushes", "Bluehill"
             , "Surrey", "RH0 6LD"});
        Site surreyOffice = new Site("Surrey Office", address, "01737 000000");
        address = string.Join(Environment.NewLine, new[]{"1 Edgerail Close", "Greenbushes", "Bluehill"
                , "Surrey", "RH0 6LD"});
        Site brixtonOffice = new Site("Brixton Office", address, "020 7101 3333");
        customer.Sites.Add(surreyOffice);
        customer.Sites.Add(brixtonOffice);
        Console.WriteLine(customer.Name);
        foreach (Site site in customer.Sites)
        {
            Console.WriteLine(site.Name);
            Console.WriteLine(site.Adress);
            Console.WriteLine(site.PhoneNumber);
            Console.WriteLine();
        }
        Console.ReadKey();
    }
}

相关文章：

最新信息: