Автоматический сброс флага на выходе из функции

Автор: Guardian

Иногда требуется внутри функции выставить флаг, а на выходе из функции сбросить его.

Например, следующим образом:

void SomeClass::SomeFunction(void)
{
    m_flag = true;
    ... // тело функции
    m_flag = false;
}

Однако дело осложняется тем, что у функции может быть несколько точек выхода, в том числе неявных. Лучше доверить сброс флага проверенным средствам. Например, так:

void flag_reset(bool* flag) { assert(flag); *flag = false; }

и в теле функции:

boost::shared_ptr<void> flag_reset(&m_flag, &flag_reset);

Теперь не требуется следить за тем, везде ли на выходе из функции сброшен флаг.

Разумеется, этот вариант не единственный.

#boost, #флаг

22 сентября 2009

Комментарии [19]

Страницы: 1 2 Следующая »

Pushkoff
Удалён

#1
16:58, 13 окт 2009

template<class T>
class AutoReset
{
  T& m_variable;
  T m_oldvalue;
public:
  AutoReset( T& var, const T& val ) : m_variable(var)
  {
    m_oldvalue = m_variable;
    m_variable = val;
  }

  ~AutoReset()
  {
    m_variable = m_oldvalue;
  }
}

по моему так более очевидно и удобно... ( потратил 7 минут, не проверял )

Ghost2
Постоялец

#2
17:52, 13 окт 2009

Ну и подсказки пошли...

X512
Постоялец

#3
18:02, 13 окт 2009

Смысл? Писать одну строку в начале в место конца?

Ghost2
> Ну и подсказки пошли...
Ещё это посмотри :)

my.name
Постоялец

#4
18:16, 13 окт 2009

кстати, поэтому и рекомендуют умные дядьки, из функции иметь один выход в конце, а не ретурнить по всякому поводу или без.

Ghost2
Постоялец

#5
18:24, 13 окт 2009

Я к тому, что использование автоматических переменных это как раз то, что можно было ожидать от подсказки. Но:
1. Об этих самых переменных тут ни слова.
2. Причем тут boost::shared_ptr?

my.name
Постоялец

#6
18:28, 13 окт 2009

1. они внутри щаред птр
2. поэтому он и юзается, работа перекладывается на него

Ghost2
Постоялец

#7
18:33, 13 окт 2009

>1. они внутри щаред птр
Видимо аудитория, на которую эти подсказки рассчитаны, должна до этого догадаться.

Pushkoff
Удалён

#8
18:35, 13 окт 2009

my.name
Это удобно в отдельных блоках, один раз объявил и забыл (типа while, for)...

hellokote
Пользователь

#9
19:45, 13 окт 2009

Госпади, зачем boost?

void func()
{
  struct A
  { 
    A() {flag = true;}
    ~A() {flag = false;}
  } a;
...
}

JokerR
Постоялец

#10
21:00, 13 окт 2009

hellokote
> Госпади, зачем boost?
затем киса, что наш соотечественник зафигачил scope_exit,
http://www.boost.org/doc/libs/1_40_0/libs/scope_exit/doc/html/index.html

велосипедостроителям пламенный привет.

du_hast
Удалён

#11
21:04, 13 окт 2009

За это можно стать участником? Может, меня тоже впишут?

    struct operation_t
    {
        operation_t(bool we = false)
        {
            assert(SSP::mutex0.IsLocked());
            
            asserts_idle();
            
            WE(we);
            CS(false);

[...]
            
            asserts_active();
        }
        
        ~operation_t(void)
        {
            asserts_active();
            
[...]
            
            CS(true);
            WE(false);

            asserts_idle();
        }
    };

gexogen
Постоялец

#12
22:14, 13 окт 2009

Сейчас пришло в голову.
Красивые крит секции :)

class AutoScopedLock
{
protected:
  CRITICAL_SECTION* m_cs;

public:
  AutoScopedLock(CRITICAL_SECTION* cs)
    : m_cs(cs)
  {}
  ~AutoScopedLock()
  {
    LeaveCriticalSection(m_cs);
  }

  operator bool() const
  {
    EnterCriticalSection(m_cs);
    return true;
  }
};
#define ScopedLock(_cs_)    if( AutoScopedLock _scoped_lock = _cs_ )

// использование
ScopedLock(criticalSection)
{
   // тут делать операции под защитой крит секции
}

и раз уж тут if, то легко добавить Try.

class AutoScopedLock
{
protected:
  CRITICAL_SECTION* m_cs;
            bool m_acquired;

public:
  AutoScopedLock(CRITICAL_SECTION* cs)
    : m_cs(cs), m_acquired(false)
  {}
  ~AutoScopedLock()
  {
    if( m_acquired )
      LeaveCriticalSection(m_cs);
  }

  operator bool() const
  {
    return (m_acquired = (TryEnterCriticalSection(m_cs) != 0));
  }
};
#define TryScopedLock(_cs_)    if( AutoScopedLock _scoped_lock = _cs_ )

// использование
TryScopedLock(criticalSection)
{
   // смогли войти в крит секцию, делаем свои дела тут
}

хотя вариант с Try возможно стоит делать не TryScopedLock(criticalSection), а как нибудь типа if( TryLock(criticalSection) ) для наглядности.
где TryLock:
#define TryLock(_cs_) AutoScopedLock _scoped_lock = _cs_

Pushkoff
Удалён

#13
23:53, 15 окт 2009

gexogen
как бы это уже давно многие используют, например попкап...

gexogen
Постоялец

#14
9:34, 16 окт 2009

Pushkoff
да нет, нигде такого не видел и в попкапе такого нет. Там есть AutoCrit, но разница в том, что его надо объявлять как переменную, внутри блока, а моя конструкция - больше похожа на блоки if\for\while или using(C#), логически более понятна и наглядна. Это не замена таким классам, как AutoCrit, а дополнение, ведь часто нужно ограничить не всю область внутри функции, а только маленький блок, в таких случаях она и пригодится.

Страницы: 1 2 Следующая »

Тема в архиве.