一行にまとめる方法について

Question

def fifthchar(filename):
    data = ""
    with open(filename, "r") as f:
        for text in f.readlines():
            if len(text) > 4:
                data += text[4]
        data.replace('\n','')
    f.close()
    return data

ファイル内にたくさんの文字列が入っているとき、すべての文字列の五番目の文字を一行にまとめて抜き出したいです。
私のコードでは一行で文字を抜き出せないというエラーが出てしまいました。
どうしてこのようなエラーが出るのかがわからないです。
解決方法を教えていただけないでしょうか？

AssertionError: False is not true : fifthchar(nevhzacc.txt) returned 
z
vvxv
xda.  It should have returned zvvxvxda.  Check your logic and try again.

Comments

[javahack]

ファイルの改行コードが'\n'ではなく'\r\n'または'\r'だったとか。

[Miyu_Oyama]

返信ありがとうございます。
さっそく試してみたのですが同じようなエラーがでてしまいました。
でも私のやり方が間違っていたのかもです(>_<)
アドバイスありがとうございました！！

Answer 1

.replaceは自身は変わらずに置換した結果を返します。
data.replace('\n','')をdata = data.replace('\n','')に修正してください。

Comments

[Miyu_Oyama]

返信ありがとうございます！
無事に解決することができました。
とても助かりました（^^）

Answer 2

def fifthchar(filename):
    data = ""
    with open(filename, "r") as f:
        for text in f.readlines():
            if len(text) > 4:
                data += text[4]
        data = data.replace('\n','')
    f.close()
    return data

can110さんのくれたアドバイス通りなのですが一応解決方法を貼っておきます。
お二人とも返信ありがとうございました(^^)