1. C++ / Говнокод #21980

    −15

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 16 Января 2017

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

    • ava huge_cock 16.01.2017 19:02 # −1

      Заднепроходно воспользовался твоим архитектором, проверь.
      Ответить

    • ava bormand 16.01.2017 19:04 # +6 

      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
      Это как? В операторе == таится побочный эффект?
      Ответить

    • ava barop 16.01.2017 19:06 # +2 

      #define 15 ForeignPassport 
#define 14 TemporaryIdentityCard 

// и потом такой
 else if (doc_type == 15) //ForeignPassport 
    {
        return std::make_unique<ExpireDateValidator>(
               std::make_unique<DefaultDocumentValidator>());
    }

      зы: я знаю что не сработает, не занудствуйте
      Ответить

    • ava guestinho 17.01.2017 21:14 # +2

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

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