Python3 引数に応じてJsonデータの中身を書き換えたい。

Question

#実現したいこと
引数に応じてJsonデータの中身を書き換えたいです。

以下のコマンド実行。
python3 /etc/main.py -g "TestGraph" -i "192.168.251.10" "192.168.251.11" "192.168.251.12"
①[/etc/main.py]を実行しその中で②[/etc/module/definition.py]を呼び出す仕様になっています。
「- i」の引数に応じて[/etc/module/definition.py]の中の「#」の箇所を自動で増やす仕様にしたいです。
※今は、例として3つのIPの引数の為、「'name': ip*」が3つあります。
1つ5つ6つとIPの引数となったときに「'name': ip*」が1つ5つ6つ自動で増やたり、減ったりするやり方ありますでしょうか？
引数に応じてJsonデータとして別ファイルに専用で書き出し、json.dumps時にそのファイルを読み込むしか実現できないでしょうか？
お力沿いお願いいたします。

graphDef = {
        'graphs': {
            graphID: {
                'unit': 'integer',
                'metrics': [
                    {
                       'name': ip1        #引数に応じて増やしたい
                    }, {
                       'name': ip2       #
                    }, {
                       'name': ip3        #
                    }
                ]
            }
        }
    }

お力沿いお願いいたします。

①[/etc/main.py]

import os
import sys
import argparse
import module.definition

def main():
    # 引数の処理
    parser = argparse.ArgumentParser(description='プラグイン')
    parser.add_argument('-g', '--graphID', help='グラフIDを設定')
    parser.add_argument('-i', '--ipaddress', nargs='*', help='複数の場合はスペースで区切る')

    args = parser.parse_args()
    
    module.definition._Definition(args.graphID,args.ipaddress)

if __name__ == "__main__":
    main()

②[/etc/module/definition.py]

import os
import sys
import json

def Definition(graphID,ipaddress):

    graphID = ''.join(graphID)
    displayName = ''.join(displayName)

    ip1 = ipaddress[0]
    ip1 = ''.join(ip1)
    ip2 = ipaddress[1]
    ip2 = ''.join(ip2)
    ip3 = ipaddress[2]
    ip3 = ''.join(ip3)

    graphDef = {
        'graphs': {
            graphID: {
                'unit': 'integer',
                'metrics': [
                    {
                       'name': ip1        #引数に応じて増やしたい
                    }, {
                       'name': ip2       #
                    }, {
                       'name': ip3        #
                    }
                ]
            }
        }
    }
    json.dumps(graphDef, ensure_ascii=False)
    sys.exit(0)

if __name__ == "__main__":
    Definition(graphID,ipaddress)

Answer 1

こんなふうにできそうに見えます。

'metrics': [
    {'name': ip} for ip in ipaddress
]

---

その他、気になること

• ''.join(...) については必要無いのでは。どういう意図で用いているのでしょうか？
• sys.exit(0) は本当に必要でしょうか。
• definition.py がメインモジュールとして起動されることはあるのでしょうか？

Comments

[ITOMO5963]

ご回答ありがとうございます。
.joinは不要でした。
sys.exit()も不要でした。
ご指摘ありがとうございます。

definition.pyはメインモジュールとして起動される想定ではありません。
if __name__ == "__main__":も不要でしょうか？

[LouiS0616]

> if __name__ == "__main__":も不要でしょうか？

はい、逆に有ると『このモジュールはメインモジュールとして呼ばれることがある』と第三者が読みとってしまう場合があるので、取っ払った方が良いと思います。
なお私はそのようなとき assert __name__ != '__main__' と書く過激派です。

[ITOMO5963]

ありがとうございます。
追加で伺いたいことがあります。
Metricの中に{name}が{label}あり、{name}には「.」ドットが使えない仕様のため、
’’.joinで文字列→replaceで「.」ドットを「_」に置換しています。
この場合、同じように引数に応じて対応できないでしょうか？
お力沿いおねがいいたします。
re_ipaddress = ''.join(ipaddress)
re_ip = re_ipaddress.replace('.', '_')

'metrics': [
{'name': re_ip,'label': ip} for ip in ipaddress
]

[LouiS0616]

{'name': ip.replace('.', '_'),'label': ip}

[ITOMO5963]

ありがとうございます
最高の形で実現できました。