url存在チェックの結果をCSV に書き出したい

Question

実現したいこと

• original.csvに記載されている各URLからURLの存在チェックをおこなう
• チェックの結果をoriginal.csvのとなりの列に書き出しを行いたい

該当のソースコード

import urllib.request, urllib.error

def checkURL(url):
    try:
        f = urllib.request.urlopen(url)
        print ("OK:" + url )
        f.close()
    except:
        print ("NotFound:" + url)

if __name__ == '__main__':

    with open('C:\\Users\\original.csv') as f:
        for line in f:
            # print(line, end='')
            checkURL(line)

試したこと

以下のようにcsvに書き出しできないかと試したのですが期待通りには書き出しされました。
エラー：UnsupportedOperation: not readable

import urllib.request, urllib.error

def checkURL(url):
    try:
        f = urllib.request.urlopen(url)
        print ("OK:" + url )
        f.close()
    except:
        print ("NotFound:" + url)

if __name__ == '__main__':

    with open('C:\\Users\\original.csv','a', encoding='utf-8',newline='') as f:
        writer = csv.writer(f)
        for line in f:
            # print(line, end='')
            checkURL(line)
            row = [url,checkURL(line)]
            writer.writerow(row)

実現方法のヒントをいただければと思います。よろしくお願いいたします。

Comments

[y_waiwai]

このままではコードが読めないので、質問を編集し、</>(コードの挿入)ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[y_waiwai]

んで、提示のコードではどういう動作となるんでしょうか。
そして、それをどう言う風にしたいのか説明しよう

[c-man]

すみません。更新しました。

上のコードでは結果が出力はしました。
下のコードで出力したコードをCSVに書き出しを行いたいのですがエラーになる状態です。
エラー：UnsupportedOperation: not readable

[n24bass]

別ファイルに書き出してからリネームとか。

Answer 1

2つめのコード：

with open('C:\Users\original.csv','a', encoding='utf-8',newline='') as f:

と、書き込み用のオープンをしているにもかかわらず、

for line in f:

で読み込みが発生するので、「書き込みオープンしたファイルから読もうとしている」というエラーです。

別のファイルに書くというのが簡明かと思います。

Answer 2

checkURL関数は値を返してません。
ちゃんとあなたの思うような値を返すようにしましょう

row = [url,checkURL(check)]

そのうえ、どこにも定義されてないcheckってのを使ってますね

Comments

[c-man]

コメントありがとうございます。
checkは別で試したときに使用したものが残ってしまっていました。
以下のように修正してみたのですが変化がなかったです。
row = [url,checkURL(line)]

[y_waiwai]

checkURL関数は値を返してません。
返してないのに、関数の返り値を使おうとしているので、
ちょっとそれはおかしいなやいんか？とPCさんが怒ってらっしゃいます

Answer 3

たくさんのコメントありがとうございました。
以下のようにして実現が可能となりました。

HEADER = ['OriginUrl','url_link']

def checkURL(url):
    try:
        f = urllib.request.urlopen(url)
        f.close()
        return ("OK")
    except:
        return ("NotFound")

if __name__ == '__main__':
    with open('C:\\Users\\original.csv', encoding='utf-8') as f:
        with open('C:\\Users\\original.csv'01','a', encoding='utf-8',newline='') as f0:
            writer = csv.writer(f0)
            writer.writerow(HEADER)
            for line in f:
                # print(line, end='')
                print(line)
                print(checkURL(line))
                row = [line,checkURL(line)]
                writer.writerow(row)

Answer 4

コードには問題が2つあります。

1. 読み込み用に開いたファイルに書き込もうとしている。
   そもそも、読んでいる最中のファイルに、何かをうまく書き込むのは難しい処理です。Excelなどはそうしているように思うかもしれませんが、実は裏では違うこと(別のファイルに書いて置き換える)をしています。
   なので、別の回答にもあるとおり、結果は違うファイルに書き出し、必要であれば元のファイルと置き換えるようにするのがいいでしょう。

2. 関数の計算結果の扱いが間違えている
   「row = [url,checkURL(line)]」と書けば、checkURL()関数の処理結果がここに入ると考えているのだと思います。 ここで、その結果とは何なのかが問題です。
   checkURL() 関数を実行するこ結果が画面に出力されますが、これは、よくプログラミングでは副作用と呼ばれるもので、 プログラム上の関数の実行結果ではないので「row = [url,checkURL(line)]」このリストには入らなのです。
   関数の実行結果をプログラム内で受け取るためには、関数はその結果を返す(returnする)必要があります。

def checkURL(url):
    try:
        f = urllib.request.urlopen(url)
        f.close()
        return ("OK:" + url )
    except:
        return ("NotFound:" + url)