Zsumuj liczby parzyste i nieparzyste tablicy mxn. [Java]

Question

Cześć, na zajęciach dostałem do zrobienia zadanie o takiej treści: "Napisz program, który utworzy tablicę dwuwymiarową 5x6 i wypełni ją losowymi liczbami całkowitymi z przedziału [0,9]. Następnie program powinien wyświetlić informację o ilości liczb parzystych i nieparzystych znajdujących się w tablicy, a także policzyć sumę liczb parzystych oraz średnią liczb nieparzystych." Udało mi się napisać kod, znaczy tak mi się wydawało, ponieważ gdy kompiluje ten program, w wyniku sum brakuje zazwyczaj 5 liczb, zależy od wylosowanych. Wydaję mi się, że program może nie zliczać 0 w przypadku liczb parzystych, ale nie wiem co z liczbami nieparzystymi i jak to rowiązać. 

import java.util.Random;
import java.util.Scanner;
public class zad4_4
{
    public static void main(){
        Random los = new Random();
        Scanner in = new Scanner(System.in);
        int[][] A = new int[5][6];
        int sumaParzystych = 0;
        int sumaNieparzystych = 0;
        int suma = 0;
        double avr = 0.0;
        int temp = 0;
        for(int i=0; i<A.length; i++){ 
            for(int j=0;j<A[i].length;j++){
                A[i][j] = los.nextInt(10);
            }
        }
        for(int i=0; i<A.length; i++){ 
            for(int j=0;j<A[i].length;j++){
                System.out.format("%4d",A[i][j]);
            }
            System.out.println();
        }
        for(int i=0; i<A.length;i++){
            for(int j=0; j<A.length; j++){ 
                if(A[i][j] % 2 == 0){
                    sumaParzystych = sumaParzystych + 1;
                    suma = suma + A[i][j];
                }else{
                    sumaNieparzystych = sumaNieparzystych + 1;
                    temp = temp + A[i][j];
                }
            }
        }
        for(int i=0; i<A.length;i++){
            for(int j=0; j<A.length; j++){ 
                if(A[i][j] % 2 != 0){
                    avr = (double)temp / (double)sumaNieparzystych;
                }
            }
        }
        System.out.println("Liczb parzystych: "+sumaParzystych);
        System.out.println("Liczb nieparzystych: "+sumaNieparzystych);
        System.out.println("Suma liczb parzystych: "+suma);
        System.out.println("Średnia liczb nieparzystych: "+avr);
    }
}

 

Answer 1

Tu masz błąd linia 26

masz

        for(int i=0; i<A.length;i++){
            for(int j=0; j<A.length; j++){ 
                if(A[i][j] % 2 == 0){

powinno być

        for(int i=0; i<A.length;i++){
            for(int j=0; j<A[i].length; j++){ 
                if(A[i][j] % 2 == 0){

 

 

 

P.S Można ograniczyć ilość tych for-ów  

propozycja

import java.util.Random;

public class zad4_4
{
	public static void main() {
        Random los = new Random();
        // utworzy tablicę dwuwymiarową 5x6
        int[][] A = new int[5][6]; 
        int ileParzystych = 0;
        int ileNieparzystych = 0;
        int sumaParzystych = 0;
        int sumaNieparzystych = 0;
        
        for(int i=0; i<A.length; i++){ 
            for(int j=0;j<A[i].length;j++){
                // wypełni ją losowymi liczbami całkowitymi z przedziału [0,9]
                A[i][j] = los.nextInt(10);
                
                System.out.format("%4d",A[i][j]);
                
                if(A[i][j] % 2 == 0){
                    ileParzystych++; // ilości liczb parzystych
                    sumaParzystych += A[i][j]; // policzyć sumę liczb parzystych
                }else{
                    ileNieparzystych++; // ilości liczb nieparzystych
                    sumaNieparzystych += A[i][j];
                }               
            }
            System.out.println();
        }

        System.out.println("Liczb parzystych: " + ileParzystych);
        System.out.println("Liczb nieparzystych: " + ileNieparzystych);
        System.out.println("Suma liczb parzystych: " + sumaParzystych);
        System.out.println("Średnia liczb nieparzystych: " + (double)(sumaNieparzystych / ileNieparzystych));
    }
}

 

Answer 2

Do jakiego typu sumujesz liczby zmiennoprzecinkowe? Kodu dalej nie analizuję, to pierwsza rzecz która rzuciła mi się w oczy.

Comments

Jedyne liczby zmiennoprzecinkowe jakie mogą się pojawić w tym programie to wynik średniej, który jest już jako double.

---

ale kolega dobrze Ci radzi... uzywać chcesz jako wynik podwójnej precyzji po co inta rzutujesz do podwójnej precyzji skoro integer nie jest zmienno przecinkowy 

  avr = (double)temp / (double)sumaNieparzystych;

sumaNieparzystych -> suma nie bedzie zmiennoprzecinkowa ani zmienna temp