2010-11-30 157 views
4

This site声称set_union等同于下面的代码:什么是正确的std :: set_union代码?

template <class InputIterator1, class InputIterator2, class OutputIterator> 
    OutputIterator set_union (InputIterator1 first1, InputIterator1 last1, 
          InputIterator2 first2, InputIterator2 last2, 
          OutputIterator result) 
{ 
    while (true) 
    { 
    if (*first1<*first2) *result++ = *first1++; 
    else if (*first2<*first1) *result++ = *first2++; 
    else { *result++ = *first1++; first2++; } 

    if (first1==last1) return copy(first2,last2,result); 
    if (first2==last2) return copy(first1,last1,result); 
    } 
} 

但是,这似乎有些奇怪:不就是崩溃(或导致其他未定义的行为)如果范围之一是空的?这两个if子句是不是应该在while循环的开头,而不是结尾?

+4

cplusplus.com不是权威的参考。 ISO/IEC 14882是,并且不包含`set_union`的示例代码,只是规范(允许输入序列为空)。 – 2010-11-30 20:53:27

回答

4

我同意它看起来完全坏了。作为比较,这里是STLport代码:

 
template <class _InputIter1, class _InputIter2, class _OutputIter, 
      class _Compare> 
_OutputIter __set_union(_InputIter1 __first1, _InputIter1 __last1, 
         _InputIter2 __first2, _InputIter2 __last2, 
         _OutputIter __result, _Compare __comp) { 
    _STLP_DEBUG_CHECK(_STLP_PRIV __check_range(__first1, __last1)) 
    _STLP_DEBUG_CHECK(_STLP_PRIV __check_range(__first2, __last2)) 
    while (__first1 != __last1 && __first2 != __last2) { 
    if (__comp(*__first1, *__first2)) { 
     _STLP_VERBOSE_ASSERT(!__comp(*__first2, *__first1), _StlMsg_INVALID_STRICT_WEAK_PREDICATE) 
     *__result = *__first1; 
     ++__first1; 
    } 
    else if (__comp(*__first2, *__first1)) { 
     _STLP_VERBOSE_ASSERT(!__comp(*__first1, *__first2), _StlMsg_INVALID_STRICT_WEAK_PREDICATE) 
     *__result = *__first2; 
     ++__first2; 
    } 
    else { 
     *__result = *__first1; 
     ++__first1; 
     ++__first2; 
    } 
    ++__result; 
    } 
    return _STLP_STD::copy(__first2, __last2, _STLP_STD::copy(__first1, __last1, __result)); 
}