How to make an if/or in 1 line

skite2001 · Post by **skite2001** » Wed Jun 30, 2010 4:51 pm

Hello, first my script:

if (s == "wizmet1_2")
		{
			r = rint(random() * 3);			
			if (r == 0)
				sound (e, CHAN_AUTO, "walk/metal1.wav", 0.5, ATTN_NORM);
			else if (r == 1)
				sound (e, CHAN_AUTO, "walk/metal2.wav", 0.5, ATTN_NORM);
			else if (r == 2)
				sound (e, CHAN_AUTO, "walk/metal3.wav", 0.5, ATTN_NORM);
			else
				sound (e, CHAN_AUTO, "walk/metal4.wav", 0.5, ATTN_NORM);
		}
		if (s == "ecop1_6")
		{
			r = rint(random() * 3);			
			if (r == 0)
				sound (e, CHAN_AUTO, "walk/metal1.wav", 0.5, ATTN_NORM);
			else if (r == 1)
				sound (e, CHAN_AUTO, "walk/metal2.wav", 0.5, ATTN_NORM);
			else if (r == 2)
				sound (e, CHAN_AUTO, "walk/metal3.wav", 0.5, ATTN_NORM);
			else
				sound (e, CHAN_AUTO, "walk/metal4.wav", 0.5, ATTN_NORM);
		}

and now my question...how can i add the 2nd unneeded bunch of script to the 1st one, like:

if (s == "wizmet1_2") or (s == "ecop1_6")

?

Spike · Post by **Spike** » Wed Jun 30, 2010 4:55 pm

if (a==c || b==d)

|| is logical or.
| is binary or.

Mexicouger · Post by **Mexicouger** » Wed Jun 30, 2010 4:58 pm

Code: Select all

if (s == "wizmet1_2" | s == "ecop1_6")
      {
         r = rint(random() * 3);         
         if (r == 0)
            sound (e, CHAN_AUTO, "walk/metal1.wav", 0.5, ATTN_NORM);
         else if (r == 1)
            sound (e, CHAN_AUTO, "walk/metal2.wav", 0.5, ATTN_NORM);
         else if (r == 2)
            sound (e, CHAN_AUTO, "walk/metal3.wav", 0.5, ATTN_NORM);
         else
            sound (e, CHAN_AUTO, "walk/metal4.wav", 0.5, ATTN_NORM);
      }

skite2001 · Post by **skite2001** » Wed Jun 30, 2010 8:31 pm

Mexicouger wrote:

Code: Select all

if (s == "wizmet1_2" | s == "ecop1_6")
      {
         r = rint(random() * 3);         
         if (r == 0)
            sound (e, CHAN_AUTO, "walk/metal1.wav", 0.5, ATTN_NORM);
         else if (r == 1)
            sound (e, CHAN_AUTO, "walk/metal2.wav", 0.5, ATTN_NORM);
         else if (r == 2)
            sound (e, CHAN_AUTO, "walk/metal3.wav", 0.5, ATTN_NORM);
         else
            sound (e, CHAN_AUTO, "walk/metal4.wav", 0.5, ATTN_NORM);
      }

sry mexi, doesn't work!
if i try to compile the files, i get the error:
"type mismatch for | (string and string)"

@Spike
sry, i want that "s==" is always "s=="! means i only want different textures be the same string

changed to:

Code: Select all

if (s == "wizmet1_2" || s == "ecop1_6")

now it works^^
thx for the help

mankrip · Post by **mankrip** » Wed Jun 30, 2010 8:34 pm

Because you must use the logical 'or', not the binary 'or'.

Mexicouger · Post by **Mexicouger** » Wed Jun 30, 2010 9:37 pm

So close

Anyways, When do you use those Logics? I don't understand what the Difference is.

Spike · Post by **Spike** » Wed Jun 30, 2010 10:26 pm

(1 || 2) == 1
(1 | 2) == 3

is the difference. :P

More seriously though..
(string | float) == syntax error
(string || float) == compares each side against 0/empty string.

They also have a different order of precidence, eg:
if (s == "wizmet1_2" | s == "ecop1_6")
is interpreted as:
if (s == ("wizmet1_2" | s) == "ecop1_6")
and then you're doing ((string == float) == string) which is two syntax errors.