Results 1 to 9 of 9

Hybrid View

Previous Post Previous Post   Next Post Next Post
  1. #1
    1+1=11 svr's Avatar
    Join Date: Sep:2006
    Location: Inferno, Act 1
    Posts: 790

    Ìàñîâà çàìÿíà íà 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

  2. #2
    ɐ-əpoɔᴉu⋂ ɐí íəÔ anrieff's Avatar
    Join Date: Apr:2004
    Location: Sofia
    Posts: 8,448
    Ñìåíÿé ñìåëî. Ùîì "îõëàáâàø" êîíòðîëà, íå áè òðÿáâàëî äà ñðåùíåø ïðîáëåì.
    Åòî òàêà ñå ïëîäÿò òå, êàçâàì ñè àç. Ïîä íîñà òè.
    "640K îught to be enough for anybody" - Bill Gates, 1981
    Òåñòîâå íà áàòåðèè::Machine specs::Fract::AGG::ÍåÑåÑúðäè×îâå÷å::Baileys::blog::YouTube channel

  3. #3
    Registered User
    Join Date: Aug:2008
    Location: Varna
    Posts: 291
    Ùîì ñà ÷àñòíè çíà÷è íå ñà ïðåäâèäåíè çà íàñëåäÿâàíå. Àêî òúðñèø êà÷åñòâî -ðåäèçàéí + ÷àñòè÷íî ïðåíàïèñâàíå. Èçíàñÿíå íà âúçìîæíîñòèòå êîèòî òè ëèïñâàò â protected ìåòîäè, åòö. Òàêà å ïî-äîáðå â ïåðñïåêòèâà.

  4. #4
    Registered User tedych's Avatar
    Join Date: Nov:2003
    Location: Ïëîâäèâ
    Posts: 17,654
    Ìäà, à ùîì íå ñà ïðåäâèäåíè çà íàñëåäÿâàíå, ìîæå â ìåòîäèòå äà ñå ðàç÷èòà äà íå áúäàò áàðàíè îò íÿêîé íàñëåäíèê.. Òîé êîäúò ìîæå è äà ñå êîìïèëèðà, íî â ïåðñïåêòèâà ìîæå äà ïðè÷èíè òðóäíîîòêðèâàåìè áúãîâå. Åäèí îò ðèñêîâåòå íà îòâîðåíèÿ êîä . Âñåêè ìàæå, êúðïè..

  5. #5
    1+1=11 svr's Avatar
    Join Date: Sep:2006
    Location: Inferno, Act 1
    Posts: 790
    Åìè òîé ïè÷à äåòî ãî å ïèñàë ìíîãî ïðîôåñèîíàëíî ãî å íàïðàâèë, íî ïî÷òè âñè÷êî å 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

  6. #6
    philosophus duratea icaci's Avatar
    Join Date: Oct:2006
    Location: Aachen
    Posts: 2,698
    Ïðîáëåìúò íà òâîÿ ïîäõîä å, ÷å â åäèí õóáàâ ñëúí÷åâ äåí, íÿêúäå â (íå äîòàì) áëèçêîòî áúäåùå, àêî ðåøèø äà èíòåãðèðàø íàïðàâåíèòå äî íåãî ïðîìåíè â îðèãèíàëíèÿ ïðîåêò (òîé ñå ðàçâèâà âñå îùå, ïðåäïîëàãàì?) â òâîÿòà âåðñèÿ íà êîäà, òî ùå èìàø íåïðèÿòíè òðóäíîñòè ïðè ñëèâàíåòî.
    Internet - it doesn't make you stupid, it just makes your stupidity more accessible to others

  7. #7
    1+1=11 svr's Avatar
    Join Date: Sep:2006
    Location: Inferno, Act 1
    Posts: 790
    Quote Originally Posted by icaci View Post
    Ïðîáëåìúò íà òâîÿ ïîäõîä å, ÷å â åäèí õóáàâ ñëúí÷åâ äåí, íÿêúäå â (íå äîòàì) áëèçêîòî áúäåùå, àêî ðåøèø äà èíòåãðèðàø íàïðàâåíèòå äî íåãî ïðîìåíè â îðèãèíàëíèÿ ïðîåêò (òîé ñå ðàçâèâà âñå îùå, ïðåäïîëàãàì?) â òâîÿòà âåðñèÿ íà êîäà, òî ùå èìàø íåïðèÿòíè òðóäíîñòè ïðè ñëèâàíåòî.
    Äà, òî÷íî ïî òîâà ìèñëÿ îùå îò ñàìîòî íà÷àëî, ùîòî ñúì íàÿñíî ñ "ïîäâîäíèòå êàìúíè". Êàòî ïî÷íàõ áÿõ íà 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

  8. #8
    Registered User
    Join Date: Dec:2007
    Location: Sofia
    Posts: 366
    Ñïîðåä ìåí ñè òðúãíàë â íàïúëíî ïîãðåøíà ïîñîêà. Ìîÿò ñúâåò å äà èçòðèåø âñè÷êè ïðîìåíè, êîèòî ñè íàïðàâèë äî ìîìåíòà, è äà çàïî÷íåø îòíà÷àëî (ñ ïîâå÷å ìèñëåíå è ïî-ìàëêî search/replace).

    Ñúùåñòâóâàù êîä/êëàñîâå ìîãàò äà ñå èçïîëçâàò è áåç äåôèíèðàíå íà äúùåðíè êëàñîâå. Âñúùíîñò äåôèíèðàíåòî íà äúùåðíè êëàñîâå/íàñëåäÿâàíå òðÿáâà äà å åäíà îò ïîñëåäíèòå òè îïöèè. (Ãîâîðÿ ïî ïðèíöèï, à íå êîíêðåòíî çà òàçè ïðîãðàìà, òúé êàíî íå ñúì è ðàçãëåæäàë êîäà).

    Òîâà, ÷å ðàçøèðÿåìîñòòà íà òåêóùàòà âåðñèÿ å çàìèñëåíà äà ñòàâà ñàìî ñ ïëúãèíè, íå å ëîøî, äàæå íàïðîòèâ. Ìîæå áè ñè ñòðóâà äà ïîëîæèø óñèëèÿ â ïîñîêà óñúâúðøåíñòâàíåòî íà ïëúãèí ìîäåëà, òàêà ÷å ñ íåãîâà ïîìîù äà å âúçìîæíî äîáàâÿíåòî è íà íÿêîè íå òîëêîâà "äðåáíè" íåùà.

  9. #9
    Ïðèìèòèâ Pesho®'s Avatar
    Join Date: Nov:2001
    Location: Sofia
    Posts: 5,169
    Quote Originally Posted by icaci View Post
    Ïðîáëåìúò íà òâîÿ ïîäõîä å, ÷å â åäèí õóáàâ ñëúí÷åâ äåí, íÿêúäå â (íå äîòàì) áëèçêîòî áúäåùå, àêî ðåøèø äà èíòåãðèðàø íàïðàâåíèòå äî íåãî ïðîìåíè â îðèãèíàëíèÿ ïðîåêò (òîé ñå ðàçâèâà âñå îùå, ïðåäïîëàãàì?) â òâîÿòà âåðñèÿ íà êîäà, òî ùå èìàø íåïðèÿòíè òðóäíîñòè ïðè ñëèâàíåòî.
    Quote Originally Posted by svr View Post
    Äà, òî÷íî ïî òîâà ìèñëÿ îùå îò ñàìîòî íà÷àëî, ùîòî ñúì íàÿñíî ñ "ïîäâîäíèòå êàìúíè". Êàòî ïî÷íàõ áÿõ íà v2.1 ñåãà å 2.4. Äàæå ñè èçìèñëèõ ìîè òàãîâå, êîèòî ìàðêèðàò ìåñòàòà íà ïðîìåíèòå è âèäà íà ïðîìÿíàòà. Àêî íå ñòàíå úïãðåéäâàíåòî, ùå ñå çàêîòâÿ íà âåðñèÿòà îò íà÷àëîòî. Êàêâî äà ñå ïðàâè. Íà ïðàêòèêà íîâèòå âåðñèè íå ïðåäëàãàò êîé çíàå êàêâè ïîäîáðåíèÿ, ÷å äà ñè ñòðóâà ðèñêà ïî úïãðåéäâàíåòî.

     òàêàâà ñèòóàöèÿ ùå ñè óëåñíèø æèâîòà, àêî äúðæèø êîäà (è òâîÿ è îðèãèíàëíèÿ) â source control ñèñòåìà, êîÿòî ñå ñïðàâÿ îòëè÷íî ñ branch-âàíå è merge-âàíå. Ïî-êîíêðåòíî - Mercurial, Git, èëè Bazaar. Äî ãîëÿìà ñòåïåí ùå òè àâòîìàòèçèðàò rebase-âàíåòî êúì ïî-íîâà âåðñèÿ íà ïðîåêòà. Òîâà ãî ïèøà, çàùîòî äîêîëêîòî âèæäàì SweetHome3D ñè äúðæàò êîäà â CVS, êîåòî îòêðîâåíî êàçàíî å ìàñà çëå, íå ñàìî â òîâà îòíîøåíèå.
    Êîéòî ìè öúêíå ÷åðâåí ïàëåö, äà ìó îìåêíå ÷óðêàòà!

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  

Copyright © 1999-2011 Õàðäóåð ÁÃ. Âúçìîæíî å ñúäúðæàíèåòî íà òàçè ñòðàíèöà äà å îáåêò íà àâòîðñêè ïðàâà.
iskamPC.com | mobility.BG | Bloody's Techblog | Êðèïòîâàëóòè è ìàéíèíã | 3D Vision Blog | Ìàãàçèí çà åëåêòðîííè öèãàðè