Ubiquituous language tradeoff

gistfile1.txt

Which do you prefer and why?
Alternative ideas?

        public Tickettransaction Buy_ticket(Attendee attendee)
        {
            if (tickets_available)
                return Issue_Ticket(attendee);
            else
                return Not_Issued.Because("Box office should not allow to buy tickets, when there are none available.");
        }


        public Tickettransaction Buy_ticket(Attendee attendee)
        {
            return 
                tickets_available 
                    ? Issue_Ticket(attendee)
                    : Not_Issued.Because("Box office should not allow to buy tickets, when there are none available.");
        }

Very clearly a binary decision. Both branches are very simple. For these reasons, ternary operator is perfect.

I prefer early returns for these things

public Tickettransaction Buy_ticket(Attendee attendee) {
    if (no_tickets_available) {
        return Not_Issued.Because("Box office should not allow to buy tickets, when there are none available.");
    }

    return Issue_Ticket(attendee);
}

...but it's probably a matter of taste

@bwaidelich I would be happy with Guard Clause here, too, but I tend towards ternary operator for clearly a simple binary decision. Similarly, I see it has merely a preference.