Why is this like this?

[Rugarus]

if (!UnitHasItemOfTypeBJ(u, 'I00Z') == false || !UnitHasItemOfTypeBJ(u, 'I00J') == false)  {

   }
   else
   {
   DisplayTextToPlayer(GetLocalPlayer(), 0, 0, "Yes");
    SetUnitState(u, UNIT_STATE_MANA, currMana);
   
   }
   
  }

So I added the else because for some reason regardless if I set false or true or != true, etc. It would always run even though I had the item, and when I didn't have the item it wouldn't run.

Before I had this

if (UnitHasItemOfTypeBJ(u, 'I00Z') != true|| UnitHasItemOfTypeBJ(u, 'I00J') != true) {

I had the item, and it would still run the code. And now I added the !unithasitem part, and it still runs it and I needed to add the else statement. Why is that? Am I reading this incorrectly? Sorry if this is confusing not sure how to explain it better.

 

[nedio95]

This does not look like JASS... what language are you using ?

You got way too many turnarounds in that comparison. It makes me confused what you are trying to achieve. Mind explaining ?

if you want it to fire if a unit has the item just do:

//This is how it would look like... yours look like C-kinda language.
if ( UnitHasItemOfTypeBJ(u, 'I00Z') || UnitHasItemOfTypeBJ(u, 'I00J') )
{
  //Unit has either items
}
else
{
  //Unit has neither items
}

//Jass should be like:
if ( UnitHasItemOfTypeBJ(u, 'I00Z') || UnitHasItemOfTypeBJ(u, 'I00J') ) //Shouldn't "||" be "or" instead ?
then
  //Unit has either item here
else
  //unit has no item here
endif

regards
-Ned

 

[Jampion]

Looks like zinc to me.
The boolean logic here is very complicated.
!A == false || !B== false
A == true || B == true
A || B
now with the else you turn it around
! (A || B)
!A && !B

it's all the same, just the last two are a lot more readable and more efficient, if the code is not optimized

 

[Rugarus]

Yeah it's zinc, but it's weird cause I had it like (UnitHasItemOfTypeBJ(u, 'I00Z') == true)

And if the unit didn't have the item it would run the statement, and if I set it to false, it would also run the statement even if I did or didnt have the item. And the only way it didn't when I added the else part.