C#でネストが深いコードを解消する手法｜可読性を高めるリファクタリングのコツ

public void UpdateUserProfile(User user)
{
    if (user != null)
    {
        if (!string.IsNullOrEmpty(user.Name))
        {
            if (user.Age >= 18)
            {
                // 本来行いたいメインの更新処理
                Console.WriteLine($"{user.Name}のプロファイルを更新しました。");
            }
            else
            {
                Console.WriteLine("18歳未満は更新できません。");
            }
        }
        else
        {
            Console.WriteLine("名前が空です。");
        }
    }
}

public void UpdateUserProfile(User user)
{
    // ガード節：無効な条件を先に排除する
    if (user == null) return;

    if (string.IsNullOrEmpty(user.Name))
    {
        Console.WriteLine("名前が空です。");
        return;
    }

    if (user.Age < 18)
    {
        Console.WriteLine("18歳未満は更新できません。");
        return;
    }

    // メインの処理（ネストが浅くなる）
    Console.WriteLine($"{user.Name}のプロファイルを更新しました。");
}

public List<string> GetExpensiveProductNames(List<Order> orders)
{
    var result = new List<string>();
    foreach (var order in orders)
    {
        if (order.IsActive)
        {
            foreach (var item in order.Items)
            {
                if (item.Price > 10000)
                {
                    result.Add(item.Name);
                }
            }
        }
    }
    return result;
}

using System.Linq;

public List<string> GetExpensiveProductNames(List<Order> orders)
{
    return orders
        .Where(o => o.IsActive)
        .SelectMany(o => o.Items)
        .Where(item => item.Price > 10000)
        .Select(item => item.Name)
        .ToList();
}

public double CalculateDiscount(object person)
{
    if (person is Student s)
    {
        if (s.Grade > 3)
        {
            return 0.2;
        }
        return 0.1;
    }
    else if (person is Senior sn)
    {
        if (sn.Age >= 80)
        {
            return 0.3;
        }
        return 0.15;
    }
    return 0;
}

public double CalculateDiscount(object person) => person switch
{
    Student { Grade: > 3 } => 0.2,
    Student                => 0.1,
    Senior { Age: >= 80 }  => 0.3,
    Senior                 => 0.15,
    _                      => 0 // デフォルトケース
};

public string GetCityName(Customer customer)
{
    if (customer != null)
    {
        if (customer.Address != null)
        {
            return customer.Address.City;
        }
    }
    return "Unknown";
}

public string GetCityName(Customer customer)
{
    // null条件演算子とnull合体演算子を組み合わせる
    return customer?.Address?.City ?? "Unknown";
}

public void ProcessData(Data data)
{
    if (data.IsValid)
    {
        // データの集計ロジック
        foreach (var item in data.Items)
        {
            // 複雑な計算
            if (item.Status == "Active")
            {
                // ...
            }
        }

        // 保存ロジック
        try
        {
            // DBアクセスなど
        }
        catch (Exception ex)
        {
            // エラー処理
        }
    }
}

public void ProcessData(Data data)
{
    if (!data.IsValid) return;

    var activeItems = FilterActiveItems(data.Items);
    var summary = AggregateData(activeItems);
    
    SaveData(summary);
}

private List<Item> FilterActiveItems(IEnumerable<Item> items) => 
    items.Where(i => i.Status == "Active").ToList();

private Summary AggregateData(List<Item> items)
{
    // 集計ロジックに集中できる
    return new Summary();
}

private void SaveData(Summary summary)
{
    try { /* 保存処理 */ }
    catch (Exception ex) { /* エラー処理 */ }
}

string statusMessage;
if (score >= 80)
{
    statusMessage = "合格";
}
else
{
    statusMessage = "不合格";
}

string statusMessage = score >= 80 ? "合格" : "不合格";

public void CompleteOrder(Order order)
{
    if (order != null)
    {
        if (order.Status == OrderStatus.Pending)
        {
            if (order.Items.Count > 0)
            {
                bool allInStock = true;
                foreach (var item in order.Items)
                {
                    if (!InventoryService.CheckStock(item.Id))
                    {
                        allInStock = false;
                        break;
                    }
                }

                if (allInStock)
                {
                    order.Status = OrderStatus.Completed;
                    Database.Save(order);
                    EmailService.SendConfirmation(order.CustomerEmail);
                }
                else
                {
                    throw new InvalidOperationException("在庫不足の商品があります。");
                }
            }
            else
            {
                throw new InvalidOperationException("注文アイテムが空です。");
            }
        }
    }
}

public void CompleteOrder(Order order)
{
    // 1. 基本的なガード節
    if (order == null) return;
    if (order.Status != OrderStatus.Pending) return;

    // 2. 業務ルールのバリデーションをメソッド化
    ValidateOrder(order);

    // 3. 在庫チェックにLINQを活用
    if (!order.Items.All(item => InventoryService.CheckStock(item.Id)))
    {
        throw new InvalidOperationException("在庫不足の商品があります。");
    }

    // 4. メイン処理の実行
    ExecuteOrderCompletion(order);
}

private void ValidateOrder(Order order)
{
    if (!order.Items.Any())
    {
        throw new InvalidOperationException("注文アイテムが空です。");
    }
}

private void ExecuteOrderCompletion(Order order)
{
    order.Status = OrderStatus.Completed;
    Database.Save(order);
    EmailService.SendConfirmation(order.CustomerEmail);
}

// 従来のスタイル（中括弧で囲むため一段階ネストする）
namespace MyProject.Services
{
    public class OrderService { }
}

// ファイルスコープ名前空間（ネストが解消される）
namespace MyProject.Services;

public class OrderService { }