Лучший способ написать собственное исключение ⇐ JAVA

[Anonymous]

Я ищу лучший способ написать собственное исключение:

Код: Выделить всё

public class NoSuchTaskException extends BaseImpactException {
private final String taskId;
private final String taskType;
private final Long userId;
private final DocumentType documentType;
private final Long documentId;

public NoSuchTaskException(String taskId, String taskType, Long userId,
DocumentType documentType, Long documentId) {
this(taskType, userId, documentType, documentId, taskId, null);
}

public NoSuchTaskException(String taskType, Long userId, DocumentType documentType,
Long documentId) {
this(taskType, userId, documentType, documentId, null, null);
}

public NoSuchTaskException(String taskType, Long userId, DocumentType documentType,
Long documentId, String taskId, Throwable throwable) {
super("exception.task.noSuchTask", throwable);
this.taskId = taskId;
this.taskType = taskType;
this.userId = userId;
this.documentType = documentType;
this.documentId = documentId;
}
// ...
}

который я затем использую в разных местах:

• Код: Выделить всё

  throw new NoSuchTaskException(taskId, null, isUserPresent ? null : addedById, null, null)

  ;

• Код: Выделить всё

  throw new NoSuchTaskException(taskId, taskType.name(), taskExecutorId, null, null)

  ;

• Код: Выделить всё

  throw new NoSuchTaskException(taskId, null, null, null, null)

  , ...

Думаю, из примеров использования становится ясно, почему я считаю, что это не лучшая реализация/использование. Моя идея заключалась в том, чтобы улучшить это, используя шаблон компоновщика:

Код: Выделить всё

public class NoSuchTaskException extends BaseImpactException {
private final String taskId;
private final String taskType;
private final Long userId;
private final DocumentType documentType;
private final Long documentId;

private NoSuchTaskException(Builder builder) {
super("exception.task.noSuchTask", builder.throwable);
this.taskId = builder.taskId;
this.taskType = builder.taskType;
this.userId = builder.userId;
this.documentType = builder.documentType;
this.documentId = builder.documentId;
}

public static Builder builder(String taskId) {
return new Builder(taskId);
}

@RequiredArgsConstructor
public static class Builder {
private String taskId;
private String taskType;
private Long userId;
private DocumentType documentType;
private Long documentId;
private Throwable throwable;

public Builder taskId(String taskId) {
this.taskId = taskId;
return this;
}

public Builder taskType(String taskType) {
this.taskType = taskType;
return this;
}

public Builder userId(Long userId) {
this.userId = userId;
return this;
}

public Builder documentType(DocumentType documentType) {
this.documentType = documentType;
return this;
}

public Builder documentId(Long documentId) {
this.documentId = documentId;
return this;
}

public Builder throwable(Throwable throwable) {
this.throwable = throwable;
return this;
}

public NoSuchTaskException build() {
return new NoSuchTaskException(this);
}
}
}

который я затем могу использовать как:

Код: Выделить всё

throw NoSuchTaskException.builder("task123")
.documentType(DocumentType.DOCUMENT_B)
.documentId(500L)
.build();

вместо того, чтобы использовать его следующим образом:

Код: Выделить всё

throw new NoSuchTaskException("task123", null, null, DocumentType.DOCUMENT_B, 500L);

Это приемлемый подход? Должен ли я использовать шаблон построителя в своем исключении, или это излишество (или даже запах кода)?


Подробнее здесь: https://stackoverflow.com/questions/793 ... -exception