Tablice - zle liczy

Question

Witam,

 
 

#include <bits/stdc++.h>

using namespace std;


int main()
{

    int m, n, hasze=0,ruchy=0, najwiekszaW, najwiekszaK;
    cin >> m >> n;
    char plansza [m+1] [n+1];
    int wiersz=1, kolumna=1;
    int haszeW [m+1] = {};
    int haszeK [m+1] = {};
    for (int i=1; i<=m; i++ )
    {
        for (int i=1; i<=n; i++)
        {
            cin >> plansza [wiersz][i];
            if (plansza [wiersz][i] == '#'){
                haszeW[wiersz]++;
                hasze++;
                }
        }
        wiersz++;
    }

    for (int i=1; i<=n; i++)
    {
        for (int i=1; i<=m; i++)
        {
            if (plansza [i][kolumna] == '#'){
                haszeK[kolumna]++;

            }
       }
       kolumna++;
    }
// Szuka najwiekszego wiersza
    najwiekszaW = haszeW[1];
    for (int i=1; i<=m; i++)
    {
        if (haszeW[i] > najwiekszaW)
            najwiekszaW = haszeW[i];
    }
// Szuka najwieksza kolumne
    najwiekszaK= haszeK[1];
    for (int i=1; i<=m; i++)
    {
        if (haszeK[i] > najwiekszaK)
            najwiekszaK = haszeK[i];
    }

    cout <<najwiekszaK << " " << najwiekszaW << " " << hasze;


    return 0;
}

 

Kod pokazuje zły wynik "NajwiekszaW", lecz jeżeli usunę haszeK[kolumna]++ to będzie działać poprawnie

Comments

Kompletnie nie mam pojęcia co ten program ma robić (może podaj założenia), ale widzę kilka dziwnych kontrukcji.

1. Dziwnie ideksujesz tablice - od 1, ale kompensujesz to dająć o 1 więcej pozycji. OK, tylko trochę szkoda pamięci.

2. Masz macierz m * n i wektory - domyślam się że kolumn i wierszy.  Tylko dlaczego oba wektory mają taki sam rozmiar - chyba jeden powinnien mieć m a drugi n?

3. Dwie pętle, jedna w drugiej i obie indeksowana tak samo nazwaną zmienną - może to zadziała, ale niedostępna będzie zewnętrzny indeks. Załatwiasz to dodatkową zmienną, ale ogólnie to nieczytelne.

3. Dziwnie je zapełniasz - jak napotkasz # to przechodzisz do następnej kolumny/wierszy, ale nie 'zerujesz' drugiego indeksu, wiec będzie coś takiego jak wydruk z LF bez CR

1234567890123
             abcdefhg