Как сделать данный код более надежным?

Question

Я несколько раз читал, что ключевое слово new, лежащее где-нибудь в вашем коде, является запахом кода и затрудняет тестирование. Вот фрагмент кода, я работаю над:

private void uploadTagsToCloudStorage(Optional<String> filename, String tags) throws IOException { 
    if (!filename.isPresent()) { 
     return; 
    } 
    GcsFilename gcsfilename = new GcsFilename(gcsBucketName, filename.get()); 
    GcsFileOptions options = 
     new GcsFileOptions.Builder().mimeType("text/html").acl("public-read").build(); 
    try (GcsOutputChannel writeChannel = gcsService.createOrReplace(gcsfilename, options)) { 
     PrintWriter gcswriter = 
      new PrintWriter(Channels.newWriter(writeChannel, StandardCharsets.UTF_8.toString())); 
     gcswriter.println(decode(tags)); 
     gcswriter.flush(); 
     writeChannel.waitForOutstandingWrites(); 
    } 
    } 

Я использую Guice впрыснуть gcsService, но до сих пор я вижу много new ключевых слов вокруг. Я страдаю от плохого дизайна? или такой сценарий неизбежен?

Answer 1

Использование «нового» для создания объектов затрудняет тестирование, потому что вы не можете легко изменить между различными реализациями. Dependency Injection позволяет вам, например, изменять между mock gcsService или реальным gcsService impl без переписывания кода.

На вашем примере ни одно новое GcsFilename() и новый PrintWriter() не изменится, поэтому я не вижу проблемы при использовании «нового». GcsFileOptions.Builder() .. build() выглядит как статический метод, вам не нужно «новое», чтобы получить параметры.