zadanie,losowanie bez powtorzen

Question

Witam robię zadanie z kursu programowania na cpp0x które brzmi następująco:

"1. Napisz program, który wczyta 3 liczby podane przez użytkownika do tablicy, a następnie wylosuje 2 z nich bez powtórzeń. Wynik wypisz na ekran. Postaraj się napisać ten program w oparciu o funkcje"

Na razie napisałem taki kod który o dziwo działa i mam pytanie czy da się to zrobić jakoś lepiej? Bo wiem że ten mój sposób to tak na skróty i w praktyce nieprzydatny a nie mogłem wpaść na inny pomysł.

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <windows.h>

using namespace std;


int main()
{
    int tablica[3];
    cout << "Podaj 3 liczby: " << endl;
    int i=0;
    while(i<3)
        {
        cin>>tablica[i];
        i++;
        }
    srand(time(NULL));
    int losulosu1=rand()%3+1;
    int losulosu2=rand()%3+1;
    while (losulosu1==losulosu2)
       losulosu2=rand()%3+1;


    switch(losulosu1)
    {
    case 1: cout<<"Wylosowana liczba: "<<tablica[0]<<endl; break;
    case 2: cout<<"Wylosowana liczba: "<<tablica[1]<<endl; break;
    case 3: cout<<"Wylosowana liczba: "<<tablica[2]<<endl; break;
    }
    switch(losulosu2)
    {
    case 1: cout<<"Wylosowana liczba: "<<tablica[0]<<endl; break;
    case 2: cout<<"Wylosowana liczba: "<<tablica[1]<<endl; break;
    case 3: cout<<"Wylosowana liczba: "<<tablica[2]<<endl; break;
    }


    return 0;
}

 

Comments

ten switch w ogóle niepotrzebny jest

szybciej by było wypisać zawartość używając poleceń: tablica[losulosu1] i tablica[losulosu2]

---

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <windows.h>

using namespace std;





int main()
{
    int tablica[3];
    cout << "Podaj 3 liczby: " << endl;
    int i=0;
    while(i<3)
        {
        cin>>tablica[i];
        i++;
        }
    srand(time(NULL));
    int losulosu1=rand()%3+1;
    int losulosu2=rand()%3+1;
    while (losulosu1==losulosu2)
       losulosu2=rand()%3+1;

    cout<<"Wylosowana liczba: "<<tablica[losulosu1];
    cout<<"Wylosowana liczba: "<<tablica[losulosu2];



    return 0;
}

 

---

Tak?

---

tak, tylko w tej drugiej dodanej linii losolosu2 powinno byc

---

Ok dzięki

Answer 1

Wczytywanie elementów tablicy zwykle robi się for'em, tak jak ty zrobiłeś wygląda to strasznie dziwnie - jest poprawne ale for zmieściłby się w 2 linijkach i byłby czytelniejszy, poza tym te switche są kompletnie niepotrzebne, tylko przeszkadzają, po prostu zamiast losować liczbę 1,2 albo 3 wylosuj 0,1 albo 2 - czyli indeks tablicy a potem to wypisz - zajmie 2 linijki. Samo losowanie jest ok i nie ma za bardzo co kombinować z innymi metodami unikania powtórzeń dla 3 liczb.