Pastable version of What is bad in this code?

<pre class='prettyprint' language='php'>&lt;?php
    public function get_usd_value($date)
    {
        $allow_currency = array('USD', 'EUR', 'GBP');
        $link = &quot;http://www.cbr.ru/scripts/XML_daily.asp?date_req=$date&quot;;
        $text = @file_get_contents($link);

$xml = new SimpleXMLElement($text);

foreach ($xml-&gt;Valute as $curr)
        {
            if(in_array($curr-&gt;CharCode, $allow_currency))
            {
                $currency[] = UTF8::str_ireplace(',', '.', (string)$curr-&gt;Value);
            }
        }

$USD = $currency[0];
		$EUR = round($currency[2]/$USD, 5);
		$GBP = round($currency[2]/$USD, 5);
		
	
		$rate_array = array(&quot;USD&quot; =&gt; $USD/$USD, &quot;EUR&quot; =&gt; $EUR, &quot;GBP&quot; =&gt; $GBP);
		return $rate_array;
    }
?&gt;</pre>
<a href="http://www.refactormycode.com/codes/1403-what-is-bad-in-this-code" style="color:#fff" title="As seen on RefactorMyCode.com"><img alt="Small_logo" src="http://www.refactormycode.com/images/small_logo.gif" style="border:0" /></a>

Refactor :my => 'code'

Recent

Popular

Pastable version of What is bad in this code?