Как дезинфицировать/скрывать конфиденциальные данные в C#

Question

Для ведения транзакций с транзакциями мне нужно указать, предоставляют ли пользователи необходимые поля/информацию или нет. Если да, я заменю реальную информацию на *. Если нет, я сделаю этот запрос неудачным, возвращая значение «false».

Ниже приведены класс и метод определения данных.

public class CreditCard 
{ 
public string brand { get; set; } 
public string billingphone { get; set; } 
} 

class Program 
{ 
    static void Main(string[] args) 
    { 
    CreditCard creditcard =new CreditCard(){brand="VISA", billingphone="777-111-2222"}; 
    bool IsVerified= SanitizePaymnetData(creditcard);   
    } 

    private bool SanitizePaymnetData(ref CreditCard creditcard) 
    { 
     bool isDataSanitized = true; 

     if(!String.IsNullOrEmpty(creditcard.brand)) 
     { 
      creditcard.brand ="*";    
     } 
     else 
     { 
      isDataSanitized = false; 
     } 

     if (!String.IsNullOrEmpty(creditcard.billingphone)) 
     { 
      creditcard.billing_phone = "*"; 
     } 
     else 
     { 
      isDataSanitized = false; 
     } 

     return isDataSanitized; 
    } 
} 

Метод «SanitizePaymentData» выглядит как уродливый. Есть ли способ сделать его более чистым/аккуратным?

Answer 1

Основная проблема, которую я вижу здесь, потенциально является ремонтопригодностью. По мере добавления полей в CreditCard вам нужно будет добавить дополнительные проверки в SantitzePaymentData(). Если другие разработчики позже будут работать над этим, они могут не знать об этом требовании, или вы можете забыть о нем позже, а конфиденциальная информация начнет появляться в вашем журнале. Так как это не приведет к ошибке исключения или компиляции, возможно, потребуется некоторое время, прежде чем вы заметите, что это произошло.

Я не знаю, действительно ли это настоящий проект или что-то, что вы используете для изучения, но возникает другая проблема, когда вы решите добавить другой способ оплаты, например, банковский перевод. Затем вам придется создать еще одну функцию SanitizePaymentData(), которая работает в новом классе платежей, но у нее будет по существу дублированный код из вашего исходного метода. Это дублирует работу, нарушает принцип DRY и создает беспорядок в вашем проекте.

На мой взгляд, более чистым подходом было бы использование атрибутов в вашем классе, которые отмечают чувствительные поля, которые нуждаются в дезинфекции, а затем отвечают соответствующим образом на ваш журнал.

public class CreditCard 
{ 
    [SanitizeInLog] 
    public string Brand {get; set;} 

    [SanitizeInLog] 
    public string BillingPhone {get; set;} 
} 

Теперь, ваш метод регистрации/класс может просто проверить, существуют ли эти атрибуты на поле, и включите их для «*» или что-то еще.