Czemu nic się nie wyświetla. Powinno wypisywać wszystkie możliwości wypisania liczb od 1 do 3 bez powtorzeń.

Question

Wyświetlić powinno coś takiego:

1 2 3

3 2 1

1 3 2

3 1 2

2 1 3

2 3 1

#include <iostream>
#include <ctime>
#include <cstdlib>

using namespace std;

void losbezpow(int tab[], int n)
{
    srand(time(NULL));

    int i, j, x;
    bool jest=false;

    tab[0] = rand() % n + 1;

    for (i = 1; i < n; i++)
    {
        jest = false;
        x = rand() % n + 1;
        for (j = 0; j < i; j++)
        {
            if (x == tab[j])
            {
                jest = true;
                i--;
            }
        }
        if (jest == false)
        {
            tab[i] = x;
        }
    }
}

int main()
{
    int n = 3;
    int tab[n][6];

    for (int i = 0; i < 6; i++)
    {
        losbezpow(tab[i], n);
    }

    for (int i = 0; i < 6; i++)
    {
        for (int j = 0; j < n; j++)
        {
            cout << tab[i][j] << " ";
        }
        cout << endl;
    }

    return 0;
}

 

Answer 1

Twoja tablica ma wymiary 3x6. Twoja pętla losująca dane próbuje używać tablicy 6x3.

    int n = 3;
    int tab[n][6];
 
    for (int i = 0; i < 6; i++)
    {
        losbezpow(tab[i], n);
    }

Dlatego wyskakujesz poza tablicę i piszesz po innej pamięci (stos funkcji main, nadpisujesz zmienne lokalne, w tym zmienną i).

Comments

Nie bardzo rozumiem o co chodzi w tym nawiasie:

(stos funkcje main...zmienną i)

byłbym bardzo wdzięczny gdybyś wytłumaczył.

---

https://en.wikipedia.org/wiki/Stack_buffer_overflow

Answer 2

Odpowiedź już padła, ale mógłbyś wygenerować wszystkie permutacje zbioru {1, 2, 3} (std::next_permutation) i następnie je losowo przemieszać (std::shuffle). Przykład kodu w C++20:

#include <iostream>
#include <algorithm>
#include <vector>
#include <random>

int main() {
    std::vector nums{1, 2, 3};
    std::vector permutations(1, nums);
    while (std::ranges::next_permutation(nums).found) {
        permutations.push_back(nums);                 } 

    std::default_random_engine gen{ std::random_device{}() };
    std::ranges::shuffle(permutations, gen);

    for (const auto& permutation : permutations) {
        for (const auto& num : permutation) {
            std::cout << num << " ";
        }   std::cout << "\n";
    }
}