首页 > 解决方案 > 使用基于此标志的条件语句是否比添加更多代码行更有效?

问题描述

我有一个排序函数,它接受一个布尔参数(降序),如果&是一个选择算法的枚举类(这里代码的子集用于(bubblesort))desc,则它以相反的顺序排序truealgoalgo::BUBBLE

使用这个内联条件语句 ( if (!desc ? A[j] > A[j + 1] : A[j] < A[j + 1])),我可以避免为反向排序重写整个代码,因为它会根据desc标志评估适当的条件。但我想知道这是否会产生不必要的开销,因为它反复检查标志 [ (n-1)*(1+2+...+n-1) 次]。对于更大的数据元素,这种开销会很大吗?更多代码还是更多开销?

void Array<T>::sort(bool desc = false, algo a)
{

 if (algo == algo::BUBBLE)
 {
    bool wasSwapped = true;
        for (size_t i = 0; i < size - 1 && wasSwapped; i++)
        {
            switched = false;
            for (size_t j = 0; j < size - i - 1; j++)
            {
                if (!desc ? A[j] > A[j + 1] : A[j] < A[j + 1])
                {
                    wasSwapped = true;
                    swap(A[j], A[j + 1]);
                }
            }
        }
  }
}

A并且size是私有数据成员(分别是数组指针和大小)。

标签: c++sortingconditional-statementsbubble-sortflags

解决方案


为了代码清晰起见,最好将其设为非成员函数模板并将其传递给比较函子。确保将函数放在应用程序的命名空间中,这样就不会与命名空间中的同名函数混淆std

假设Array<T>::A是可访问的,

namespace MyApp
{
   template <typename T, typename Compare = std::less<T>>
   void sort(Array<T>& array, algo a, Compare compare = Compare());
   {
      if (a == algo::BUBBLE)
      {
         bool wasSwapped = true;
         for (size_t i = 0; i < size - 1 && wasSwapped; i++)
         {
            switched = false;
            for (size_t j = 0; j < size - i - 1; j++)
            {
               if (!compare(array.A[j], array.A[j + 1]))
               {
                  wasSwapped = true;
                  swap(array.A[j], array.A[j + 1]);
               }
            }
         }
      }
   }
}

现在您可以使用:

Array<int> a = { ... };
MyApp::sort(a, algo::BUBBLE);                      // std::less<int> is the default functor.
MyApp::sort(a, algo::BUBBLE, std::greater<int>()); // Explicit compare functor.

推荐阅读