Warning 215: Expression has no effect
#1

PHP код:
SetVehicleStatus(vehicleid, status){
    new 
engine, lights, alarm, doors, bonnet, boot, objective;
    
GetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, boot, objective);
    switch(
status){
        case 
0:{
            (
engine == 0) ? SetVehicleParamsEx(vehicleid, VEHICLE_PARAMS_ON, lights, alarm, doors, bonnet, boot, objective) : SetVehicleParamsEx(vehicleid, VEHICLE_PARAMS_OFF, VEHICLE_PARAMS_OFF, alarm, doors, bonnet, boot, objective); // ERROR
        
}
        case 
1:{
            (
lights == 0) ? SetVehicleParamsEx(vehicleid, engine, VEHICLE_PARAMS_ON, alarm, doors, bonnet, boot, objective) : SetVehicleParamsEx(vehicleid, engine, VEHICLE_PARAMS_OFF, alarm, doors, bonnet, boot, objective); // ERROR
        
}
        case 
2:{
            (
bonnet == 0) ? SetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, VEHICLE_PARAMS_ON, boot, objective) : SetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, VEHICLE_PARAMS_OFF, boot, objective); // ERROR
        
}
        case 
3:{
            (
boot == 0) ? SetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, VEHICLE_PARAMS_ON, objective) : SetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, VEHICLE_PARAMS_OFF, objective); // ERROR
        
}
        case 
4:{
            (
doors == 0) ? SetVehicleParamsEx(vehicleid, engine, lights, alarm, VEHICLE_PARAMS_ON, bonnet, boot, objective) : SetVehicleParamsEx(vehicleid, engine, VEHICLE_PARAMS_OFF, alarm, doors, bonnet, boot, objective); // ERROR
        
}
    }
    return 
1;
} 
Help me please!
Reply
#2

if statement before the conditions.
Reply
#3

Can I use "?" and ":" condition? It is also a condition.
Reply
#4

Try this
PHP код:
SetVehicleStatus(vehicleid, status){ 
    new 
engine, lights, alarm, doors, bonnet, boot, objective; 
    
GetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, boot, objective); 
    switch(
status){ 
        case 
0:{ 
            (
engine = 0) ? SetVehicleParamsEx(vehicleid, VEHICLE_PARAMS_ON, lights, alarm, doors, bonnet, boot, objective) : SetVehicleParamsEx(vehicleid, VEHICLE_PARAMS_OFF, VEHICLE_PARAMS_OFF, alarm, doors, bonnet, boot, objective); // ERROR 
        
} 
        case 
1:{ 
            (
lights = 0) ? SetVehicleParamsEx(vehicleid, engine, VEHICLE_PARAMS_ON, alarm, doors, bonnet, boot, objective) : SetVehicleParamsEx(vehicleid, engine, VEHICLE_PARAMS_OFF, alarm, doors, bonnet, boot, objective); // ERROR 
        
} 
        case 
2:{ 
            (
bonnet = 0) ? SetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, VEHICLE_PARAMS_ON, boot, objective) : SetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, VEHICLE_PARAMS_OFF, boot, objective); // ERROR 
        
} 
        case 
3:{ 
            (
boot = 0) ? SetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, VEHICLE_PARAMS_ON, objective) : SetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, VEHICLE_PARAMS_OFF, objective); // ERROR 
        
} 
        case 
4:{ 
            (
doors = 0) ? SetVehicleParamsEx(vehicleid, engine, lights, alarm, VEHICLE_PARAMS_ON, bonnet, boot, objective) : SetVehicleParamsEx(vehicleid, engine, VEHICLE_PARAMS_OFF, alarm, doors, bonnet, boot, objective); // ERROR 
        
} 
    } 
    return 
1; 
} 
Reply
#5

https://sampforum.blast.hk/showthread.php?tid=216730 read about the ternary operator. Also I think it's better to use the if-else statement here.
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)