Good vs bad engineering

Good:

if (already_reset) {
    say "sorry, you've already reset before";
    exit;
}
if (have_cart || weight > 0 ...) {
    say "i can't reset your skills if you have a falcon, cart, ...";
    exit;
}

Bad:

if (have_cart || weight > 0 ...) {
    say "i can't reset your skills if you have a falcon, cart, ...";
    exit;
}
if (already_reset) {
    say "sorry, you've already reset before. i hope you enjoyed "
        "putting everything into storage and npcing all your mercenary "
        "scrolls as much as you're going to enjoy spending the next ten "
        "minutes getting your cart back and trying to remember exactly "
        "which and how many of each item you had--all for NOTHING."
        "oh, and have fun doing this 50 more times!";
    exit;
}
This entry was posted in bugs, rants. Bookmark the permalink.

Leave a Reply

This site uses Akismet to reduce spam. Learn how your comment data is processed.