On 04/06/2012 13:52, AK wrote:
> Użytkownik "Maciej Sobczak" <s...@g...com> napisał:
>
>> Ten kod może być zarówno dobry jak i niedobry, zależnie od kontekstu.
>
> Ten kod (A) jest zawsze zly.

tylko przy założeniu że "rollback" jest w sekcji catch. A mógłby być w
destruktorze, np:


class Connection
{
friend class Transaction;

void begin();
void rollback();
void commit();
void execute(const Statement&);
// . . .
};

class Transaction
{
Connection& conn_;
bool active_;

public
Transaction(Connection& c) conn_(c), active_(false)
{}

~Transaction()
{
if (active_)
conn_.rollback();
}

void commit()
{
if (active_)
{
conn_.commit();
active_ = false;
}
}

void execute(const Statement& s)
{
if (!active_)
{
conn_.begin();
active_ = true;
}
conn_.execute(s);
}
};



try
{
Transaction t(c);
t.execute(s1);
t.execute(s2);
t.commit();
t.execute(s3);
t.commit();
}
catch (Error e)
{}