What's a better way of typing this?

[bartbes]

Code: Select all

variable = math.max(variable, 0)

or, in the style of the 'ternary' operator

Code: Select all

variable = variable < 0 and 0 or variable

[Tesselode]

My goodness, why do you guys have to come up with such short and confusing ways to do things? How about things that make sense to the average (code) reader?

[Taehl]

Short circuit evaluation does make sense (glorious sense, at that), as long as your mind isn't chained down by restrictive languages like C (before I get flamed by C-lovers: Yes, I know C can do it, but it doesn't promote the right /mindset/ to use such things).

[vrld]

My goodness, why do you guys have to come up with such short and confusing ways to do things? How about things that make sense to the average (code) reader?

provided the expressions (and condition) are not too complicated, i find

Code: Select all

var = condition and expression_one or expression_two

easier to read and understand than

Code: Select all

if condition then
    var = expression_one
else
    var = expression_two
end

This way I can see whats happening reading only one line instead of three.

Also: because we can.

[Robin]

My goodness, why do you guys have to come up with such short and confusing ways to do things? How about things that make sense to the average (code) reader?

How is

Code: Select all

if someState then
    someState = false
else
    someState = true
end

easier to understand than

Code: Select all

someState = not someState

?

If you prefer the former to the latter, you might need to read The Daily WTF more often.

[Tesselode]

I'm not talking about that one.

[Robin]

What, bartbes' one?

[Tesselode]

Code: Select all

variable = math.max(variable, 0)

or, in the style of the 'ternary' operator

Code: Select all

variable = variable < 0 and 0 or variable

This one.

[knorke]

i was superproud when i invented

Code: Select all

players[i].is_ai = players[i].is_active and not players[i].is_ai
players[i].is_active = players[i].is_ai or not players[i].is_active

to toggle a player being a human player, ai player or not active when a button is clicked.

[thelinx]

Assuming I understand your setup correctly, this should work:

Code: Select all

players[i].is_ai,players[i].is_active = players[i].is_active,players[i].is_ai