getRoom.rmview
. में आपके द्वारा भरी जाने वाली विधि rmview
. के फ़ील्ड असाइन करें उदाहरण के लिए बार-बार, लेकिन आप कभी भी roomlist
को संशोधित नहीं करते हैं rmview
. का कभी संशोधित नहीं किया जाता है; यह खाली रहता है। चूंकि आप roomlist
. का उपयोग करते हैं TableView
. में , दिखाने के लिए कोई डेटा नहीं है।
इसके बजाय आपको डेटाबेस क्वेरी की प्रत्येक पंक्ति के लिए एक नया तत्व जोड़ना चाहिए:
// remove data previously in the list
rmList.roomlist.clear();
while (rs.next()){
rmList.roomlist.add(new ListRoom(rs.getString(1),
rs.getString(2),
rs.getString(3),
rs.getString(4),
rs.getString(5)));
}
इसके अलावा मैं नामकरण परंपराओं
का पालन करने की सलाह देता हूं। . विशेष रूप से संक्षेप के बारे में हिस्सा क्योंकि इससे आपके कोड को दूसरों के लिए पढ़ना मुश्किल हो जाता है। इसके अलावा object
एक प्रकार के पैरामीटर के लिए नाम के रूप में एक खराब विकल्प है। यह भ्रम पैदा करता है और आमतौर पर टाइप पैरामीटर के लिए सिंगल अपरकेस अक्षरों का उपयोग किया जाता है।
साथ ही rmlist
. का उद्देश्य वर्ग अस्पष्ट है। इसमें ListRoom
. के समान फ़ील्ड शामिल हैं , लेकिन इसमें एक सूची भी शामिल है जहां आप चाहते हैं कि डेटा वास्तव में संग्रहीत किया जाए। आपको उन क्षेत्रों की आवश्यकता क्यों है? क्या आपको rmlist
की आवश्यकता है? या इसे केवल ObservableList<ListRoom>
. से बदला जा सकता है ?