[FIXED] How to display the entire list of currencies?

Issue

I need to output the exchange rate given by the ECB API. But the output shows an error

"TypeError: string indices must be integers"

How to fix this error?

import requests, config
from bs4 import BeautifulSoup

r = requests.get(config.ecb).text
soup = BeautifulSoup(r, "lxml")

course = soup.findAll("cube")
for i in course:
    for x in i("cube"):
        for y in x:
            print(y['currency'], y['rate'])

Solution

You have too many for-loops

for i in course:
    print(i['currency'], i['rate'])

But this need also to search <cube> with attribute currency

course = soup.findAll("cube", currency=True)

course = soup.findAll("cube", {"currenc": True})

or you would have to check if item has attribute currency

for i in course:
    if 'currency' in i.attrs:
        print(i['currency'], i['rate'])

---

Full code:

import requests
from bs4 import BeautifulSoup

url = 'https://www.ecb.europa.eu/stats/eurofxref/eurofxref-daily.xml?c892a2e0fae19504ef05028330310886'

response = requests.get(url)
soup = BeautifulSoup(response.text, "lxml")

course = soup.find_all("cube", currency=True)

for i in course:
    #print(i)
    print(i['currency'], i['rate'])

Answered By - furas