#c #unit-testing #countdownlatch
#c #модульное тестирование #обратный отсчет
Вопрос:
Я попытался реализовать обратный отсчет, используя повышающие мьютексы и переменную условия. Ниже приведен код, и я хотел бы знать, нужно ли мне добавить что-нибудь еще.
Как я могу также модульно протестировать этот код?
template< class TypeVal >
class AtomicCounter
{
public:
AtomicCounter( TypeVal val ) : m_typeVal( val )
{
}
AtomicCounter() : m_typeVal(0)
{
}
AtomicCounter(const AtomicCounteramp; cpy) : m_typeVal(cpy.m_typeVal)
{
boost::mutex::scoped_lock scoped_lock(cpy.m_atomicMutex);
m_typeVal = cpy.m_typeVal;
}
const AtomicCounteramp; operator=(const AtomicCounteramp; other)
{
if (this == amp;other)
return *this;
boost::mutex::scoped_lock lock1(amp;m_atomicMutex < amp;other.m_atomicMutex ? m_atomicMutex : other.m_atomicMutex);
boost::mutex::scoped_lock lock2(amp;m_atomicMutex > amp;other.m_atomicMutex ? m_atomicMutex : other.m_atomicMutex);
m_typeVal = other.m_typeVal;
return *this;
}
virtual ~AtomicCounter()
{
}
const TypeValamp; getCount() const
{
boost::mutex::scoped_lock lock( m_atomicMutex );
return m_typeVal;
}
const TypeValamp; setCount( const TypeVal amp;val )
{
boost::mutex::scoped_lock lock( m_atomicMutex );
m_typeVal = val;
return m_typeVal;
}
const TypeValamp; increment()
{
boost::mutex::scoped_lock lock( m_atomicMutex );
m_typeVal ;
return m_typeVal;
}
const TypeValamp; decrement()
{
boost::mutex::scoped_lock lock( m_atomicMutex );
m_typeVal-- ;
return m_typeVal;
}
const TypeValamp; increment(const TypeValamp; t)
{
boost::mutex::scoped_lock lock( m_atomicMutex );
m_typeVal =t ;
return m_typeVal;
}
const TypeValamp; decrement(const TypeValamp; t)
{
boost::mutex::scoped_lock lock( m_atomicMutex );
m_typeVal-=t ;
return m_typeVal;
}
private:
mutable boost::mutex m_atomicMutex;
TypeVal m_typeVal;
};
class CountDownLatch
{
public:
CountDownLatch( int count ): m_cdlCount( count )
{
}
CountDownLatch(const CountDownLatchamp; cpy)
{
boost::unique_lock<boost::mutex>::unique_lock(const_cast<boost::mutexamp;>(cpy.m_cdlMutex));
m_cdlCount = cpy.m_cdlCount;
}
const CountDownLatchamp; operator=(const CountDownLatchamp; other)
{
if (this == amp;other)
return *this;
boost::mutex::scoped_lock lock1(const_cast<boost::mutexamp;>(amp;m_cdlMutex < amp;other.m_cdlMutex ? m_cdlMutex : other.m_cdlMutex));
boost::mutex::scoped_lock lock2(const_cast<boost::mutexamp;>(amp;m_cdlMutex > amp;other.m_cdlMutex ? m_cdlMutex : other.m_cdlMutex));
m_cdlCount = other.m_cdlCount;
return *this;
}
virtual ~CountDownLatch()
{
}
void wait()
{
boost::mutex::scoped_lock lock( m_cdlMutex );
if( m_cdlCount.getCount() > 0 )
{
m_cdlCondition.wait( lock );
}
}
void wait( uint64_t timeoutMicros )
{
boost::mutex::scoped_lock lock( m_cdlMutex );
if( m_cdlCount.getCount() > 0 )
{
boost::posix_time::time_duration td = boost::posix_time::milliseconds( timeoutMicros );
m_cdlCondition.timed_wait( lock, td );
}
}
void countDown()
{
boost::mutex::scoped_lock lock( m_cdlMutex );
if( m_cdlCount.decrement() == 0 )
{
m_cdlCondition.notify_all();
}
}
int getCount()
{
return m_cdlCount.getCount();
}
private:
boost::mutex m_cdlMutex;
boost::condition_variable m_cdlCondition;
AtomicCounter< int > m_cdlCount;
};
Комментарии:
1. Код ошибочен. Возьмем, к примеру, метод AtomicCounter::getCount, вы возвращаете по ссылке элемент, который должен быть изменен другим потоком (иначе зачем использовать блокировки?) Вам нужно вернуть счетчик в этом методе путем копирования. На некоторые другие методы влияет тот же недостаток дизайна. Кроме того, почему вы используете (тратите ресурсы) приращения post, когда вам нужно предварительное увеличение?
2. К ВАШЕМУ СВЕДЕНИЮ класс защелки boost (экспериментальный, версия v1.67)
Ответ №1:
Для модульного тестирования вы можете попробовать стресс-тестирование. Например, для обратного отсчета создайте 25 тестовых потоков, которые одновременно вызывают CountDownLatch::countDown()
, 25 других потоков, которые одновременно вызывают CountDownLatch::getCount()
, и 25 других потоков, которые вызывают CountDownLatch::wait()
. Чтобы сделать процесс более одновременным, используйте барьер или переводите потоки в режим ожидания до того же абсолютного времени. Убедитесь, что все потоки завершаются должным образом (без взаимоблокировок), объединив их все. Убедитесь, что CountDownLatch::m_cdlCount
значение равно нулю.
Запускайте один и тот же тест много раз (в течение разумного промежутка времени).
Вы можете использовать ту же основную идею для AtomicCounter.
Вероятно, существуют другие методы тестирования многоглавого кода, но это тот, с которым я наиболее знаком.