这是解析函数参数的有效方法吗？

Question

所以我是C#的新手,但是我已经想从其他更高级别的语言中得到的一件事就是能够在(接近)C中进行按位操作.我有一堆函数,其中一些或所有参数都是可选的,我喜欢开关,所以我构建了一个函数,将布尔数组转换为无符号的Shorts,这允许我基本上将一个布尔数组Mux为交换机的单个值:

namespace firstAsp.Helpers{
    public class argMux{                       
        public static ushort ba2ushort (bool[] parms){  
            //initialize position and output                
            ushort result = 0;
            int i = parms.Length-1;
            foreach (bool b in parms){
                if (b)//put a one in byte at position of b
                    //bitwise or with position
                    result |= (ushort)(1<<i);
                i--;
            }
            return result;               
        }
    }
}

这是一个示例用例:

public IActionResult Cheese(string fname,string lname)
    {
        bool[] tf = {fname!=null,lname!=null};

        switch(argMux.ba2ushort(tf)){
         case 3:
            @ViewData["Data"]=$"Hello, {fname} {lname}";
            break;
         case 2:
            @ViewData["Data"]=$"Hello, {fname}";
            break;
         case 1:
            @ViewData["Data"]=$"Hello, Dr. {lname}";
            break;
         case 0:
            @ViewData["Data"]="Hello, Dr. CheeseBurger";
            break;
        }
        return View();
    }

我的问题是,这是一种有效的方法,还是有一种优越的方式？我的目标是简单的使用,这绝对为我提供,但我也希望它是在运行时快速的高效代码.有什么指针吗？这是一种愚蠢的方式吗？欢迎任何和所有的反馈,如果你相信它,你甚至可以称我为白痴,我不是太敏感.谢谢!

Answer 1

这一切都很糟糕.

编写方法的正确方法是不使用以下方法:

public IActionResult Cheese(string firstName, string lastName)
{
    @ViewData["Data"]=$"Hello, {firstName ?? "Dr."} {lastName ?? "Cheeseburger"}";
    return View();
}

我已经喜欢来自其他更高级语言的一件事是能够在(接近)C中进行按位操作.

如果你正在努力解决高级别的业务问题,那么你可能做错了什么.使用高级业务代码解决高级业务问题.

另外,如果你在C#中使用无符号类型,那么你做错了几率很好.无符号类型用于与非托管代码进行互操作.在C#中使用ushort或uint或ulong是非常罕见的.逻辑上无符号的数量(如数组的长度)始终表示为有符号数量.

C#有许多功能,旨在确保拥有COM库的人可以继续使用他们的库,以便需要原始的,未经检查的指针运算性能的人在合理安全时可以这样做.不要将这些低级编程功能的存在误认为是C#通常用作低级编程语言的证据.编写代码,使其清晰地显示为业务工作流的实现.

代码的业务是将名称呈现为字符串,因此应该清楚地将其视为将字符串呈现为字符串.如果我要求你在一张纸上写下一个名字,那么你要做的第一件事就是不要做一点帮助你,所以它也不应该在这里.

现在,有可能是某些情况下,这种事情是明智的,在这些情况下,你应该使用一个枚举,而不是治疗短位字段:

[Flags]
enum Permissions 
{
  None = 0x00,
  Read = 0x01,
  Write = 0x02,
  ReadWrite = 0x03,
  Delete = 0x04,
  ReadDelete = 0x05,
  WriteDelete = 0x06,
  ReadWriteDelete = 0x07
}
...
static Permissions GetPermission(bool read, bool write, bool delete) {
  var p1 = read ? Permissions.Read : Permissions.None;
  var p2 = write ? Permissions.Write : Permissions.None;
  var p3 = delete ? Permissions.Delete : Permissions.None;
  return p1 | p2 | p3;
}

现在你方便了

switch(p)
{
  case Permissions.None: ...
  case Permissions.Read: ...
  case Permissions.Write: ...
  case Permissions.ReadWrite: ...

但请注意,我们将所有内容保留在业务领域.我们在做什么？检查权限.那么代码是什么样的？ 就像检查权限一样.没有弄乱一堆比特,然后打开一个整数.