質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.52%

  • Python 3.x

    6368questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

python mysqlでセレクトするときのエラー

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 676

World

score 22

関数用のファイルと実行するファイル二つに分けてあります。
エラー内容は、


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ページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • seastar3

    2017/09/27 04:13

    パスワードはさらすべきではありません。早く変えなければ。mysqlで日本語のテーブル名とフィールド名を使っているのでしょうか。テーブル名は"総資産"でいいのでしょうか。"fxpro"のデータベースの仕様が分かりません。

    キャンセル

  • World

    2017/09/27 10:25

    パスワードがすぐにかえさせていただきました。ありがとうございます。テーブルなども書かせていただきました。

    キャンセル

回答 2

+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()

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/27 13:21

    丁寧な説明ありがとうございます。
    結果としては、エラーがでてしましました。
    Soushisan_check()をする前に、Insert処理をしています。それがまずいということでしょうか?
    一応gituhbにのせましたので、時間がありましたら見てほしいです。
    https://github.com/zerone08/fxpro.git

    キャンセル

  • 2017/09/27 14:21 編集

    あぁ、忘れていたことがありました。すみません。。mydb.close()が原因だと考えられます。これを消すか、mydb = MySQLdb.connect(...)をすべての関数内に書くか、どちらかをしてください。

    私は前者の方で回答を編集しました。以前の回答は履歴からみてください。m(_ _)m

    キャンセル

  • 2017/09/27 14:31

    もしかしたらmydb.close()が呼ばれないのは危ないかもしれないので、私の方法で行う場合はclose()する関数をapp_mysql.pyの中で別に用意して、fxpro.pyの方から最後に呼ぶようにするといいかもしれません。

    キャンセル

  • 2017/09/27 15:03

    追記しました。よろしくお願いします。m(_ _)m

    キャンセル

  • 2017/09/29 18:06

    ありがとうございます!!できました!!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.52%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Python 3.x

    6368questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。