関数用のファイルと実行するファイル二つに分けてあります。
エラー内容は、
Traceback (most recent call last):
File "C:\Users\rice8_000\Desktop\fx\fxpro.py", line 8, in <module>
print(Soushisan_check())
File "C:\Users\rice8_000\Desktop\fx\app\app_mysql.py", line 16, in Soushisan_check
mycursor.execute(sql)
File "C:\Users\rice8_000\AppData\Local\Programs\Python\Python36-32\lib\site-packages\MySQLdb\cursors.py", line 216, in execute
while self.nextset():
File "C:\Users\rice8_000\AppData\Local\Programs\Python\Python36-32\lib\site-packages\MySQLdb\cursors.py", line 171, in nextset
db = self._get_db()
File "C:\Users\rice8_000\AppData\Local\Programs\Python\Python36-32\lib\site-packages\MySQLdb\cursors.py", line 201, in _get_db
raise ProgrammingError("cursor closed")
_mysql_exceptions.ProgrammingError: cursor closed
関数をまとめたファイルのコード
import MySQLdb
mydb = MySQLdb.connect(host="localhost",db="fxpro",user="staff",passwd="kqZwIGGqAo2oB6Gf",charset="utf8")
mycursor = mydb.cursor()
#*********************************************
#総資産をチェックします。
#総資産がマイナスまたは0以下の場合False
#総資産がプラスまたは1以上の場合True
#*********************************************
def Soushisan_check():
sql = "Select 総資産 From 総資産 Where ID = 1"
mycursor.execute(sql)
result = mycursor.fetchall()
for row in result:
if (row[0] == 0):
return False
else:
return True
mycursor.close()
mydb.close()
実行するファイル
from app.app_fxdata import *
from app.app_mysql import *
#総資産をチェックする
print(Soushisan_check())
今回使用するデータベースの仕様は、
データベース名が、fxpro
テーブル名が、総資産 です
結果としては、Trueが出る予定です。
-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
+1
総資産がマイナスまたは0以下の場合False
ならば、
if (row[0] == 0):
は、
if (row[0] <= 0):
です。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
checkベストアンサー
0
エラーの方ですが、Soushisan_check()
が実行されるのは本当に一度だけなのでしょうか?
以下検証です。(都合上sqlite3
を使っています。が多分大差ないと思います。)
foo.db
にはテーブルtable1
、カラムpiyo
、データbar
が入っているとします。
# sqlFunc.py
import sqlite3 as sq
foo = sq.connect("./foo.db")
cur = foo.cursor()
def hoge():
cur.execute("""SELECT piyo FROM table1;""")
print(cur.fetchall()[0][0])
cur.close()
if __name__=="__main__":
hoge()
# sqlExc.py
from sqlFunc import hoge
hoge()
hoge()
以下実行結果です。
bar
Traceback (most recent call last):
File "/Users/~/sqlExc.py", line 4, in <module>
hoge()
File "/Users/~/sqlFunc.py", line 7, in hoge
cur.execute("""SELECT piyo FROM table1;""")
sqlite3.ProgrammingError: Cannot operate on a closed cursor.
1度目はうまくいきますが、
2度目のhoge()
の呼び出しでエラーが発生していることがわかります。
(そのために実行は一度なのかどうかを聞きました。)
一言でいうと、カーソルの開閉が関数内で対になっていないのが原因です。
cur.close()
は一度目の実行でなされてしまっているので、実行するにはもう一度接続する必要があります。
ですから、以下のように直してみます。
import sqlite3 as sq
foo = sq.connect("./foo.db")
#cur = foo.cursor() <- 消す
def hoge():
cur = foo.cursor() # <- こっちに移動
cur.execute("""SELECT piyo FROM table1;""")
print(cur.fetchall()[0][0])
cur.close()
if __name__=="__main__":
hoge()
こうするとちゃんと2度bar
が表示されます。
質問者様のコードの場合はこう直すといいと思います。
(9/27 14:30頃更新)
import MySQLdb
mydb = MySQLdb.connect(host="localhost",db="fxpro",user="staff",passwd="****************",charset="utf8")
#*********************************************
#総資産をチェックします。
#総資産がマイナスまたは0以下の場合False
#総資産がプラスまたは1以上の場合True
#*********************************************
def Soushisan_check():
mycursor = mydb.cursor()
sql = "Select 総資産 From 総資産 Where ID = 1"
mycursor.execute(sql)
result = mycursor.fetchall()
for row in result:
if (row[0] == 0):
return False
else:
return True
mycursor.close()
# mydb.close() <- 消す
MySQLdb
の方で試していないので、これで直るかはちょっと自信が無いですが、試してみてください。
長文失礼しました。m(_ _)m
追記
関数が一つだったので移動する解決方法をのせましたが、
コメントでも書いた通り閉じる関数を別に用意する方が早かったかもしれません。
import MySQLdb
mydb = MySQLdb.connect(host="localhost",db="fxpro",user="staff",passwd="****************",charset="utf8")
mycursor = mydb.cursor()
#*********************************************
#総資産をチェックします。
#総資産がマイナスまたは0以下の場合False
#総資産がプラスまたは1以上の場合True
#*********************************************
def Soushisan_check():
sql = "Select 総資産 From 総資産 Where ID = 1"
mycursor.execute(sql)
result = mycursor.fetchall()
for row in result:
if (row[0] == 0):
return False
else:
return True
def closeDb(): # <- 追加
mycursor.close()
mydb.close()
from app.app_fxdata import *
from app.app_mysql import *
#総資産をチェックする
print(Soushisan_check())
# 全部の処理が終わったら閉じる
closeDb()
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう!
- ただいまの回答率 88.32%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
質問への追記・修正、ベストアンサー選択の依頼
seastar3
2017/09/27 04:13
パスワードはさらすべきではありません。早く変えなければ。mysqlで日本語のテーブル名とフィールド名を使っているのでしょうか。テーブル名は"総資産"でいいのでしょうか。"fxpro"のデータベースの仕様が分かりません。
World
2017/09/27 10:25
パスワードがすぐにかえさせていただきました。ありがとうございます。テーブルなども書かせていただきました。