lid=1をdevicesのコンボボックスに変更して中身を切り替えて表示させたい

Question

コード

import tkinter as tk
# tkinterモジュールをtkとしてインポート
import tkinter.ttk as ttk
# tkinterのttkをttkとしてインポート
import matplotlib.pyplot as plt
# matplotlibモジュールのpyplotをpltとしてインポート
from matplotlib.backends.backend_tkagg import (
    FigureCanvasTkAgg, NavigationToolbar2Tk)
# FigureCanvasTkAggとNavigationToolbar2Tkのインポート
from random import random
# randomモジュールからrandom関数をインポート
from datetime import datetime
               # datetimeモジュールからdatetime関数のインポート
from functools import partial
# functoolsモジュールからpartialのインポート
import pymysql.cursors
# pymysqlモジュールのcursorsをインポート
#--------------------------------------------------------------------
def select(yyyy,MM,dd,lid):  # select関数
  connection = pymysql.connect(
    host='localhost',                             
    user='hit',                                  
    password='hit',                               
    db='THT',                                      
    charset='utf8',                                
    cursorclass=pymysql.cursors.DictCursor)      
  
  with connection.cursor() as cursor:
    FROM = yyyy+"-"+MM+"-"+dd+" 00:00:00"
    TO   = yyyy+"-"+MM+"-"+dd+" 23:00:00"
    sql = "select * from thData where lid="+str(lid)
    sql += " and date>='"+FROM+"' and date<='"+TO+"'"
    print("sql=",sql)                             

    cursor.execute(sql)                          
    results = cursor.fetchall()                    
    #for r in results:  print(r)
  return results
#--------------------------------------------------------------------
def make_graph():    # make_graph関数
  #print("<< make_graph start >>")
  x  = [0]*24
  y1 = [None]*24
  y2 = [0]*24
  yyyy = cb1.get()
  m = int(cb2.get())
  d = int(cb3.get())
  MM = "{:02d}".format(m)                 
  dd = "{:02d}".format(d)                  
  lid = 1                        
 
  #print("yyyy=",yyyy," ",type(yyyy))
  #print("MM=",MM," ",type(MM))
  #print("dd=",dd," ",type(dd))

  #--------------------------------
  results = select(yyyy,MM,dd,lid)            
  #--------------------------------
  for i in range(0,24): x[i]=i
  num = len(results)
  print("num=",num)
  if num == 0:  y1 = [None]*24

  for i in range(num):
    if i > 23: break
      
    temp = results[i].get('temp')               
    hcount = results[i].get('hcount')           
    print(i," ",temp," ",hcount)
    y1[i] = temp
    y2[i] = hcount
  
  ax1.cla()
  ax1.set_xlabel('Hour (H)')                 
  ax1.set_ylabel('Temperature (deg)')          
  ax1.set_ylim(-5,25)                           
  ax1.plot(x,y1,color="blue",marker="o")                                

  ax2.cla()
  ax2.set_ylabel('Human detected (number)',color="red")  
  ax2.set_ylim(0,65)                           
  ax2.bar(x,y2,color="red")                    

  canvas.draw()                                
  #print("<< make_graph end >>")
  
#-----------------------------------------------------------------
x  = [0]*24                                    
y1 = [None]*24                                   
y2 = [0]*24                                  

root = tk.Tk()                                  
root.title("TEST")                             
root.geometry("650x580")                       

frame1 = tk.LabelFrame(
root,labelanchor="nw",
text="Temperture & Human Sensor",foreground="green") 
# フレーム１
frame1.grid(rowspan=2,column=0)

frame2=tk.LabelFrame(root,text="Control", foreground="red")
                                                
frame2.grid(row=2,column=0,sticky="nwse")

fig = plt.Figure()
ax1 = fig.add_subplot(111) 

ax1.plot(x,y1)                               
ax1.set_xlabel('Hour (H)')                   
ax1.set_ylabel('Temperature (deg)')          
ax1.set_ylim(-5,25)                          

ax2 = ax1.twinx()                           
ax2.bar(x,y2,color="red")                  
ax2.set_ylabel('Human detected (number)',color="red")
ax2.set_ylim(0,65)                         

canvas = FigureCanvasTkAgg(fig, master=frame1) 
# キャンバスのインスタンス化
canvas.get_tk_widget().pack(side=tk.TOP, fill=tk.BOTH, expand=1)

now = datetime.now()                        
date = "{:%Y-%m-%d}".format(now) 
month = now.month                        
day = now.day                               

years = ["2020","2021","2022","2023","2024","2025"] 
# 年（コンボボックス）
months = [1]*12
for i in range(12) : months[i]=i+1        
days = [1]*31                               
for i in range(31) : days[i]=i+1         
devices = ["810C3B31","790A2D12"]

lb1 = tk.Label(frame2, text="year:",bg="cyan")  
cb1 = ttk.Combobox(frame2, values=years,width=6,justify=tk.CENTER)
cb1.current(0)                                   

