C#重构开关语句与空检查

内容导读

互联网集市收集整理的这篇技术教程文章主要介绍了C#重构开关语句与空检查，小编现在分享给大家，供广大互联网技能从业者学习和参考。文章包含2201字，纯文字阅读大概需要4分钟。

内容图文

我确实在使用几个switch语句时遇到了麻烦,并且我觉得有一种更好的方法可以实现最终目标.

所以本质上我是将viewmodel传递给方法.该方法首先从数据库中检索与视图模型相关的对象,然后switch语句对特定属性进行空检查.基于该结果,另一个switch语句对视图模型进行另一个null检查.在每个点上,都会从数据库中为对象分配值,然后在最后进行数据库更新.

这是代码

        public async Task UpdateContractWithRepository(ViewModel viewModel)
    {
        // Get the contract from db
        Contract contract = GetContract(viewModel.Id);

        switch (viewModel.RepositoryId == null)
        {
            case true:
                switch (contract.RepositoryId == null)
                {
                    case true:
                        // nothing to do
                        // no change
                        break;
                    case false:
                        // Unassign Repository
                        UpdateRepositoryAssignment(contract.RepositoryId, false);

                        // Update properties
                        contract.RepositoryId = null;
                        contract.ConsumedUnits = null;
                        break;
                }
                break;
            case false:
                switch (contract.RepositoryId == null)
                {
                    case true:
                        // assign repository
                        UpdateRepositoryAssignment(viewModel.RepositoryId, true);

                        // Get repository
                        Repository repository = GetRepository(viewModel.RepositoryId);

                        // Update properties
                        contract.RepositoryId = repository.Id;
                        contract.ConsumedUnits = repository.Units;
                        break;
                    case false:
                        // assign repository
                        UpdateRepositoryAssignment(viewModel.RepositoryId, true);

                        // Get repository
                        Repository repository = GetRepository(viewModel.RepositoryId);

                        // Update properties
                        contract.RepositoryId = repository.Id;
                        contract.ConsumedUnits = repository.Units;
                        break;
                }
                break;

        }

        UpdateContract(contract);
    }

我想我可能可以取消switch语句,而使用if语句,但根据我的判断,仍然会有一些嵌套.只是想知道是否有人有任何建议.

我在这里已经看过了重构switch语句,但是它们似乎并未真正涵盖空检查.

任何帮助表示赞赏！

解决方法:

如果只取出两个布尔值,则可以简化整个代码：

bool IsVMRepoNull = viewModel.RepositoryId == null;
bool IsContractRepoNull = contract.RepositoryId == null;

if(IsVMRepoNull && !IsContractRepoNull )
{
  UpdateRepositoryAssignment(contract.RepositoryId, false);

 // Update properties
  contract.RepositoryId = null;
  contract.ConsumedUnits = null;
}
else if(!IsVMRepoNull)
{
  UpdateRepositoryAssignment(viewModel.RepositoryId, true);

  // Get repository
  Repository repository = GetRepository(viewModel.RepositoryId);

  // Update properties
  contract.RepositoryId = repository.Id;
  contract.ConsumedUnits = repository.Units;
}

内容总结

以上是互联网集市为您收集整理的C#重构开关语句与空检查全部内容，希望文章能够帮你解决C#重构开关语句与空检查所遇到的程序开发问题。 如果觉得互联网集市技术教程内容还不错，欢迎将互联网集市网站推荐给程序员好友。

内容备注

版权声明：本文内容由互联网用户自发贡献，该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务，不拥有所有权，不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容， 请发送邮件至 gblab@vip.qq.com 举报，一经查实，本站将立刻删除。

内容手机端