Какой вариант конструктора использовать правильней?

    card(){
    }
    card (deck_1 value_1, deck_2 value_2){
        soilt_1 = value_1;
        soilt_2 = value_2;
    }
 
    ~card(){
    }

card (deck_1 value_1, deck_value2) : soilt_1(value_1), soilt_2(value_2){}

#include <iostream>
#include <cstdlib>
#include <Windows.h>
#include <iomanip>
 
 
enum deck_1 {val_a, val_b, val_c, val_d};
enum deck_2 {jack, queen, king, ace};
 
class card{
    deck_1 soilt_1;
    deck_2 soilt_2;
public:
 
    card(){
    }
    card (deck_1 value_1, deck_2 value_2){
        soilt_1 = value_1;
        soilt_2 = value_2;
    }
 
    ~card(){
    }
 
    void display ();
    bool compare (card);
 
};
 
 
 void card::display (){
 
     switch (soilt_2){
     case jack: std::cout << "Валет "; break;
     case queen: std::cout << "Королева "; break;
     case king: std::cout << "Король "; break;
     case ace: std::cout << "Туз "; break;
 
     }
 
     switch (soilt_1){
     case val_a: std::cout << "червь"; break;
     case val_b: std::cout << "бубен"; break;
     case val_c: std::cout << "треф"; break;
     case val_d: std::cout << "пик"; break;
 
     }
 
 }
 
 bool card::compare (card chose){
     return (chose.soilt_1 == soilt_1 && chose.soilt_2 == soilt_2) ? true : false;
 }
 
 
    int main(){
        SetConsoleCP (1251);
        SetConsoleOutputCP (1251);
 
        short position;
 
        card card_1 (val_d, king);
        card card_2 (val_c, queen);
        card card_3 (val_a, ace);
 
        std::cout << "1) "; card_1.display(); std::cout << std::endl;
        std::cout << "2) "; card_2.display(); std::cout << std::endl;
        std::cout << "3) "; card_3.display(); std::cout << std::endl;
 
        card temp;
        card priz;
        card chos;
 
        priz = card_3; // Эту карту нужно будет найти!
 
        std::cout << "Меняем карту 1 и 2." << std::endl;
        
        temp = card_2;                  // С дополнительным временным объектом меняем местами карты.
            card_2 = card_1;            //
                card_1 = temp;          //
 
        std::cout << "Меняем карту 2 и 3." << std::endl;
 
        temp = card_2;
            card_2 = card_3;
                card_3 = temp;
 
        std::cout << "Меняем карту 1 и 3." << std::endl;
 
            temp = card_1;
                card_1 = card_3;
                    card_3 = temp;
 
        std::cout << "Вам нужно найти карту: "; priz.display(); std::cout << std::endl;
 
        std::cout << "Введите номер карты: "; std::cin >> position;
 
 
        switch(position){
        case 1: chos = card_1; break;       // Присваемваем объекту chos
        case 2: chos = card_2; break;       //  объект с выбранной картой.
        case 3: chos = card_3; break;       //
        }
 
    
        if (chos.compare(priz))             // Передаем в логическую функцию объекта chos, объект priz
            std::cout << "Вы выйграли!" << std::endl;
        else
            std::cout << "Вы проиграли!" << std::endl;
 
        system ("pause");
        return 0;
    };