Lepší zápis podmínky – C / C++ – Fórum – Programujte.com

Zdravím,

ve škole jsem měl za úkol napsat podmínku (zbytek programu byl předepsaný), která do dvourozměrného pole znaků o N*N prvcích vepíše hvězdičky na řádky, jejichž index je dělitelný třemi nebo pěti, ale není dělitelný třemi i pěti zároveň. Přišel jsem na 3 způsoby řešení, které vedou ke správnému výsledku, možná i víc, ale teď si vzpomínám jenom na ty 3. Každé z těch řešení bylo označeno za špatné a dostal jsem 0 bodů z 20 možných. Co je na tom špatně a jak byste to napsali vy?

#include <iostream>
using namespace std;

const int N = 8;

char Grid[N][N];

void ClearGrid()
{
    for(int i = 0; i < N; i++)
        for(int j = 0; j < N; j++)
            Grid[i][j] = ' ';
}

void Star(const int Row, const int Column)
{
    if (0 <= Row && Row < N && 0 <= Column && Column < N)
        Grid[Row][Column] = '*';
    else
        cout << "Row or column is out of valid range!" << endl;
}

void PrintGrid()
{
    for(int i = 0; i < N; i++)
    {
        for(int j = 0; j < N; j++)
            cout << Grid[i][j];
            cout << endl;
    }
    cout << endl;
}

void main()
{
    ClearGrid();
    for(int i=0; i<N; i++)
    {
        for(int j=0; j<N; j++)
        {
            if ((i % 3 == 0 || i % 5 == 0) && (i % 3 != 0 || i % 5 != 0)) // moje 1. reseni
            // moje 2. reseni: if ((i % 3 == 0 && i % 5 != 0) || (i % 5 == 0 && i % 3 != 0))
            // moje 3. reseni: if ((i % 3 == 0 || i % 5 == 0) && i % 15 != 0)  
                Star(i, j);
        }
    }
    PrintGrid();
}

Ty si testoval len index i a zabudol si na index j

edit: neviem ako vahu ma u vas 1 bod, ale za to bolo fakt 20 bodov?

if ((i % 3 == 0) ^ (i % 5 == 0))
{
    for(int j=0; j<N; j++)
    {
        Star(i, j);
    }
}

#2 vitamin
Proč bych měl testovat index j? i je řádek a j je sloupec, hvězdičky se mají vypsat na celý řádek, takže pro všechny j

Mozno chceli daco taketo:

if ( (!(i % 3) ^ !(i % 5)) && (!(j % 3) ^ !(j % 5)))Star(i, j);

//alebo:

if ( (!(i % 3) ^ !(i % 5)) || (!(j % 3) ^ !(j % 5)))Star(i, j);

Na skolskych prikladoch je najtazsie zistit co od teba chcu, nie to spravyt.

#3 zlz
To vypadá líp, ale v tom "správném" řešení nemá být ^, mělo by stačit || a &&. Podle rady cvičícího z posledních minut času by to mělo být:
if (i % 3 == 0 || i % 5 == 0 && i % 3 != 0) a stačí správně dopsat závorky. Ty ale nejsem schopný napsat tak, aby to vycházelo správně.