.net - Strategie uniknąć magiczne ciąg

Question

W kodzie w pracy, mamy wiele zastosowań magicznych ciągów jak poniższym fragmencie kodu:

if (user.HasRight("Profile.View")) {...} 

więc istnieje wiele miejsc, gdzie mijamy ciąg jako parametr zobacz, czy użytkownik ma określone prawo. Nie podoba mi się to, ponieważ generuje wiele magicznych ciągów.

Jaki byłby lepszy sposób na zrobienie tego?

Enum, stała, klasa?

Answer 1

W tym konkretnym przypadku użyj Enum. Nie będzie żadnych magicznych strun i jeśli Enum się zmieni (w sposób, który przełamie magiczne rozwiązanie), aplikacja nie będzie się już kompilować.

public enum ProfilePermissions 
{ 
    View, 
    Create, 
    Edit, 
    Delete 
} 

Następnie można po prostu:

if(user.HasRight(ProfilePermissions.View)) { } 

Można również użyć klasy, ale wtedy ograniczyć się jeśli chodzi o bardziej skomplikowane scenariusze. Na przykład, prosta zmiana wyliczania do czegoś podobnego:

public enum ProfilePermissions 
{ 
    View = 1, 
    Create = 2, 
    Edit = 4, 
    Delete = 8 
} 

pozwoliłoby na korzystanie operatory bitowe dla bardziej skomplikowanych uprawnień (na przykład sytuacji, w której potrzebuje użytkownik albo utworzyć lub usunąć):

if(user.HasRight(ProfilePermissions.Create | ProfilePermissions.Delete)); 

Answer 2

Możesz wykonywać ciągłe ciągi w C#.

Można zdefiniować wszystkie struny w nagłówku tak:

const string PROFILE_VIEW "Profile.View"; 

Nie wiem, czy to jest „najlepszy” sposób, ale jest na pewno lepsze niż wartości magiczne w kodzie.

Answer 3

Jest to również dość powszechne w środowisku .NET. Przykładami są System.Windows.DataFormats i System.Net.WebRequestMethods.Http. że chcesz różnorodność readonly:

public static class MumbleRights { 
    public static readonly string ProfileView = "Profile.View"; 
    // etc.. 
} 

Answer 4

Tworzenie klasy, która silnie typy tych właściwości, jak

public static class UserInfo 
{ 
    public static bool CanViewProfile { get { return User.HasRight("Profile.View"); } } 
} 

Pozwoli to zachować swoje „magiczne ciągi” w jednym miejscu w kodzie. Enum też zadziała, ale moim zdaniem nie jest tak czytelny.

Uwaga: mój przykład ma działać jako proxy właściwości dla zalogowanego użytkownika, czyli klasy statycznej. Jeśli chciałbyś czegoś, co działałoby na bardziej bezpośrednie dane (np. Listę użytkowników), ten typ klasy musiałby być niestatyczny i utworzony na podstawie konta użytkownika.

Answer 5

Ja po drugie sposób pokazany przez "Justin Niessner". Ale w niektórych przypadkach wolałbym pisać po konstrukcji kodu.

public class User 
    { 
     public Permission Permission { get; set; } 

    } 
    public abstract class Permission 
    { 

    } 
    public class ViewPermission:Permission 
    { 

    } 

i można spożywać go jako

User user=new User(); 
      if(user.Permission is ViewPermission) 
      { 

      } 

Answer 6

metod przedłużających! Trzymaj je w tym samym miejscu, aby śledzić wszystkie magiczne ciągi.

public static class UserRightsExtensions { 
    public static bool CanReadProfile(this User user) 
    { 
    return user.HasRight("Profile.View"); 
    } 

    // etc.. 
} 

Następnie można:

if (user.CanReadProfile()) .....