Кресты / Говнокод #21980 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
  21. 21
  22. 22
  23. 23
  24. 24
  25. 25
  26. 26
  27. 27
  28. 28
  29. 29
  30. 30
  31. 31
  32. 32
  33. 33
  34. 34
  35. 35
  36. 36
  37. 37
  38. 38
  39. 39
  40. 40
  41. 41
  42. 42
  43. 43
  44. 44
  45. 45
  46. 46
  47. 47
  48. 48
  49. 49
  50. 50
  51. 51
  52. 52
  53. 53
  54. 54
  55. 55
  56. 56
  57. 57
  58. 58
  59. 59
#include "document_validator_factory.h"

std::unique_ptr<ortax::DocumentValidator>
ortax::DocumentValidatorFactory::create_validator(ortax::DocumentType doc_type)
{
    if (doc_type == DocumentType::RussianPassport) //21
    {
        return std::make_unique<BirthPlaceValidator>(
               std::make_unique<DocIssuerValidator>(
               std::make_unique<SubdivisionCodeValidator>(
               std::make_unique<PassportExpireDateValidator>(
               std::make_unique<RussianPassportValidator>()))));
    }
    else if (doc_type == DocumentType::MilitarySoldierTicket) //4
    {
        return std::make_unique<BirthPlaceValidator>(
               std::make_unique<DocIssuerValidator>(
               std::make_unique<DefaultDocumentValidator>()));
    }
    else if (doc_type == DocumentType::MilitarySoldierTicket) //7
    {
        return std::make_unique<BirthPlaceValidator>(
               std::make_unique<DocIssuerValidator>(
               std::make_unique<DefaultDocumentValidator>()));
    }
    else if (doc_type == DocumentType::SeamanPassport) //26
    {
        return std::make_unique<BirthPlaceValidator>(
               std::make_unique<ExpireDateValidator>(
               std::make_unique<DefaultDocumentValidator>()));
    }
    else if (doc_type == DocumentType::TemporaryIdentityCard) //14
    {
        return std::make_unique<BirthPlaceValidator>(
               std::make_unique<ExpireDateValidator>(
               std::make_unique<DefaultDocumentValidator>()));
    }
    else if (doc_type == DocumentType::ResidensePermitRu) //12
    {
        return std::make_unique<BirthPlaceValidator>(
               std::make_unique<ExpireDateValidator>(
               std::make_unique<DefaultDocumentValidator>()));
    }
    else if (doc_type == DocumentType::ForeignPassport) //10
    {
        return std::make_unique<BirthPlaceValidator>(
               std::make_unique<DocIssuerValidator>(
               std::make_unique<DefaultDocumentValidator>()));
    }
    else if (doc_type == DocumentType::ForeignPassport) //15
    {
        return std::make_unique<ExpireDateValidator>(
               std::make_unique<DefaultDocumentValidator>());
    }
    else
    {
        return std::make_unique<DefaultDocumentValidator>();
    }
}

Мой архитектор использует паттерны

Запостил: Koshak90 Koshak90, (Updated )

Комментарии (18) RSS

  • Заднепроходно воспользовался твоим архитектором, проверь.
    Ответить
  • else if (doc_type == DocumentType::MilitarySoldierTicket) //4
    else if (doc_type == DocumentType::MilitarySoldierTicket) //7
    
    else if (doc_type == DocumentType::ForeignPassport) //10
    else if (doc_type == DocumentType::ForeignPassport) //15
    Это как? В операторе == таится побочный эффект?
    Ответить
  • #define 15 ForeignPassport 
    #define 14 TemporaryIdentityCard 
    
    // и потом такой
     else if (doc_type == 15) //ForeignPassport 
        {
            return std::make_unique<ExpireDateValidator>(
                   std::make_unique<DefaultDocumentValidator>());
        }

    зы: я знаю что не сработает, не занудствуйте
    Ответить
  • Пездец в какое гавно превратился современный c++
    И так нихуя не было понятно, так теперь боюсь даже читать доки по этим всяким птр
    Ответить

Добавить комментарий

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?


    8