क्या इस जावा कोड में कोई गलती है?

Question

class Creature {  
    private int yearOfBirth=10; 

    public void setYearOfBirth(int year) { 
     yearOfBirth = year; 
    } 

    void setYearOfBirth(Creature other) { 
     yearOfBirth = other.yearOfBirth; // is this correct it compiles fine 
    } 

    int getYearOfBirth() { 
     return yearOfBirth; 
    } 

    public static void main(String args[]) 
    { 
     Creature c = new Creature(); 
     c.setYearOfBirth(89); 

     Creature d = new Creature(); 
     c.setYearOfBirth(d); 

     System.out.println(c.yearOfBirth); 
    } 
} 

क्या इस कोड में कोई गलती है?

"अन्य .yearOfBirth" गलत है? मेरे संकाय का कहना है कि यह गलत है लेकिन यह मेरे लिए ठीक काम करता है।

Answer 1

आपको क्यों समझाने के लिए अपने संकाय से पूछना है कि उन्हें लगता है कि यह गलत है (शायद यह शैली का सवाल है, या यहां तक ​​कि एक गलतफहमी), ताकि आप इससे सीख सकें।

आखिरकार यह व्यक्ति आपके ग्रेड को प्रभावित करेगा। यह उनके साथ सकारात्मक बातचीत करने का एक शानदार अवसर है। जितना अधिक शामिल होगा आपके शिक्षक आपको व्यक्तिगत रूप से पढ़ाने के साथ हैं, उतना ही बेहतर होगा कि आपका विषय मास्टरिंग करने का अवसर होगा।

यदि दूसरी तरफ आपको बताया जाता है कि आप कुछ गलत हैं तो आप निजी तौर पर दूर जाते हैं और सामान्य इंटरनेट समुदाय से पूछते हैं, आपको एक जोखिम है कि आपको बताया जाएगा कि आप सही हैं और आप समाप्त हो जाएंगे आपके शिक्षकों पर श्रेष्ठता की झूठी भावना जो बहुत प्रतिकूल होगी।

Answer 2

यह गलत है क्योंकि आप किसी अन्य ऑब्जेक्ट के किसी निजी सदस्य (आपने private int yearOfBirth घोषित किया है) का उपयोग कर रहे हैं, हालांकि कक्षा का प्रकार समान है। आपको इसके बजाय परिभाषित सार्वजनिक गेटटर का उपयोग करना चाहिए: yearOfBirth = other.getYearOfBirth()

Answer 3

yearofBirth एक निजी int है। इसलिए other.yearOfBirth पर कॉल संभवतः असफल हो जाएगा ...

Answer 4

जैसा लिखा गया है, यह आपके द्वारा खोजे जाने पर काम करेगा। मुझे संदेह है कि खेल में मौलिक गलतफहमी है, हालांकि।

मेरे मानसिक शक्तियों ने मुझे बताया कि अपने प्रशिक्षक निम्नलिखित की तरह कोड अधिक की उम्मीद:

class Creature {  
    private int yearOfBirth=10; 

    public void setYearOfBirth(int year) { 
     yearOfBirth = year; 
    } 

    public void setYearOfBirth(Creature other) { 
     yearOfBirth = other.yearOfBirth; 
    } 

    public int getYearOfBirth() { 
     return yearOfBirth; 
    } 
} 

class Program { 
    public static void main(String args[]) { 
     Creature c = new Creature(); 
     c.setYearOfBirth(89); 

     Creature d = new Creature(); 
     c.setYearOfBirth(d); 

     System.out.println(c.yearOfBirth); // This will not compile 
    } 
} 

गलतफहमी है कि आप केवल एक बना लिया है class-- अपने मुख्य आवेदन वर्ग। यह प्रभावी रूप से yearOfBirth एक प्रकार का संकर वैश्विक मूल्य बनाता है जिसे आप अपनी मुख्य विधि से एक्सेस कर सकते हैं। अधिक विशिष्ट डिजाइनों में, Creature एक ऐसी कक्षा है जो आपकी मुख्य विधि से पूरी तरह से स्वतंत्र है। उस स्थिति में, आपको केवल Creature को public इंटरफ़ेस के माध्यम से एक्सेस करना होगा। आप सीधे अपने निजी क्षेत्र तक पहुंच नहीं पाएंगे।

---

(वहाँ किसी भी pedants के लिए नोट: हाँ, मैं जानता हूँ कि मैं सरल बनाने रहा हूँ।)

Answer 5

नहीं है, वहाँ कोई समस्या नहीं बिल्कुल भी इसके साथ है।

देखो, यह दर्शक राय पर निर्भर करता है। लेकिन किसी दिए गए संदर्भ के लिए यह कोड बिल्कुल सही हो सकता है।

किसी अन्य संदर्भ के लिए यह सही नहीं हो सकता है। तो यह इस बात पर निर्भर करता है कि कैसे उपयोग किया जा रहा है।

