リスト内の文字列にあるM(ミリオン）やB（ビリオン）を数字にしたい。

Question

前提・実現したいこと

リスト内にM（ミリオン）やB（ビリオン）が入っている数字（文字列）に対して、数字に変更する関数を定義したいです。

下記のようなリストがあります。
l = ['4.17B', '285M', '1.72B', '3.91B', '(388.679)']

このリストに対して、欲しい結果は下記のようなものです。
Mはミリオン,000000(0が6つ)に入れ替え、Bは000000000(0が9つ)に入れ替えたいです。
[4170000000, 285000000, 1720000000, 3910000000, -388679]

発生している問題・エラーメッセージ

ところが、現在mapとLamdaを組みあわせたものをだと、
['417000000000', '285000000', '172000000000', '391000000000', '-388679']
となってしまい、小数点をうまく処理できていません。
Decimalを使ってみたりしたのですが、うまくいきませんでした。
お知恵お借りできれば幸いです。
当方プログラミング初心者です。。

エラーメッセージ

該当のソースコード

l = ['4.17B', '285M', '1.72B', '3.91B', '(388.679)']

def unit_changer(ll=None) :

    if ll is None:
        ll =[]
    LSV = 0
    for LSV in range(5):
        ll = list(map(lambda d: d.replace("M", "000000"), ll))
        ll = list(map(lambda d: d.replace("B", "000000000"), ll))
        ll = list(map(lambda d: d.replace("(", "-"), ll))
        ll = list(map(lambda d: d.replace(")", ""), ll))
        ll = list(map(lambda d: d.replace(".", ""), ll))
    return ll


print(unit_changer(l))

試したこと

色々と調べて、Decimalなども使ってみたのですが、うまくいきませんでした。

補足情報（FW/ツールのバージョンなど）

Python3.7、Pycharmを使っています。

Comments

[hayataka2049]

(388.679)はなぜ-388679になるべきなのでしょうか？　特に説明がないようですが。

[Snow_Snow]

ご質問ありがとうございます。
説明不足ですいません。
会計情報を取得しており、そこの数字が()の場合、マイナスだからでございます。

Answer 1

末尾がB、末尾がM、()で囲まれているの3つの場合分けをすればいいと思います。

l = ["4.17B", "285M", "1.72B", "3.91B", "(388.679)"]


def unit_changer(s):
    if s[-1] == "M":
        s = float(s[:-1]) * 10 ** 6
    elif s[-1] == "B":
        s = float(s[:-1]) * 10 ** 9
    elif s[0] == "(" and s[-1] == ")":
        s = float(s[1:-1])

    if s.is_integer():
        s = int(s)  # 整数なら float を int にする

    return s


l2 = list(map(unit_changer, l))
print(l2)  # [4170000000, 285000000, 1720000000, 3910000000, 388.679]

Comments

[Snow_Snow]

ありがとうございました！
条件分けをすることで、問題解決することができました。

Answer 2

Python標準ライブラリの字句解析器を使って実装。
こういった、ちょっとしたドメイン特化言語(DSL)作るのに役立ちます。

import io
import tokenize
from token import OP, NAME, NUMBER
from decimal import Decimal

UNIT_TABLE = {
    "M": 10**6,
    "B": 10**9,
}

def str_to_decimal(token, result=1):
    for info in tokenize.generate_tokens(io.StringIO(token).readline):
        if info.type == NUMBER:
            result *= Decimal(info.string)
        elif info.type == NAME: # Unit
            result *= UNIT_TABLE[info.string]
        elif info.type == OP and info.string == "(": # Nagative
            result *= -1
    return result

if __name__ == '__main__':
    xs = ['4.17B', '285M', '1.72B', '3.91B', '(388.679)']

    for x in map(str_to_decimal, xs):
        print(x)

出力

4170000000.00
285000000
1720000000.00
3910000000.00
-388.679

※ 追記: エラー処理は省いてます、"((" がプラスになる等。
正規表現で 数値・記号 に分割した方がよかったかもしれない。

Comments

[Snow_Snow]

ありがとうございます。
ドメイン特化言語を作るなど、私はまだまだ知らないことでしたので、ご教示いただきありがとうございます！
今後のプログラミングに活かしてまいります。