pythonでifが動作しない。

Question

前提・実現したいこと

クライアント側からftpサーバーにディレクトリを作成し、ファイルを転送したい。
ディレクトリ作成からファイル転送まではできるようになったが、同名のディレクトリが
ある場合にエラーが起きるため、先に同名のディレクトリがあるかを検出し、
あった場合には削除してからディレクトリを作成するようにしたい。

同名のディレクトリがない場合には、エラーが出ずに動作するが、
同名のディレクトリがある場合には、エラーをはいてしまう。
そのため、ifの部分が動作していないと考えられる。
どのようにすればifの部分が動くようになるでしょうか。

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

Traceback (most recent call last):
  File "test.py", line 72, in <module>
    main()
  File "test.py", line 64, in main
    ftp_upload(hostname, username, password, upload_file, dir)
  File "test.py", line 44, in ftp_upload
    ftp.mkd(dir)
  File "C:\Python27\lib\ftplib.py", line 589, in mkd
    resp = self.sendcmd('MKD ' + dirname)
  File "C:\Python27\lib\ftplib.py", line 251, in sendcmd
    return self.getresp()
  File "C:\Python27\lib\ftplib.py", line 226, in getresp
    raise error_perm, resp
ftplib.error_perm: 550 Cannot create a file when that file already exists.

該当のソースコード

# -*- coding: utf8 -*

import ftplib
import os.path

def  ftp_upload(hostname, username, password, upload_file, dir):
    ftp = ftplib.FTP(hostname)
    ftp.set_pasv("true")
    ftp.login(username, password)
    fp = open(upload_file, 'rb') 
    if  os.path.exists(dir):
        ftp.delete(dir + '/' + upload_file)
        ftp.rmd(dir)
           
    ftp.mkd(dir)
    s = dir + '/'+upload_file
    ftp.storbinary('STOR '+s, fp)
    ftp.close()
    fp.close()

def main():
    hostname = "***.***.**.**"
    upload_file = "test.py"
    username = "hoge"
    password = "hoge"
    dir = "hogehoge"
    ftp_upload(hostname, username, password, upload_file, dir)

if __name__ == '__main__':
    main()

試したこと

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

windowsでpython2.7をつかってます。

Answer 1

if が動作しない、んじゃなく、動作してるんだけどディレクトリの削除に失敗してるだけ、なんではないかと。

ホンマに動作してないのか確認してみるべし
＃printでも入れれば確認できるでしょ

Comments

[sekkei_no_hito]

printを入れたところ動作確認できました。
ftpのコードの中にos.pathを使ったため、Trueにならなかったみたいです。
ftpのオブジェクトで代用できるものはないでしょうか

[y_waiwai]

そのdir変数には何がはいってますか？
ちゃんとフルパスはいってるんでしょうか

[sekkei_no_hito]

サーバー側とLANケーブルで繋いでありまして、ftpのサーバーが1つしかないのでフルパスでなくてもファイル操作できることは試してあります。フルパスを入れて試したりもしましたがダメでした。

[y_waiwai]

ああ、FTPでしたね
なら、FTP.dir関数でディレクトリがあるかを確認するって話になりますね
存在するかどうか、ではなく、その名前があるかどうかって関数なので、ちょっと修正がいりますね

[sekkei_no_hito]

解決しました。
ありがとうございました。

Answer 2

print(os.path.exists(dir))

この結果はどうなりますか

Comments

[sekkei_no_hito]

ご回答ありがとうございます。
ない場合に作るではなく、ある場合に消すを動かしたいのですが、どのようにすればよろしいでしょうか

[jckkvs]

結果は一緒では？

[sekkei_no_hito]

そうなると同名のディレクトリがあった場合にそのままになってしまいます。
同名のディレクトリがあった場合には、それを消してから新たなディレクトリを作りたいのです。