[Jstylezzz]

Technically this if statement will trigger when your faction is not 1 and when it's not 4. The thing is, when faction is equal to 1, it is not equal to 4, which causes the if statement to trigger. Same the other way around. If your faction is 4, it is not 1, which will cause the first check to trigger the if statement.

I'm guessing this is part of a command? Which faction IDs should be able to use it? If both ID 1 and ID 4 are allowed to use it, you can change || to &&.