[andreblue]

I am trying to use a switch statement to be able to check which bodypart was hit in csgo.
I tried below but it would return errors. I am new to source pawn/pawn thus am unsure exactly what to do even with the help of google. Can anyone shed some light on what im doing wrong or how to fix it? Thanks

new String:bodyPart[] = "";
switch(hit_area)
{
    case 1:   bodyPart[] = "Head";
    case 2:   bodyPart[] = "Chest";
    case 3:   bodyPart[] = "Stomach";
    case 4:   bodyPart[] = "Left Arm";
    case 5:   bodyPart[] = "Right Arm";
    case 6:   bodyPart[] = "Left Leg";
    case 7:   bodyPart[] = "Right Leg";
    default:  bodyPart[] = "Unknown";
}

[Phil25]

For declaring a String variable which you're going to assign a value right away you should use decl instead of new. It's better for optimization. Also, you don't have it equal to anything, simply put the buffer size in [].

For switches you need to use {} after the colon otherwise it won't work.

And, for assigning strings, you need to use strcopy, you can't just have it equal to something as simple as that. :p Within the strcopy you need to specify the buffer size, as well.

So it would be something like this:

PHP Code:

decl String:bodyPart[16];
switch(hit_area)
{
    case 1:{    strcopy(bodyPart, sizeof(bodyPart), "Head");}    //"sizeof(bodyPart)" is 16, ofc
    case 2:{    strcopy(bodyPart, sizeof(bodyPart), "Chest");}
    case 3:{    strcopy(bodyPart, sizeof(bodyPart), "Stomach");}
    case 4:{    strcopy(bodyPart, sizeof(bodyPart), "Left Arm");}
    case 5:{    strcopy(bodyPart, sizeof(bodyPart), "Right Arm");}
    case 6:{    strcopy(bodyPart, sizeof(bodyPart), "Left Leg");}
    case 7:{    strcopy(bodyPart, sizeof(bodyPart), "Right Leg");}
    default:{    strcopy(bodyPart, sizeof(bodyPart), "Unknown");}
} 


[ddhoward]

Originally Posted by Phil25 For declaring a String variable which you're going to assign a value right away you should use decl instead of new.

decl does not exist in the new syntax. We are moving away from decl as it is unsafe.

And, for assigning strings, you need to use strcopy

No, you don't. The assignment operator should work just fine here.

PHP Code:

char bodyPart[10]; 
switch(hit_area) 
{ 
    case 1:{    bodyPart = "Head"; }
    case 2:{    bodyPart = "Chest"; }
    case 3:{    bodyPart = "Stomach"; }
    case 4:{    bodyPart = "Left Arm"; }
    case 5:{    bodyPart = "Right Arm"; }
    case 6:{    bodyPart = "Left Leg"; }
    case 7:{    bodyPart = "Right Leg"; }
    default:{    bodyPart = "Unknown"; }
} 


[Phil25]

Originally Posted by ddhoward No, you don't. The assignment operator should work just fine here.

Having the assignment operator was making one of my old plugins not work. Maybe it can be used now, in the new syntax.

[ddhoward]

Originally Posted by Phil25 Having the assignment operator was making one of my old plugins not work. Maybe it can be used now, in the new syntax.

IIRC, it's worked for me since I started writing plugins back in 2013.