Як зменшити перемикач у операторі комутатора?

Тож я створюю метод створення лінії вітання для базування двох людей із бази даних.

Існує чотири параметри: два імені ( name1і name2) та два статі ( genderі gender2).

Для кожної гендерної комбінації у мене є різний результат.

Наприклад: якщо гендер 1 є M(чоловік), а також гендер 2 M, результат повинен бути таким, як:

Dear Sir name1 and Sir name2,

Наразі мій перемикач виглядає приблизно так:

switch(gender1){
    case 'M':
        switch(gender2){
            case 'M': printf("Dear Sir %s and Sir %s", name1, name2); break;
            case 'W': printf("Dear Sir %s and Madame %s", name1, name2); break;
            case 'R': ...
        }
        break;
    case 'W':
        switch(gender2){
            case 'M': printf("Dear Madame %s and Sir %s", name1, name2); break
            case 'W': printf("Dear Madame %s and Madame %s", name1, name2); break;
            case 'R': ...
        }
        break;
    case ...etc.
}

Зверніть увагу , що у мене є кілька варіантів гендерних, як 'R'для "Dear Relation"і деякі більш , що у мене немає часу , щоб перевести.

Як я можу зменшити цю операцію подвійного перемикання?

Поставити другий перемикач методом не є варіантом, тому що також є випадок, коли обидва імена однакові, і тоді вихід слід поєднувати так: "Dear Sir and Madame name1,"

Додайте заголовок до параметрів printf:

char* title1;
switch(gender1){
    case 'M':
        title1 = "Sir";
        break;
    case 'W':
       title1 = "Madam";
        break;
    case ...etc.
}
char* title2;
switch(gender2){
    case 'M':
        title2 = "Sir";
        break;
    case 'W':
       title2 = "Madam";
        break;
    case ...etc.
}
printf("Dear %s %s and %s %s", title1, name1, title2, name2);

ви можете витягнути перемикач на власну функцію для повторного використання та компактності.

Радикальне рішення: Дайте користувачеві вказати власну назву (із заздалегідь визначеного списку, який ви надаєте).

Ваше рішення (як дивляться англійськими очима), здається, задовольняє лише лордів ("сер") та дам; більшість чоловіків вважатимуться "містером", більшість жінок - як "міс", "пані" або "пані", залежно від їх сімейного стану та особистої думки. Тоді виникла ціла низка інших почесних діянь, заснованих на професійних рейтингах - "Лікарі", "Професори", "Преподобні" і навіть, якщо ви відчуваєте справжній оптимізм щодо свого сайту, "Святість"!

Більш просте рішення: Вам потрібна функція [одиночний], щоб перевести "гендер" на загальнодоступний. Зашифруйте його один раз і назвіть його обом людям:

printf("Dear %s %s and %s %s" 
   , getTitle( gender1 ), name1 
   , getTitle( gender2 ), name2 
   ) ; 

Заголовки дійсно належать до бази даних, але ви заявили, що не маєте контролю над цим. Ви не вказали мовний тег, але синтаксис є у сімействі C, тож це буде псевдокод, що майже C ++:

map<string, string> titles;
titles.emplace("M", "Sir");
titles.emplace("F", "Madam");

cout << "Dear " << titles[gender1] << " " << name1 << " and "
     << titles[gender2] << " " << name2 << endl;

Користь для цього полягає в тому, що ви закопуєте логіку вибору в структуру даних, а не структуру коду: це схоже на делегування в базу даних і є більш гнучким. Якщо ви зберігаєте цю карту як статичну константу десь, ви майже можете використовувати її як базу даних: вона стає єдиною структурою для оновлення, яку можна використовувати в багатьох місцях у коді, не потрібно писати більше коду.

Відповідь щурячого виродка є досить хорошою ідеєю, якщо речення мають однаковий зразок, але з двома вписами, кожне з яких залежить лише від gender1відповідних gender2.

Відповідь Філа У. - це, мабуть, найбільш гнучка відповідь, оскільки він дозволяє чітко контролювати привітання, хоча він цілком правильний - це радикальна зміна. Ви можете не мати даних у цій формі.

Відповідь Кіліана Фота, мабуть, найкраща на запитання, яке йому задають, хоча він залежить від перемикання на рядок, що може бути неможливим або, мабуть, принаймні дорожчим.

Уточнення відповіді Кіліана обчислює єдине значення з обох входів та перемикання на це:

// Using a macro in C for readability. C++ would use a constexpr function
#define COMBINE(a, b) ((a<<CHAR_BIT)+b)

switch( COMBINE(gender1, gender2)) {
  case COMBINE('M', 'M'): 
    print "Dear Sirs";
    break;
  case COMBINE('M', 'F'): 
  case COMBINE('F', 'M'): 
    print "Dear Sir and Madam";
    break;
  ...
#undef COMBINE

Звичайно, оскільки ви отримуєте всі чотири входи (2 імені та 2 статі) з бази даних, додавання ще однієї таблиці та приєднання до цього, щоб отримати належне вітання, напевно, є більш гнучким і, можливо, простіше, ніж вище.

Якщо ваша мова дозволяє це робити, ви можете писати

switch(gender1+gender2) {
  case "MM": 
    print "Dear Sirs";
    break;
  case "MF": 
  case "FM":
    print "Dear Sir and Madam";
    break;
  ...

Це не обов’язково краще, ніж ваша версія, оскільки все ще є дублювання, але це уникає вкладених switch.

Зазвичай ви хочете, щоб такі рядки інтерфейсу користувача витягувалися з таблиці рядків, а не були жорстко закодовані у вихідному коді, для локалізації та зручності оновлення. Таким чином, я підходив би до використання вхідних даних для побудови ключа пошуку, щоб щось подібне:

var lookupKey = "SALUTATION_" + gender1 + "_" + gender2;
var format = GetLocalizedString(lookupKey);
printf(format, name1, name2);

Інші пропозиції щодо дозволу користувачам вибирати власні заголовки є дійсними, якщо у вас є можливість отримати цю інформацію. Я б все-таки використовував пошук розв'язувальної таблиці в рішенні.