SVGファイルの不要データを除去したいのですが…

Question

実現したいこと、全体フロー

プログラミングを初めて２週間ほどの初心者です。
自分なりにコードを書いてみましたが、上手くいきません。
内容は「SVGデータを読み込み、不要な開始タグとタグ内部の文字列、終了タグを除去」するというものです。

全体的なフローとしては
0. ファイルを選択して、ファイル名を取得する(def main():)
0. リストに記述されたタグ名で、取得ファイルから不要データを除去していく(def change():)
0. リスト内に既出でないタグがあれば処理を続行（データを除去）し、該当しない場合は新規ファイル名で保存

といった具合に動作すればと、思っております。

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

File "svg_changer.py", line 18
with open(new_name, "a")
^
SyntaxError: invalid syntax

構文エラー？？
flake8 py__でチェックしましたが、該当する点はありませんでした。

該当のソースコード

#coding utf-8

import re, os
import tkinter.filedialog as tkfd

#不要なタグ名をリストに格納
unnecessary_tags = ["title"]

#引数にファイル名、不要なタグと中身を空白に変更し保存
def change(image):
    with open(image. "r", encoding="utf-8") as f:
        for row in f:
            for tag in unnecessary_tags:
                if tag in unnecessary_tags:
                    delete_tag = re.sub(r"<{}>.*</{}>".format(tag), "", row)
                else:
                    new_file_name = "new_" + image
                    with open(new_file_name, "a")
                    break

#ファイルを選択し、ファイル名を取得する
def main():
    target_file = tkfd.askopenfilename()
    filename = os.path.dirname(target_file)
    change(filename)

if __name__="__main__":
    main()

疑問点

現時点で、３点ほど疑問があります。

• ファイルをtkfd.askopenfile()で開いた後に、with open()で再度開くのは正しいのか
• タグ内の要素が既出かどうかを判定するif文は正しいのか
• 正規表現に.format()を用いて変数を当てることができるのか

自身で、多くの文献を調べましたが、納得いく答えを見るけることができませんでした。
知識不足がお恥ずかしいですが、改善のヒントを教えていただきたいです。
また、不明な点、必要な追記情報などありましたら、お申し付けください。

Answer 1

シンタックスエラーについて、18行目の最後にコロンを追加、19行目のbreakにインデントを追加する必要があります。

Comments

[kusao]

ご指摘いただきましてありがとうございます。
dkato0077さんの言われるように構文を修正しました所、
______________________________
File "svg_changer.py", line 11
with open(image. "r", encoding="utf-8") as f:
^
SyntaxError: invalid syntax
______________________________

次はLine11に対して、エラーが発生してしまいました。
うーん。一体何がいけないのでしょうか。

[退会済みユーザー]

11行目のimageのあとの.を,に置き換えて見てください。

[kusao]

初歩的なミスをしていました。
if__name__=="__main__"と共に修正しました。お恥ずかしい限りです。
現状、画像の読み込みのプログラムまで上手く動くようになりました。
しかし、また新たなエラーが…何度も申し訳ないです。
______________________________
Traceback (most recent call last):
File "svg_changer.py", line 28, in <module>
main()
File "svg_changer.py", line 25, in main
change(filename)
File "svg_changer.py", line 11, in change
with open(image, "r", encoding="utf-8") as f:
IsADirectoryError: [Errno 21] Is a directory:
______________________________

ディレクトリエラーというのは初めてみました。
調べてみると「組み込み例外」というものがヒットしましたが、
何のことなのかさっぱりです。

[退会済みユーザー]

11行目、openの引数にはファイル名の文字列をいれなければならないのですが、それがディクトリを指す文字列が与えられたためにエラーがでてますね。imagの中身をprintなどで確認してみてください。

[kusao]

image_file = tkfd.askopenfilename()
image_name = os.path.basename(image_file)

print(image_name)
____________________
dirname()からbasename()に変更したプログラムを試験的に実行してみました。
内容は上記のものと同じで、選択したファイルからファイル名を取得するというものです。
こちらでは、ファイル名と拡張子を上手く取得することができました。
また、.pyのファイルと画像データを同じディレクトリ内に入れた所、
ファイルの読み込みに成功し、print()でSVGファイルのタグを表示させることができました。

[kusao]

Traceback (most recent call last):
File "svg_changer.py", line 29, in <module>
main()
File "svg_changer.py", line 26, in main
change(filename)
File "svg_changer.py", line 15, in change
delete_tag = re.sub(r"<{}>.*</{}>".format(tag), "", row)
IndexError: tuple index out of range

本当に何度も申し訳ないのですが、タプルインデックスというエラーが発生するようになってしまいました。

[退会済みユーザー]

formatの使い方がちょっと間違っています。delete_tag = re.sub(r"<{tag}>.*</{tag}>".format(tag=tag), "", row)

[kusao]

返信が遅くなりまして、申し訳ありません。
.format()をご指摘のように改善した所、エラーが発生しなくなりました。
if文の分岐、自分の思った通りに機能していないようなので、一度、じっくり見直して見ようと思います。
ただ、、課題点であった
・基本的なファイルを読み込み
・正規表現部分

に関しては、きちんと機能するようになりました。
何度も、親切にアドバイスを頂きまして、有難うございました。
また、何かあれば知恵をお貸し頂きたいです。