Как решит проблему с курсом валют?

Question

[Aset]

Ребята помогите с кодом. Есть код который нужно вытащать актуальный курс.

В примере должен быть 76.85 тоесть USD > RUB. Но возврашает. 75.65 buy.

Как можно лучше переписать?

<?php
    
    $currency = '{
    "currency": [
            {"from":"RUB","to":"EUR","operationType":"sell","value":"92.85"},
            {"from":"RUB","to":"USD","operationType":"sell","value":"76.85"},
            {"from":"USD","to":"EUR","operationType":"sell","value":"1.1802"},
            {"from":"RUB","to":"EUR","operationType":"buy","value":"90.35"},
            {"from":"RUB","to":"USD","operationType":"buy","value":"75.65"},
            {"from":"USD","to":"EUR","operationType":"buy","value":"0.7975"}
            
        ]}';

    $rates = json_decode($currency,true);
    
    $directRate = null;
    $reversedRate = null;
    
    $amountCurrency = "USD"; // USD SELL "76.85"
    $targetCurrency = "RUB";
    foreach($rates['currency'] as $rate) {
        
    if($rate['from'] === $amountCurrency && $rate['to'] ===  $targetCurrency) {
        $directRate = $rate;
    }

    if($rate['from'] === $targetCurrency && $rate['to'] ===  $amountCurrency) {
        $reversedRate = $rate;
    }
            
}

    if ($directRate) { // На эту строку не обращать внимания
        if ($directRate['operationType'] == 'buy') {
            $amountConverted = $directRate['value'];
        } elseif ($directRate['operationType'] == 'sell') {
            $amountConverted = $directRate['value'];
        }
    }

    if ($reversedRate) { // На эту строку не обращать внимания
        if ($reversedRate['operationType'] == 'buy') {
            $amountConverted = $reversedRate['value'];
        } elseif ($directRate['operationType'] == 'sell') {
            $amountConverted = $reversedRate['value'];
        }
    }
    
    var_dump($amountConverted);

Answer 1

[Rsa97]

foreach($rates['currency'] as $rate) {
    if($rate['from'] === $amountCurrency && $rate['to'] ===  $targetCurrency) {
        $directRate = $rate;
    }
    if($rate['from'] === $targetCurrency && $rate['to'] ===  $amountCurrency) {
        $reversedRate = $rate;
    }
}

Поскольку строки с $rate['from'] === "USD" и $rate['to'] === "RUB" в массиве нету, то $directRate останется равной null.
Поскольку в массиве есть две строки с $rate['from'] === "RUB" и $rate['to'] === "USD", то в $reversedRate будет записана последняя из этих строк, {"from":"RUB","to":"USD","operationType":"buy","value":"75.65"}.

if ($directRate) {
    ....
}

Этот блок не выполнится, так как в $directRate записан null.

if ($reversedRate) { // На эту строку не обращать внимания
    if ($reversedRate['operationType'] == 'buy') {
        $amountConverted = $reversedRate['value'];
    } elseif ($directRate['operationType'] == 'sell') {
        $amountConverted = $reversedRate['value'];
    }
}

Здесь сработает ветка с $reversedRate['operationType'] == 'buy' и в $amountConverted запишется "75.65".
Всё работает в точности так, как вы написали в своей программе.
Для того, чтобы что-то переписать, надо знать, что именно вы рассчитываете получить при таких входных данных и почему.
Предположу, что в случае совпадения пары валют вам нужна ветка buy, а в случае обратного их расположения - ветка sell. Тогда будет так:

$conversionRate = null;
foreach($rates['currency'] as $rate) {
    if (($rate['operationType'] === 'buy' &&
         $rate['from'] === $amountCurrency &&
         $rate['to'] ===  $targetCurrency) ||
        ($rate['operationType'] === 'sell' &&
         $rate['from'] === $targetCurrency &&
         $rate['to'] ===  $amountCurrency)
    ) {
        $conversionRate = $rate['value'];
    }
}
var_dump($conversionRate);

Comments

[Aset]

Проблема в том что из фронтенде не приходит информация что это sell или buy.

Как это определить это не понятно.

[Rsa97]

Aset, Видимо тем, какая валюта указана в $amountCurrency, а какая в $targetCurrency. Если нет, то без знания $operationType задача не решается.