Làm cách nào để đóng gói các biến 'toàn cầu' trong C #? /thực hành tốt nhất

Trong C #, cách thực hành tốt nhất để đóng gói các biến tôi cần sử dụng trong nhiều phương thức là gì? Có thể chỉ cần khai báo chúng ở đầu lớp của tôi trên hai phương thức không?

Ngoài ra nếu tôi đang sử dụng cài đặt ứng dụng từ tệp cấu hình của mình thì tôi có nên sử dụng getter không? như thế này...

private string mySetting{ get { return WebConfigurationManager.AppSettings["mySetting"]; } }

Thực hành tốt nhất là gì?

Nó không chỉ ổn. Theo cuốn sách Clean Code, đây thực sự là một cách thực hành rất tốt và chú Bob thực sự khuyến khích điều đó. Một biến được sử dụng bởi nhiều phương thức có thể cho thấy mức độ gắn kết cao giữa các phương thức. Hơn nữa, một mức độ cao của các biến đối tượng cũng có thể gợi ý rằng lớp nên được chia làm hai để khai báo chúng là biến đối tượng có thể giúp bạn tìm ra các ứng cử viên lớp ẩn.

Biến cấp đối tượng không phải là biến toàn cục, vì vậy đừng ngại sử dụng chúng nếu chúng được chia sẻ bằng nhiều phương pháp khác nhau.

Đóng gói các thiết lập của bạn một cách liên tục là một ý tưởng tuyệt vời.

Những gì tôi làm là tạo một lớp cài đặt hoặc một lớp toàn cầu tĩnh hoặc nhiều lớp đối tượng mà sau đó tôi sẽ quản lý bằng phép nội xạ phụ thuộc. Sau đó, tôi tải tất cả các cài đặt từ cấu hình vào lớp đó khi khởi động.

Tôi cũng đã viết một thư viện nhỏ sử dụng sự phản chiếu để làm cho việc này trở nên dễ dàng hơn.

Khi cài đặt của tôi nằm trong tệp cấu hình của tôi

<?xml version="1.0" encoding="utf-8" ?>
<configuration>   
    <appSettings>
        <add key="Domain" value="example.com" />
        <add key="PagingSize" value="30" />
        <add key="Invalid.C#.Identifier" value="test" />
    </appSettings>
</configuration>

Tôi tạo một lớp tĩnh hoặc cá thể tùy thuộc vào nhu cầu của tôi. Đối với các ứng dụng đơn giản chỉ có một vài cài đặt, một lớp tĩnh là ổn.

private static class Settings
{
    public string Domain { get; set; }

    public int PagingSize { get; set; }

    [Named("Invalid.C#.Identifier")]
    public string ICID { get; set; }

}

Sau đó, sử dụng thư viện của tôi gọi một trong hai Inflate.Statichoặc Inflate.Instanceđiều thú vị là tôi có thể sử dụng bất kỳ nguồn giá trị quan trọng nào.

using Fire.Configuration;

Inflate.Static( typeof(Settings), x => ConfigurationManager.AppSettings[x] );

Tất cả mã cho điều này là trong GitHub tại https://github.com/Enexure/Enexure.Fire.Configuration

Thậm chí còn có một gói nuget:

PM> Cài đặt-Gói Enexure.Fire.Configuration

Mã để tham khảo:

using System;
using System.Linq;
using System.Reflection;
using Fire.Extensions;

namespace Fire.Configuration
{
    public static class Inflate
    {
        public static void Static( Type type, Func<string, string> dictionary )
        {
            Fill( null, type, dictionary );
        }

        public static void Instance( object instance, Func<string, string> dictionary )
        {
            Fill( instance, instance.GetType(), dictionary );
        }


        private static void Fill( object instance, Type type, Func<string, string> dictionary ) 
        {

            PropertyInfo[] properties;
            if (instance == null) {

                // Static
                properties = type.GetProperties( BindingFlags.Public | BindingFlags.Static | BindingFlags.DeclaredOnly );
            } else {

                // Instance
                properties = type.GetProperties( BindingFlags.Public | BindingFlags.Instance | BindingFlags.DeclaredOnly );
            }

            // Get app settings and convert
            foreach (PropertyInfo property in properties) {
                var attributes = property.GetCustomAttributes( true );
                if (!attributes.Any( x => x is Ignore )) {

                    var named = attributes.FirstOrDefault( x => x is Named ) as Named;

                    var value = dictionary((named != null)? named.Name : property.Name);

                    object result;
                    if (ExtendConversion.ConvertTo(value, property.PropertyType, out result)) {
                        property.SetValue( instance, result, null );
                    }
                }
            }
        }
    }
}