python def内のfor文が動かない？

Question

まだまだpython初心者のものです。classやdefなどを使い始めたばかりです。

実現したいこと

複数あるエクセルから、wx.Comboboxを使い、ファイル名を選択します。そのファイルから情報を読み取りたいのですが、
読み取るための定義の中のforが動きません。

###エクセルの中身

###ソースコード

#!/usr/bin/env python
# -*- coding: utf-8 -*-
# pip install openpyxl

import wx
import wx.xrc
import openpyxl
import os
import glob

################ 実行ファイルのパス##########
dir = os.path.dirname(os.path.abspath(__file__))

#####フォルダのエクセルファイルをリスト化######
excels =glob.glob(dir + '/*.xls*')	#xls,xlsxの拡張子のファイルをフルパス付リスト化
excels.remove(dir + '\meibo.xlsx')     #meibo.xlsxを除く
excels_file =[]
for e in excels :			#meibo.xlsxを除いて、ファイル名のみのリストを作成
	excels_file.append(e.replace(dir+'\', ''))


####定義変数名####
kaigi = ""	#会議セルの値
seki = 0	#座席の数
meibo =[]	#名簿
prognum = []	#プログラム用番号
report = []	#初期レポート名
m_comboBox1Choices = report
m_comboBox0Choices = excels_file

class MyFrame1 ( wx.Frame ):

	def __init__( self, parent ):
		wx.Frame.__init__ ( self, parent, id = wx.ID_ANY, title = "TEST!! " + kaigi, pos = wx.DefaultPosition, size = wx.Size(800,600 ), style = wx.DEFAULT_FRAME_STYLE|wx.TAB_TRAVERSAL )

		self.SetSizeHints( wx.DefaultSize, wx.DefaultSize )
		self.SetBackgroundColour( wx.SystemSettings.GetColour( wx.SYS_COLOUR_WINDOW ) )

		bSizer1 = wx.BoxSizer( wx.VERTICAL )

	#####0列目の項目######
		global m_comboBox0
		bSizer0 = wx.BoxSizer( wx.HORIZONTAL )
		self.m_comboBox0 = wx.ComboBox( self, wx.ID_ANY, u"エクセルファイルを選択", wx.DefaultPosition, wx.DefaultSize, excels_file, 0 )
		self.m_comboBox0.SetMinSize( wx.Size( 200,-1 ) )
		bSizer0.Add( self.m_comboBox0, 0, wx.ALL, 5 )
		bSizer1.Add( bSizer0, 0, wx.EXPAND|wx.FIXED_MINSIZE, 1 )
		bSizer0 = wx.BoxSizer( wx.VERTICAL )


	#####座席表#####
		self.mainPanel = wx.Panel(self)
		self.darea = MyFrame1.ZasekiArea(self.mainPanel)
		self.SetSizer( bSizer1 )
		self.Layout()
		self.Centre( wx.BOTH )
		self.m_comboBox0.Bind(wx.EVT_COMBOBOX, self.excel_change_func )  #コンボボックスを変更時の動作

	def __del__( self ):
		pass

    #####エクセルファイル選択時の動作#####
	def excel_change_func(self , event ):
		col = excels_file.index(self.m_comboBox0.GetStringSelection())	#ボタンが何番目かを取得
		self.file = excels[col]						#エクセルファイルのフルパスをっ取得
		self.excell_import(self.file)

    #####エクセルから情報を読み取る#####
	def excell_import (self ,file) :
		global meibo
		self.wb = openpyxl.load_workbook(file ,data_only=True,read_only=False)　#エクセルファイルを開く
		self.sheet = self.wb['date']						#シートを選択

		#会議セルの値を取得
		self.kaigi = self.sheet.cell(1,3).value

		####座席配列####
		self.seki = int(self.sheet.cell(3,6).value)

		####名簿の情報を取得####			←ここで情報を読み取りたいが、何も動作していない様子
		for row in self.sheet['A5:IV'+ str(seki + 4)] :
			for col in row :
				meibo.append(col.value)
		print(meibo)					←空のリストしか表示されない。


	class ZasekiArea(wx.Panel):
		def __init__(self, parent):
			wx.Panel.__init__(self, parent, id = wx.ID_ANY,  pos = wx.DefaultPosition, size = wx.Size(800,600 ), style = wx.DEFAULT_FRAME_STYLE|wx.TAB_TRAVERSAL)
			.......
			self.mainPanel.SetSizer(sizer)
			self.mainPanel.Bind(wx.EVT_COMBOBOX , MyFrame1.excel_change_func)

if __name__ == '__main__':
	app = wx.App(False)
	frame = MyFrame1(None)
	frame.Show(True)
	app.MainLoop()

なぜ、for文が機能しないのか、教えてください。また、どのようにすれば、機能するのか教えてください。
よろしくお願いします。

Comments

[meg_]

row、colの値はデバッグして確かめましたか？

[hiro04kon]

返信、ありがとうございます。
両方とも、空、というか、反応しませんでした…

Answer 1

下記にselfが抜けておりグローバルで定義されている方のseki=0として判断されてforループがから配列で抜けているのではないでしょうか？

for row in self.sheet['A5:IV'+ str(seki + 4)] :

下記の形にするといかがでしょうか？

for row in self.sheet['A5:IV'+ str(self.seki + 4)] :

Comments

[hiro04kon]

すいません！その通りでした！なんたるボンミス…
動きました！ありがとうございました！

[m2l]

定義ミスは中々気付きにくいですよね。。
動いてよかったです！