[Jelle]

PHP Code:

if ( get_user_weapon(id) == CSW_GALIL) 
{
    //set galil model here
}

else if ( get_user_weapon(id) == CSW_DEAGLE )
{
    //set deagle model here
} 


[DarkGod]

Originally Posted by Jelle The way it is now it just checks if user has galil and does nothing, and if user has deagle it sets both deagle and galil model on the deagle.

That's actually wrong. Right now it does this:

if(get_user_weapon(index) == CSW_GALIL)
{
   if(get_user_weapon(index) == CSW_DEAGLE)
   {
      //Code
   }
}

Anyways, the snippet you provided afterwards is correct.

[Fr33m@n]

switch is the way to go here. I don't know why people never think to use it.

[DarkGod]

Originally Posted by Fr33m@n switch is the way to go here. I don't know why people never think to use it.

Never? Most people do use it. It's barely any advantage in efficiency anyways. It's good practice, sure, but you exaggerate.

[Fr33m@n]

i'm talking about here : Jelly Art Exploited and so on.
I obviously know that switch is used a lot. But here...

[Jelle]

Originally Posted by DarkGod switch_model(id) { if (!sh_is_active() || !is_user_alive(id) || !gHasBones[id] ) return if (get_user_weapon(id) == CSW_GALIL) if (get_user_weapon(id) == CSW_DEAGLE) { set_pev(id, pev_viewmodel2, gBonesGalil) set_pev(id, pev_weaponmodel2, gBonesGalil2) set_pev(id, pev_viewmodel2, gBonesDeagle) set_pev(id, pev_weaponmodel2, gBonesDeagle2) } } You can't have two weapons up at the same time. Use else if.

Originally Posted by DarkGod That's actually wrong. Right now it does this: Code: if(get_user_weapon(index) == CSW_GALIL) { if(get_user_weapon(index) == CSW_DEAGLE) { //Code } } Anyways, the snippet you provided afterwards is correct.

I can't see an opening brace after the check for the galil weapon, so if user has galil, nothing happens. Or is there an automatically opening brace when you do an if condition? Not really sure about that, since using a return on the next line works. Weird, never though of that.

Originally Posted by Fr33m@n i'm talking about here : Jelly Art Exploited and so on. I obviously know that switch is used a lot. But here...

MY NAME IS JELLE, NOT JELLY!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!!!!!!!!!!!

Also, I do use switch, but only when there is more than two if conditions.

[DarkGod]

Originally Posted by Jelle I can't see an opening brace after the check for the galil weapon, so if user has galil, nothing happens. Or is there an automatically opening brace when you do an if condition? Not really sure about that, since using a return on the next line works. Weird, never though of that.

You can do a function like this:

 function(index)
    if(something == true)
       DoThis()
    else if(something else == false)
       if(whatever == false)
          DoThat()
       else
          DoSomething()
   else
      DoWhatever()

indenting is a bit off, lol, I just did this in the browser.

[MuzzMikkel]

I used your method Jelle but the models still not working? I did like this:

PHP Code:

switch_model(id)
{
    if (!sh_is_active() || !is_user_alive(id) || !gHasBones[id] ) return
    
if ( get_user_weapon(id) == CSW_GALIL) 
{
        set_pev(id, pev_viewmodel2, gBonesGalil)
    set_pev(id, pev_weaponmodel2, gBonesGalil2)
}

else if ( get_user_weapon(id) == CSW_DEAGLE )
{
    set_pev(id, pev_viewmodel2, gBonesDeagle)
    set_pev(id, pev_weaponmodel2, gBonesDeagle2)
    }
} 


Can you fix the problem?

[MuzzMikkel]

Originally Posted by DarkGod That's actually wrong. Right now it does this: if(get_user_weapon(index) == CSW_GALIL) {    if(get_user_weapon(index) == CSW_DEAGLE)    {       //Code    } } Anyways, the snippet you provided afterwards is correct.

When i try your method it dosen't work either, the compiler say:

<115> : error 017: undefined symbol "index"
<117> : error 017: undefined symbol "index"

[Jelle]

Originally Posted by DarkGod You can do a function like this: Code: function(index) if(something == true) DoThis() else if(something else == false) if(whatever == false) DoThat() else DoSomething() else DoWhatever() indenting is a bit off, lol, I just did this in the browser.

I wasn't really aware of that, thank you.

Originally Posted by MuzzMikkel I used your method Jelle but the models still not working? I did like this: PHP Code: switch_model(id) {     if (!sh_is_active() || !is_user_alive(id) || !gHasBones[id] ) return      if ( get_user_weapon(id) == CSW_GALIL)  {         set_pev(id, pev_viewmodel2, gBonesGalil)     set_pev(id, pev_weaponmodel2, gBonesGalil2) } else if ( get_user_weapon(id) == CSW_DEAGLE ) {     set_pev(id, pev_viewmodel2, gBonesDeagle)     set_pev(id, pev_weaponmodel2, gBonesDeagle2)     } }  Can you fix the problem?

Post the whole code. Have you precached the weapon model correct?