Tablice wielowymiarowe z lnu c++

Question

System kontroli jakości materiałów wykorzystywanych w przemyśle kosmicznym, dokonuje oceny próbki na podstawie zdjęcia rentgenowskiego. Program bada fragment zdjęcia o rozmiarach 25x25 pikseli i zlicza liczbę pikseli, których jasność odbiega o więcej niż 50% od średniej jasności dla całego badanego obszaru. Takie miejsca mogą świadczyć o nieprawidłowej strukturze materiału.
Tablica unsigned short rtg_picture[25][25] zawiera informacje o jasności wszystkich pikseli w badanym fragmencie zdjęcia. Uzupełnij funkcję int rtg_check( unsigned short rtg_picture[25][25] ) tak, aby zwracała liczbę pikseli, których jasność wyrażona w wartości typu unsigned short odbiega od średniej jasności badanego fragmentu zdjęcia o więcej niż 50%.  

int rtg_check( unsigned short rtg_picture[25][25] )
{
    unsigned short average_brightness = 0;
    int bad_pixel_number = 0;
    int sum = 0;
    int divider=0;
    
    for(int i=0; i<25; i++)
    for(int j=0; j<25; i++)
    {
        sum = sum + rtg_picture[i][j];
        divider=(i+1)*(j+1);
    }
    average_brightness = sum/divider;
    for(int i=0; i<25; i++)
    for(int j=0; j<25; i++)
    {
        if(rtg_picture[i][j] > (average_brightness*1.5) || rtg_picture[i][j] <= (average_brightness/2))
        bad_pixel_number = bad_pixel_number + 1;
    }
    return bad_pixel_number;
}

Kod nie działa nie potrafie znalezc bledu co jest nie tak???

Comments

for(int j=0; j<25; i++) czy for(int j=0; j<25; j++) ?

[18 - 19] Ostatni zły piksel?

---

1, Widzisz. Jak zmienne nazywały by się row (wiersz) i col (kolumna), to prawdopodobieństwo popełnienia błędu będzie mniejsze niż dla abstrakcyjnych i i j.

2. Dlaczego nie wydzielić funkcji obliczającej średnią?

3. Czy nie warto zdefiniować stałej dla liczby 25?

4. Po co za każdym razem liczyć stałą dla dolnego i górnego ograniczenia przedziału? Można to zrobić raz.

5. Do dobrych praktyk w C i C++, należy otaczanie warunków gdzie występują operatory logiczne, nawiasami. Determinuje to jednoznacznie kolejność operacji.

Moja rada na początku (ogólna). Jeśli funkcja ma więcej niż 5 zmiennych których trzeba pilnować/pamiętać/rozumieć czego dotyczą, kwalifikuje się do podziału na kilka funkcji.

BTW. Po co liczysz divider? Przecież przy stałych 25, jest on równy 25 * 25 już na wstępie funkcji.