Является ли публичное перечисление неуважительным к инкапсуляции? ⇐ JAVA

[Anonymous]

Это вопрос к заданию для Университета. Для этого задания мы должны соблюдать инкапсуляцию и наследование в Java.
Сейчас важно знать следующие классы:
Cliente Class

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

import java.util.EnumSet;
import java.util.List;

public class Cliente {

// Definicao de enum da localizacao
private enum Localizacao {
CONTINENTE,
ACORES,
MADEIRA;
}

private String nome;
private int numContribuinte;
private Localizacao localizacao;

public Cliente(String nome, int numContribuinte, Localizacao localizacao){
this.nome = nome;
this.numContribuinte = numContribuinte;
this.localizacao = localizacao;
}

public Cliente() {
}
public void setLocalizacao(Localizacao localizacao) {
this.localizacao = localizacao;
}

public void setNome(String nome) {
this.nome = nome;
}

public void setNumContribuinte(int numContribuinte) {
this.numContribuinte = numContribuinte;
}

public String getNome() {
return nome;
}

public int getNumContribuinte() {
return numContribuinte;
}

public Localizacao getLocalizacao() {
return localizacao;
}

public Localizacao[] getLocalizacoes(){
return Localizacao.values();
}

public void printDadosCliente() {
System.out.println("\n");
System.out.println("Dados do Cliente:");
System.out.println("Nome: " + nome);
System.out.println("Número de contribuinte: " + numContribuinte);
System.out.println("Localização do cliente: " + localizacao.name());
}
}

класс Produtos:

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

public abstract class Produtos {
private int codigo;
private String nome;
private String descricao;
private int quantidade;
private double valorUnitario;

//Construtor

public Produtos(int codigo, String nome, String descricao, int quantidade, double valorUnitario){
this.codigo = codigo;
this.nome = nome;
this.descricao = descricao;
this.quantidade = quantidade;
this.valorUnitario = valorUnitario;
}

// Getters
public int getCodigo() {
return codigo;
}

public String getNome() {
return nome;
}

public String getDescricao() {
return descricao;
}

public int getQuantidade() {
return quantidade;
}

public double getValorUnitario() {
return valorUnitario;
}

public double getPrecoSemIva() {
return valorUnitario * quantidade;
}

// Abstract method to calculate IVA
public abstract double calculaIva(Cliente.Localizacao localizacao);

public abstract void printDadosProduto();
}

и класс ProdutoAlimentarNormal, который является подклассом ProdutoAlimentar, который является подклассом Produtos, но я опустил его, поскольку он добавляет в класс только переменную isBiologic boolean:

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

public class ProdutoAlimentarNormal extends ProdutosAlimentares {
public ProdutoAlimentarNormal(int codigo, String nome, String descricao, int quantidade, double valorUnitario, boolean isBiologico)
{
super(codigo, nome, descricao, quantidade, valorUnitario, isBiologico);
}

@Override
public double calculaIva(Cliente.Localizacao localizacao){
double taxaIva = 0.23;

if (localizacao.equals(Cliente.Localizacao.MADEIRA)){
taxaIva = 0.22;
} else if (localizacao.equals(Cliente.Localizacao.ACORES)) {
taxaIva = 0.16;
}
if (getBiologico()){
taxaIva *= 0.9;
}

return getQuantidade()* getValorUnitario() * taxaIva;
}

@Override
public void printDadosProduto(){
System.out.println("Dados do Produto:");
System.out.println("Tipo: Produto Alimentar");

if (getBiologico()){
System.out.print("Biologico: Sim");
} else {
System.out.print("Biologico: Não");
}

System.out.println("Taxa de IVA: Normal");

System.out.println("Nome do produto: "  + getNome());
System.out.println("Código do produto: " + getCodigo());
System.out.println("Quantidade: " + getQuantidade());
System.out.println("Preço por unidade: " + getValorUnitario() + "€");
System.out.println("Preço total (s/ IVA): " + getValorUnitario() * getQuantidade() + "€");
}
}

Как видите, я установил Localizacao как приватный:

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

// Definicao de enum da localizacao
private enum Localizacao {
CONTINENTE,
ACORES,
MADEIRA;
}

Но расчеты IVA для продуктов AlimentarNormal зависят от этих значений:

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

@Override
public double calculaIva(Cliente.Localizacao localizacao){
double taxaIva = 0.23;

if (localizacao.equals(Cliente.Localizacao.MADEIRA)){
taxaIva = 0.22;
} else if (localizacao.equals(Cliente.Localizacao.ACORES)) {
taxaIva = 0.16;
}
if (getBiologico()){
taxaIva *= 0.9;
}

return getQuantidade()* getValorUnitario() * taxaIva;
}

Я попробовал написать это так, передавая вместо этого клиента,

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

@Override
public double calculaIva(Cliente cliente){
double taxaIva = 0.13;

if (cliente.getLocalizacao() == cliente.getLocalizacoes().NOVALUE){
taxaIva = 0.12;
} else if (localizacao.equals()) {
taxaIva = 0.16;
}
if (categoria == CategoriasAlimentar.VINHO){
taxaIva += 0.01;
}

if(getBiologico()){
taxaIva *= 0.9;
}

return getQuantidade()* getValorUnitario() * taxaIva;
}

но никогда не мог заставить cliente.getLocalizacoes().NOVALUE отображать ACORES,MADEIRA или CONTINENTE в NOVALUE< /em> пространство.
Не нарушаю ли я инкапсуляцию, делая перечисление общедоступным?
Я пробовал использовать различные геттеры и сеттеры, но так и не смог получить ни одного, который бы мне что-то возвращал. что я мог бы просто использовать Localizacao.ACORES, пока он является частным.
Хотя он был общедоступным, он работал как шарм, но боюсь, я не уважаю инкапсуляцию.

Подробнее здесь: https://stackoverflow.com/questions/792 ... apsulation