lb2 = tk.Label(frame2, text="month:",bg="cyan") 
cb2 = ttk.Combobox(frame2, values = months,width=6,justify=tk.CENTER)
cb2.current(month-1)                              

lb3 = tk.Label(frame2, text="day:",bg="cyan")  # ラベル３（lb3）
cb3 = ttk.Combobox(frame2, values = days,width=6,justify=tk.CENTER)
cb3.current(day-1)                                
lb4 = tk.Label(frame2, text="Device ID :",bg="cyan")  
cb4 = ttk.Combobox(frame2, values= devices,width=6,justify=tk.CENTER)
cb4.current(0) 

y = cb1.current()                              
m = cb2.current()                               
d = cb3.current()                               
selected_date = str(years[y])+"-"+str(months[m])+"-"+str(days[d])
print("selected_date=",selected_date)           



print("cb1=",cb1.get()," ",end="")             
print("cb2=",cb2.get()," ",end="")            
print("cb3=",cb3.get())                         

bt1 = tk.Button(frame2, text="Display", bg="yellow",command=make_graph)
                                                
lb1.grid(row=1, column=0 ,padx=5, pady=5)     
cb1.grid(row=1, column=1 ,padx=5, pady=5)    
lb2.grid(row=1, column=2 ,padx=5, pady=5)     
cb2.grid(row=1, column=3 ,padx=5, pady=5)     
lb3.grid(row=1, column=4 ,padx=5, pady=5)      
cb3.grid(row=1, column=5 ,padx=5, pady=5)      
lb4.grid(row=1, column=6 ,padx=5, pady=5)      
cb4.grid(row=1, column=7 ,padx=5, pady=5)      
bt1.grid(row=1, column=8 ,padx=5, pady=5)       

make_graph()                                    

root.mainloop()    

試したこと
lid = cb4.get()とした。devices = ["1","2"]等の数字のみでは、この書き換えて出来たが
810C3B31、790A2D12の文字を含むとエラーが出る。

Comments

[attakei]

質問内にてコードと思われる部分が、コードブロックで囲まれていません。
そのため、一見してどのようなコードを書いているのかが判別できません。

以下URLの内容などを確認しつつ、コードの内状況がわかるように編集してください。
https://teratail.com/help/question-tips#questionTips3-5-1

※Pythonは言語構造としてインデントをブロックとみなすため、インデントの状況が不明瞭だと「何もわからない」と言っていい状態になります

[hinata_0123]

ご返信ありがとうございます。編集致しました。

Answer 1

エラー出力についての内容が質問にないことと、細かい挙動の確認まではしてませんが、
select()内で組み立ててるクエリが変です。

def select(yyyy,MM,dd,lid):  # select関数
  connection = pymysql.connect(
    host='localhost',                             
    user='hit',                                  
    password='hit',                               
    db='THT',                                      
    charset='utf8',                                
    cursorclass=pymysql.cursors.DictCursor)      

  with connection.cursor() as cursor:
    FROM = yyyy+"-"+MM+"-"+dd+" 00:00:00"
    TO   = yyyy+"-"+MM+"-"+dd+" 23:00:00"
+   # lidに文字列が来るならクオートが必要
+   sql = "select * from thData where lid= '"+ str(lid) + "'"
-   sql = "select * from thData where lid="+str(lid)
    sql += " and date>='"+FROM+"' and date<='"+TO+"'"
    print("sql=",sql)                             

    cursor.execute(sql)                          
    results = cursor.fetchall()                    
    #for r in results:  print(r)
  return results

lidとして1が来る分にはlid=1で整数の1とみなされますが、810C3B31だとlid=810C3B31なのでSQLとしては文法エラーのはず。

補足

PyMySQLを使うのであれば、パラメーター形式で渡せるのでこっちを使ったほうが安全（エラーが起きにくい）かと思います。

ドキュメントをもと、試しに書いてみた例（動作確認してません）

def select(yyyy,MM,dd,lid):  # select関数
  connection = pymysql.connect()

  with connection.cursor() as cursor:
    FROM = yyyy+"-"+MM+"-"+dd+" 00:00:00"
    TO   = yyyy+"-"+MM+"-"+dd+" 23:00:00"

    sql = "select * from thData where lid = %s and date >= %s and date <= %s"
    cursor.execute(sql, [lid, FROM, TO])
    results = cursor.fetchall()
  return results

Comments

[hinata_0123]

ありがとうございます。解答して頂いた方法で出来たのですが、グラフが表示されませんでした。1には810C3EDC、2には790A2D12が対応しており、コンボボックスで810C3EDCと790A2D12を切り替えることで、グラフを表示させたいです。

[attakei]

私には、
> 1には810C3EDC、2には790A2D12が対応しており

を実現しようとしている箇所が見当たらないので、まずはこうなるように実装するところから始めてください。