Results 1 to 9 of 9
Hybrid View
-
2nd August 2010 16:43 #1
Ìàñîâà çàìÿíà íà private -> protected
Çäðàâåéòå,
Èìàì åäèí ìàëêî òåîðåòè÷åí ïðîáëåì. Ðàáîòÿ ïî äîñòà ãîëÿìî ðàçøèðåíèå íà Java-ïðîåêòà Sweet Home 3D (www.sweethome3d.eu). Òîé å ñ îòâîðåí êîä. Ïðîáëåìúò å, ÷å íà ïðàêòèêà íàâñÿêúäå ñà èçïîëçâàíè private ñïåöèôèêàòîðè, êîèòî ìè ïðå÷àò äà äåôèíèðàì äúùåðíè êëàñîâå. Òà íàâñÿêúäå, êúäåòî ñðåùàì private, êîéòî ìè ïðå÷è, ãî ïðîìåíÿì íà protected. Àêî íàïðàâÿ åäíî àâòîìàòèçèðàíî ïðåòúðñâúíå è çàìåñòâàíå íà private ñ protected, äàëè áèõà èçíèêíàëè íÿêàêâè ïðîáëåìè - ïîíå íà òåîðèÿ? Äîñåãà ñúì ãî ñìåíèë íà ñòîòèöè ìåñòà è íå ñå ñåùàì äà å èìàëî êîìïèëàöèîííè èëè ëîãè÷åñêè ãðåøêè ñëåä òîâà. Êàêâî ìèñëèòå?MSI B450 Gaming Pro Carbon AC | Ryzen 9 5900x | HyperX Predator 3200 | Asus Strix 3090 | Kingston KC3000 2TB | WD Red 4TB | Dell G3223Q + LG 27UK650-W | Arctic Freezer 360 | Seasonic Focus GX-1000 | Lian Li O11 Dynamic
-
2nd August 2010 17:03 #2
Ñìåíÿé ñìåëî. Ùîì "îõëàáâàø" êîíòðîëà, íå áè òðÿáâàëî äà ñðåùíåø ïðîáëåì.
Åòî òàêà ñå ïëîäÿò òå, êàçâàì ñè àç. Ïîä íîñà òè.
"640K îught to be enough for anybody" - Bill Gates, 1981
Òåñòîâå íà áàòåðèè::Machine specs::Fract::AGG::ÍåÑåÑúðäè×îâå÷å::Baileys::blog::YouTube channel
-
2nd August 2010 17:56 #3Registered User
Join Date: Aug:2008
Location: Varna
Posts: 291
Ùîì ñà ÷àñòíè çíà÷è íå ñà ïðåäâèäåíè çà íàñëåäÿâàíå. Àêî òúðñèø êà÷åñòâî -ðåäèçàéí + ÷àñòè÷íî ïðåíàïèñâàíå. Èçíàñÿíå íà âúçìîæíîñòèòå êîèòî òè ëèïñâàò â protected ìåòîäè, åòö. Òàêà å ïî-äîáðå â ïåðñïåêòèâà.
-
2nd August 2010 18:41 #4
Ìäà, à ùîì íå ñà ïðåäâèäåíè çà íàñëåäÿâàíå, ìîæå â ìåòîäèòå äà ñå ðàç÷èòà äà íå áúäàò áàðàíè îò íÿêîé íàñëåäíèê.. Òîé êîäúò ìîæå è äà ñå êîìïèëèðà, íî â ïåðñïåêòèâà ìîæå äà ïðè÷èíè òðóäíîîòêðèâàåìè áúãîâå. Åäèí îò ðèñêîâåòå íà îòâîðåíèÿ êîä
. Âñåêè ìàæå, êúðïè..
-
2nd August 2010 19:21 #5
Åìè òîé ïè÷à äåòî ãî å ïèñàë ìíîãî ïðîôåñèîíàëíî ãî å íàïðàâèë, íî ïî÷òè âñè÷êî å private. Öåëòà íà îðèãèíàëíèÿ ïðîåêòà å áèëà äà ñòàíå ïðîñòî è åôåêòèâíî, òà çàòîâà ðàçøèðÿåìîñòòà å çàìèñëåíà äà å ñàìî ïðåç ïëúãèíè è òî çà íÿêàêâè äðåáíè íåùà. À àç ïðàâÿ èçöÿëî íîâà ïðîãðàìà, ñ ïî-ðàçëè÷íà íàñî÷åíîñò - âñå åäíî àøëàäèñâàì ôèäàíêà íà áàçàòà íà äðóãî äúðâî.
Ãîëÿìî ìàçàíå è êúðïåíå íå ñå ïîëó÷è, ïî÷òè íå ïèïàì áàçîâèòå êëàñîâå ñ èçêëþ÷åíèå íà òèÿ private.
Ìåðñè çà ñúâåòèòå!MSI B450 Gaming Pro Carbon AC | Ryzen 9 5900x | HyperX Predator 3200 | Asus Strix 3090 | Kingston KC3000 2TB | WD Red 4TB | Dell G3223Q + LG 27UK650-W | Arctic Freezer 360 | Seasonic Focus GX-1000 | Lian Li O11 Dynamic
-
2nd August 2010 20:39 #6
Ïðîáëåìúò íà òâîÿ ïîäõîä å, ÷å â åäèí õóáàâ ñëúí÷åâ äåí, íÿêúäå â (íå äîòàì) áëèçêîòî áúäåùå, àêî ðåøèø äà èíòåãðèðàø íàïðàâåíèòå äî íåãî ïðîìåíè â îðèãèíàëíèÿ ïðîåêò (òîé ñå ðàçâèâà âñå îùå, ïðåäïîëàãàì?) â òâîÿòà âåðñèÿ íà êîäà, òî ùå èìàø íåïðèÿòíè òðóäíîñòè ïðè ñëèâàíåòî.
Internet - it doesn't make you stupid, it just makes your stupidity more accessible to others
-
2nd August 2010 22:51 #7
Äà, òî÷íî ïî òîâà ìèñëÿ îùå îò ñàìîòî íà÷àëî, ùîòî ñúì íàÿñíî ñ "ïîäâîäíèòå êàìúíè". Êàòî ïî÷íàõ áÿõ íà v2.1 ñåãà å 2.4. Äàæå ñè èçìèñëèõ ìîè òàãîâå, êîèòî ìàðêèðàò ìåñòàòà íà ïðîìåíèòå è âèäà íà ïðîìÿíàòà. Àêî íå ñòàíå úïãðåéäâàíåòî, ùå ñå çàêîòâÿ íà âåðñèÿòà îò íà÷àëîòî. Êàêâî äà ñå ïðàâè. Íà ïðàêòèêà íîâèòå âåðñèè íå ïðåäëàãàò êîé çíàå êàêâè ïîäîáðåíèÿ, ÷å äà ñè ñòðóâà ðèñêà ïî úïãðåéäâàíåòî.
MSI B450 Gaming Pro Carbon AC | Ryzen 9 5900x | HyperX Predator 3200 | Asus Strix 3090 | Kingston KC3000 2TB | WD Red 4TB | Dell G3223Q + LG 27UK650-W | Arctic Freezer 360 | Seasonic Focus GX-1000 | Lian Li O11 Dynamic
-
2nd August 2010 22:59 #8Registered User
Join Date: Dec:2007
Location: Sofia
Posts: 366
Ñïîðåä ìåí ñè òðúãíàë â íàïúëíî ïîãðåøíà ïîñîêà. Ìîÿò ñúâåò å äà èçòðèåø âñè÷êè ïðîìåíè, êîèòî ñè íàïðàâèë äî ìîìåíòà, è äà çàïî÷íåø îòíà÷àëî (ñ ïîâå÷å ìèñëåíå è ïî-ìàëêî search/replace).
Ñúùåñòâóâàù êîä/êëàñîâå ìîãàò äà ñå èçïîëçâàò è áåç äåôèíèðàíå íà äúùåðíè êëàñîâå. Âñúùíîñò äåôèíèðàíåòî íà äúùåðíè êëàñîâå/íàñëåäÿâàíå òðÿáâà äà å åäíà îò ïîñëåäíèòå òè îïöèè. (Ãîâîðÿ ïî ïðèíöèï, à íå êîíêðåòíî çà òàçè ïðîãðàìà, òúé êàíî íå ñúì è ðàçãëåæäàë êîäà).
Òîâà, ÷å ðàçøèðÿåìîñòòà íà òåêóùàòà âåðñèÿ å çàìèñëåíà äà ñòàâà ñàìî ñ ïëúãèíè, íå å ëîøî, äàæå íàïðîòèâ. Ìîæå áè ñè ñòðóâà äà ïîëîæèø óñèëèÿ â ïîñîêà óñúâúðøåíñòâàíåòî íà ïëúãèí ìîäåëà, òàêà ÷å ñ íåãîâà ïîìîù äà å âúçìîæíî äîáàâÿíåòî è íà íÿêîè íå òîëêîâà "äðåáíè" íåùà.
-
3rd August 2010 01:25 #9
 òàêàâà ñèòóàöèÿ ùå ñè óëåñíèø æèâîòà, àêî äúðæèø êîäà (è òâîÿ è îðèãèíàëíèÿ) â source control ñèñòåìà, êîÿòî ñå ñïðàâÿ îòëè÷íî ñ branch-âàíå è merge-âàíå. Ïî-êîíêðåòíî - Mercurial, Git, èëè Bazaar. Äî ãîëÿìà ñòåïåí ùå òè àâòîìàòèçèðàò rebase-âàíåòî êúì ïî-íîâà âåðñèÿ íà ïðîåêòà. Òîâà ãî ïèøà, çàùîòî äîêîëêîòî âèæäàì SweetHome3D ñè äúðæàò êîäà â CVS, êîåòî îòêðîâåíî êàçàíî å ìàñà çëå, íå ñàìî â òîâà îòíîøåíèå.Êîéòî ìè öúêíå ÷åðâåí ïàëåö, äà ìó îìåêíå ÷óðêàòà!




Reply With Quote

Lenovo ThinkPad 15 èëè IdeaPad 15
5th May 2023, 22:16 in Ìîáèëíè êîìïþòðè