Salve, scusate una info se possibile, ho un pezzettino di codice che a volte funziona a volte no, volevo chiedere se secondo voi il codice è corretto o si può fare in altri modi

Codice PHP:
if ($posizione == '1')
            {
            $guadagno=($valorep - $prezzo1);
            }
            if ($posizione == '2')
            {
            $guadagno=($prezzo1 - $valorep);
            }