如何重构这个很大的条件

Question

如何重构这么大的条件？武器是一个清单，它的游戏对象

如何重新分配这个很大的条件？武器是一个清单，它的gameObject

public class Customanger : MonoBehaviour
{
    public List<GameObject> weapon;
    public static Customanger singleton;
    private void Awake() => singleton = this;
};

if (Customanger.singleton.weapon[1].activeSelf ||
    Customanger.singleton.weapon[2].activeSelf ||
    Customanger.singleton.weapon[3].activeSelf ||
    Customanger.singleton.weapon[4].activeSelf ||
    Customanger.singleton.weapon[5].activeSelf ||
    Customanger.singleton.weapon[8].activeSelf ||
    Customanger.singleton.weapon[10].activeSelf ||
    Customanger.singleton.weapon[12].activeSelf ||
    Customanger.singleton.weapon[13].activeSelf ||
    Customanger.singleton.weapon[14].activeSelf ||
    Customanger.singleton.weapon[15].activeSelf ||
    Customanger.singleton.weapon[16].activeSelf ||
    Customanger.singleton.weapon[17].activeSelf)
{
    dosomething();
}

Answer 1

您可以尝试 .any（），

确定序列的任何元素是否存在或满足
条件。

//Here I considered length of Weapon array/list is 17
if (Customanger.singleton.weapon.Any(x => x.activeSelf))
{
    dosomething();
}

---

如果您想检查相同的武器子集，则可以在下面尝试，

var subSetOfWeapons = Customanger.singleton.weapon.GetRange(1, 17);

if (subSetOfWeapons.Any(x => x.activeSelf))
{
    dosomething();
}

有关更多详细信息： list.getrange（int index，int count）

Answer 2

我不知道您正在使用哪种类型。我所做的是检查哪种模式经常出现。我看到您始终访问武器列表/数组。通常可以迭代索引器。

有些假设是由于信息的不完整而进行的。

您可以为此编写一个循环：

private bool CheckActiveSelf(List<WeaponThing> weapons)
{
    // iterate each weapon
    foreach(var weapon in weapons)
        // when activeSelf, return true
        if(weapon.activeSelf)
            return true;
   
    return false;
}


// instead of passing the Customanger, you should pass the deepest
// level. So when the weapon system is used elsewhere, you can still
// use the method.
if(CheckActiveSelf(Customanger.singleton.weapon))
{
    DoSomething();
}

Answer 3

无论此检查的含义如何，在OOP世界中，应将其作为实例方法（或属性）在ustomanger class中的实例方法（或属性）实现：

class Customanger
{
    private static int[] activeWeaponIndexes = { 1, 2, 3, 4, 5, 8, 10, 12, 13, 14, 15, 16, 17 };
    public bool HasActiveWeapon => activeWeaponIndexes.Any(x => weapon[x].activeSelf);
}

然后，条件将减少到：

if (Customanger.singleton.HasActiveWeapon) { \*do smth*\ }

Answer 4

您可以做这样的事情：

var weaponIds = new List<int>() { 1, 2, 3, 4, 5, 8, 10, 12, 13, 14, 15, 16, 17 };

var weapons = weaponIds.Select(x => Customanger.singleton.weapon[x]);

if (weapons.Where(x => x.activeSelf).Any())
{
    DoSomething();
}

也许可以将武器阵列分为类别，您可能会做得更好吗？我无法为您提供更多的代码。

编辑：删除完全不必要的Tolist（）

Answer 5

您是如何确定{1、2、3、4、5、8、10、12、13、13、14、15、16、17}的相关武器指数的？

---

如果它们都是通过履行某种条件来区分自己与其余武器的武器指数，则可以直接识别相关的武器，而不会打扰这些指数。

例如，您可以使用.where（）来滤除相关武器：

Customanger.singleton.weapon
    .Where(weapon => <weapon fulfills condition>)

假设情况是相关武器都有一个共同的特征：它们都属于已加载的场景。 可以将条件定义为

<weapon fulfills condition> = weapon.scene.isLoaded

：

IEnumerable<GameObject> weaponsInLoadedScenes = Customanger.singleton.weapon
    .Where(weapon => weapon.scene.isLoaded);

然后 相关武器：

if (weaponsInLoadedScenes.Any(weapon => weapon.activeSelf))
{
    DoSomething();
}

Where（）和.yany（）都在命名空间 system.linq 中找到。

---

另一方面，如果相关武器指数不是基于共同武器特征选择的 ，我基本上是指<​​a href =“ https://stackoverflow.com/questions/questions/72486634/72486634/ -to-Refactor-this-if-if条件＃72487116“>另一个人的答案。

首先，创建相关武器指数的集合（最好使用以某种方式描述为什么这些是相关武器指数的变量名称）：

var relevantWeaponIndices = new List<int>() { 1, 2, 3, 4, 5, 8, 10, 12, 13, 14, 15, 16, 17 };

然后，验证条件；首先要明确定义相关武器：

var relevantWeapons = relevantWeaponIndices
    .Select(index => Customanger.singleton.weapon[index]);

if (relevantWeapons.Any(weapon => weapon.activeSelf))
{
    DoSomething();
}

或者通过获取相关武器内联武器：

if (relevantWeaponIndices.Any(index => Customanger.singleton.weapon[index].activeSelf))
{
    DoSomething();
}

.select（）和和.any（）（）在命名空间系统中找到.linq 。

Answer 6

有多种方法可以这样做，您可以包括或排除收藏中的项目，而不是检查“ ActiveSelf”标志
前任：

List<GameObject> excludedList = new List<GameObject>() { 
Customanger.singleton.weapon[1], 
Customanger.singleton.weapon[3], 
Customanger.singleton.weapon[5]};
if (Customanger.singleton.weapon.Except(excludedList).Any(x => x.activeSelf))
    dosomething();
    ```