Spec-Kit: Revisão de Código

// Código gerado por IA com problemas — ANTES da revisão
// Pontuação Spec-Kit: ~10/28. Identifique todos os problemas!

public class Sistema {                    // ❌ SRP — faz tudo
    public String nome;               // ❌ atributo public
    public ArrayList<String> itens; // ❌ raw + tipo concreto

    public void processar(String tipo) {
        if (tipo.equals("pagamento")) {  // ❌ if/else — use polimorfismo
            // lógica de pagamento
        } else if (tipo.equals("estoque")) {
            // lógica de estoque
        }
    }

    public String buscarItem(String id) {
        for (String s : itens)         // ❌ O(n) — use HashMap
            if (s.startsWith(id)) return s;
        return null;
    }

    public void salvar() {
        try { /* escreve arquivo */ }
        catch (Exception e) {}        // ❌ catch vazio!
    }
}

// APÓS revisão com Spec-Kit — versão corrigida:
public class CatalogoItens {             // ✅ nome descreve responsabilidade
    private Map<String, Item> indice;   // ✅ private, Map, generic

    public CatalogoItens() {
        this.indice = new HashMap<>();    // ✅ HashMap para O(1)
    }

    public Optional<Item> buscar(String id) {
        return Optional.ofNullable(indice.get(id)); // ✅ sem null
    }

    public void salvar() throws IOException { // ✅ declara exceção
        /* escreve arquivo */                   // ✅ não engole
    }
}