QTableWidgetのテーブルの最終行を取得したい

Question

pyqtで画像のようなテーブルがあるアプリを作成中です。

テーブル上の行がすべて空白だった場合、最終行とみなしてその行にデータを入力していく
プログラムを組みたいのですがうまくいきません。

1行分の入力されたデータ1つずつを全て連結して内容が空白であれば最終行とみなそうとすると、
空白セルを連結しようとした時にエラーメッセージも残さずアプリケーションが終了してしまいます。

上記の組み方がスマートか不明なのですが、空白行の判定の仕方、または別のスマートな方法を
ご存知の方がいらっしゃったらご教示いただけませんでしょうか。

よろしくお願いします。

#追記
組み上げ中のコードが入力数制限で記載できなかった為、
似たようなものを別途作成しました。
self.tableWidget.item(j, i).text()　を空白のセル部分で取得しようとすると
アプリケーションが終了してしまいます。
引き続きご教示いただけますと幸いです。

import sys
from PyQt5.QtWidgets import QMainWindow, QApplication, QWidget, QAction, QTableWidget,QTableWidgetItem,QVBoxLayout
from PyQt5.QtGui import QIcon
from PyQt5.QtCore import pyqtSlot

class App(QWidget):

    def __init__(self):
        super().__init__()
        self.title = "PyQt5 table - pythonspot.com"
        self.left = 0
        self.top = 0
        self.width = 300
        self.height = 200
        self.initUI()

    def initUI(self):
        self.setWindowTitle(self.title)
        self.setGeometry(self.left, self.top, self.width, self.height)

        self.createTable()

        # Add box layout, add table to box layout and add box layout to widget
        self.layout = QVBoxLayout()
        self.layout.addWidget(self.tableWidget)
        self.setLayout(self.layout)

        # Show widget
        self.show()

    def createTable(self):
        # Create table
        self.tableWidget = QTableWidget()
        self.tableWidget.setRowCount(4)
        self.tableWidget.setColumnCount(4)
        self.tableWidget.setItem(0,0, QTableWidgetItem("Cell (1,1)"))
        self.tableWidget.setItem(0,1, QTableWidgetItem("Cell (1,2)"))
        self.tableWidget.setItem(1,0, QTableWidgetItem("Cell (2,1)"))
        #self.tableWidget.setItem(1,1, QTableWidgetItem("Cell (2,2)"))
        #self.tableWidget.setItem(2,0, QTableWidgetItem("Cell (3,1)"))
        self.tableWidget.setItem(2,1, QTableWidgetItem("Cell (3,2)"))
        self.tableWidget.setItem(3,0, QTableWidgetItem("Cell (4,1)"))
        self.tableWidget.setItem(3,1, QTableWidgetItem("Cell (4,2)"))
        self.tableWidget.move(0,0)

        # table selection change
        self.tableWidget.doubleClicked.connect(self.on_click)


    @pyqtSlot()
    def on_click(self):
        print("\n")
        for currentQTableWidgetItem in self.tableWidget.selectedItems():
            fragA = True
            fragB = False
            word =""
            i = 0
            j = 0
            while fragA:
                for s in range(4):
                    if  self.tableWidget.item(j, i).text() != "":
                        fragB = True
                    else:break

                    i += 1
                    if fragB == True:
                           break
                j=j+1
                i = 0

            #
            print(self.tableWidget.item(1,1).text())

if __name__ == "__main__":
    app = QApplication(sys.argv)
    ex = App()
    sys.exit(app.exec_())

Comments

[tiitoi]

現状のコードを記載してください

[ookura]

遅くなりまして申し訳ありません。
入力制限の為、コード再編して記載しました。
※最終行取得は割愛しています。（空白セル取得部分で引っかかっています。）
ご確認をお願いします。

Answer 1

空白セルを連結しようとした時にエラーメッセージも残さずアプリケーションが終了してしまいます。

再現できる最小のコードを提示してください。

または別のスマートな方法をご存知の方がいらっしゃったらご教示いただけませんでしょうか。

QTableWidget ではなく QTableView と TableModel の利用がスマートな方法です。
データに関しては、終端の判別の必要はなくなり、リストに追加するのみで済みます。

但し、コードの記述量は増えるので Model/View 分離のコンセプトへの理解がないと、
恩恵が分かり難いかもしれません。
（拡張性や保守性に優れた方法という意図で、簡単な方法では有りません）

Comments

[ookura]

コードが最小に収まらなかった為、近いものを再編して追記しました。
お手数ですがご確認をお願いします。
Qtableviewは一度入力されたリストの書き換え、編集はできますか？
Qtablewidgetだと書き換え・編集が容易だった為使用しているのですが、
書き換えが出来るのであれば多少コードが長くなってもviewに置き換えをしようと思います。
お手数ですが、ご教示いただけますと幸いです。

[teamikl]

＞Qtableviewは一度入力されたリストの書き換え、編集はできますか？

可能です。QTableView は、高機能になった Table と考えて良いです。但し、
小規模な場合は、手間に見合う恩恵があるかどうかという点で QTableWidget で十分なことも有ります。

使い分けに関しては、主に中～大規模向 例えばデータベースの内容を表示するとき等、
メモリに乗り切らないデータを扱う際に、QTableView/TableModel が使われます。

小規模な場合でも、今回のように、末尾の判定をQTableView の内部状態に依存する場合、
データ(Modelに当たる部分)を別の変数にリストとして持っておけば、
末尾判定の必要なく、(データに関しては) 要素の追加のみで済みます。
その後に GUI の表示部分を更新といった手順を取ります。
QTableWidget を使っても同様のアプローチで実装できますが、
このデータ部分と表示の分離を定型化したのが QTableView/TableModel です。

※ 恩恵部分が解りにくいと思うので、注釈
末尾の空行判定では、計算量オーダーは O(n) リストの長さに比例して時間がかかります。
通常のリストに要素を追加(list.append) では O(1) リストの長さに関わらず一定です。

----
コードに関しては、(while flagA で無限ループになって) 正常に実行できてませんが

- item() が None を帰す場合があるかもしれない。 item(j, i).text()
- ループが多重になっていて終了条件がない
　- break は内側のforループを抜けます
　- while flagA: ... flagA を変更するコードがない (※最終行取得は割愛しています。・・・に当たる部分？）

空白セル取得部分に関して言えば、.text() を呼ぶ前に、先に item() の戻り値チェックが必要です

item = self.tableWidget.item(j, i)
if item and item.text() != "":

[teamikl]

ループの骨格は、 i, j のみでよいのでは。変数 s が未使用です。
（コメント内の為、インデントは全角スペースに置き換えてます）

for j in range(行数):
　for i in range(列数):
　　item = self.tableWidget.item(j, i)

[ookura]

とても丁寧にお教えいただき本当にありがとうございます。
すごく勉強になります。
完全に問題解決した上にループ文もスマートにできました！
tableViewに関しては使ったことがなかったですが、興味がわいたので使ってみようと思います。
ありがとうございました！