خطأ في الكود

الطريقة الصحيحة لإنشاء معدل من مجموعة أرقام داخل كلاس في لغة الجافا هي كالتالي :  - 

public class myCalculater{
	int input1 = 10 ; 
  	int input2 = 20 ; 
  	int input3 = 30 ; 
  	int input4 = 40 ; 
  	int input5 = 50 ; 
  	// إقتران الحصول على معدل محموعة من الأرقام 
	Public int getAverage(){
      // حساب مجموع الأرقام من 10 - 50 
    	int sumOfInputs = this.input1 + this.input2 + this.input3 + this.input4 + this.input5 ; 
      // عدد الأرقم و هو في هذه الحالة يساوي 5 
      	int numOfInputs = 5 ;
      // عملية حسابية لحساب المعدل و هي بقسمة المجموع على العدد
      	int avg = sumOfInputs/numOfInputs ;
      // إرجاع قيمة المعدل 
      	return avg ;
    }
}

عند قيامك باستدعاء الإقتران getAvarege() سوف تحصل على معدل الأرقام المعطاه من 10 - 50 

مرحباً علي: 

• لديك خطأ في كلمة Public يجب أن تكون public لأنها كلمة محجوزة.
• أري أنك لا تستفيد بالمتغيرات التي تم تعريفها في البداية فأنت تحسب المتوسط يدوياً  ولكن كنصيحة استعمل array بدل المتغيرات مباشرة 

   public static void main(String arg[])	
   {
     // n = عدد العناصر فإذا أردت زيادة العناصر يجب أن تزيدها
     // result = هو مجموع العناصر ولأننا سنحتاجه في حساب المتوسط 
     // المتوسط = مجموع العناصر / عدد العناصر 
     int n=5,result=0;
   	// هنا إستخدمت مصفوفة لكي أجعل الكود يعمل ديناميكياً ويقبل الزيادة 
     int a[]=new int[5];
   	// هنا أضيف العناصر إلى المصفوفة لإستخدامهم في حساب المجموع لاحقاً
     a[0]=10;
   
     a[1]=20;
   
     a[2]=30;
   
     a[3]=40;
   
     a[4]=50;
  	// هنا أجمع جميع عناصر المصفوفة وأخزنهم في المتغير result
     for(int i=0;i<n;i++)
      result=result+a[i];
     // هنا أحسب المتوسط طبقاً للمعادلة السابقة
     System.out.println("average of  ("+a[0]+","+a[1]+","+a[2]+","+a[3]+","+a[4]+")  is ="+result/n);
   
   }

   

• أما بالنسبة للكود خاصتك فيجب أن تصلح Public إلى public - وتستخدم المتغيرات بداخل الدالة وليس الأرقام مباشرة - وتعدل إسم المتغير الداخلي لكي لا يطابق إسم الدالة average
   

  public class mycalculator{
  
    int input1 =10;
    int input2 =20;
    int input3 =30;
    int input4 = 40;
    int input5 = 50;
  
    public int average(){
      int av =(input1 + input2 + input3 + input4 + input5)/5;
      
      return av;
    }
  
  }