• एक निजी सदस्य को एक्सेस करना एक और उदाहरण से सीधे, सही नहीं है (हमेशा वांछनीय हालांकि, उदाहरण के लिए जब आप उपवर्गीकरण रहे हैं के लिए) यही कारण है कि यह पहली जगह में private है है।आप कह रहे हैं "अरे, यह मेरा है और मुझे पता है कि इसका उपयोग कैसे करें"

• अन्य दो तरीकों के लिए default एक्सेस संशोधक का उपयोग करके, आपका इरादा है कि उन्हें पैकेज के बाहर अन्य कक्षाओं द्वारा उपयोग नहीं किया जाना चाहिए।

शायद एकमात्र चीज जो मैं जोड़ूंगा वह कक्षा को अंतिम बनाना है।

final class Creature 

आप इसे दाय आप शायद yearOfBirth विशेषता के लिए प्राप्त/सेट की समीक्षा करने के लिए है बनाना चाहते हैं, लेकिन वे जिस तरह से यह है मेरे लिए एकदम सही है।

अब सबसे महत्वपूर्ण यहाँ बात है, कि आप समझ क्या अपने कोड के प्रत्येक भाग करता है, और यह कैसे अपने व्यवहार को प्रभावित करता है।

आपको भाग्य द्वारा कोई कोड नहीं चाहिए (क्षमा करें मुझे नहीं पता कि इसके लिए सही अभिव्यक्ति क्या है) लेकिन आपको पता होना चाहिए कि जब भी आप कुछ टाइप करते हैं तो आप क्या कर रहे हैं, और आप कैसे उपयोग करना चाहते हैं ।

Answer 6

मैं दो "मुद्दों," देख, हालांकि मैं उन्हें कॉल करने में संकोच गलतियों:

1. आप स्पष्ट रूप से 89 के रूप में प्राणी ग की उम्र स्थापित कर रहे हैं, और फिर अप्रारंभीकृत डिफ़ॉल्ट के साथ उस उम्र पुनर्लेखन (!) जीव डी। यदि आप यही करना चाहते हैं, तो ठीक है, लेकिन कम से कम आप कुछ चक्रों को बर्बाद कर रहे हैं ताकि आप उस मूल्य को निर्धारित कर सकें जिसे आप बाद में फेंकना चाहते हैं।

2. आप संभावित रूप से जावाबीन नामकरण सम्मेलनों का उल्लंघन कर रहे हैं।

उत्तरार्द्ध बिंदु व्याख्या करने के लिए: जावा पुस्तकालयों और चौखटे (विशेष रूप से JSP) का एक बहुत एक घटक के रूप वस्तु के इलाज के लिए JavaBeans पर भरोसा करते हैं। मैंने उपयोग की जाने वाली वास्तविक तंत्र में गहराई से डाला नहीं है, लेकिन मैंने जो पढ़ा है, उससे यह गुणों और उन गुणों के प्रकारों का अनुमान लगाने के लिए जावाबींस कक्षा के आत्मनिरीक्षण पर निर्भर करता है। एक इंट और एक प्राणी दोनों को स्वीकार करने के लिए अपने सेटर सेट YearOfBirth() को अधिभारित करके, आप आत्मनिरीक्षण को फेंक सकते हैं। (Java12eans के लिए एक सभ्य परिचय के लिए here देखें।)

यह एक बड़ा सौदा नहीं है - यह पूरी तरह से संभव है कि आप इस वर्ग का उपयोग जावाबीन के रूप में नहीं करेंगे, और यदि आप इसे रिफैक्टर करने के लिए तुच्छ हैं तो यह काम करता है । लेकिन अपने शिक्षकों शायद निम्नलिखित की तरह एक छोटे से क्लीनर कुछ पसंद करेंगे,:

class Creature {  
    private int yearOfBirth=10; 

    public void setYearOfBirth(int year) { 
     yearOfBirth = year; 
    } 

    int getYearOfBirth() { 
     return yearOfBirth; 
    } 

    public static void main(String args[]) 
    { 
     Creature c = new Creature(); 
     c.setYearOfBirth(89); 

     Creature d = new Creature(); 
     c.setYearOfBirth(d.getYearOfBirth()); 

     System.out.println(c.getYearOfBirth()); 
    } 
} 

अब yearOfBirth तक आपकी पहुंच के सभी सार्वजनिक गेटर विधियों के माध्यम से आता है, जो कैप्सूलीकरण मदद करता है, और अपने मुख्य यदि तोड़ने से कोड पाएगा विधि दूसरे वर्ग में जाती है। (ग्रेग डी के रूप में सही ढंग से बताया।)

इसके अलावा, यह आपके कोड स्पष्ट है, जो अधिक से अधिक महत्वपूर्ण हो जाता है जब आप दूसरों को बनाए रखने और संशोधित करने के लिए के लिए कोड लिखना शुरू की आशय बनाने का अतिरिक्त लाभ है।

Answer 7

मुझे कुछ भी गलत नहीं पता है।

कोड काम करता है, क्योंकि एक उदाहरण या कक्षा एक ही कक्षा के अन्य उदाहरणों के निजी सदस्यों तक पहुंच सकती है। यह डिजाइन द्वारा है।