Что мне делать если я хочу стать разработчиком игр но не чего по этому не знаю?

German Jet, сомневаюсь. Да это и отношения к вопросу автора не имеет.

Как можно оптимизировать данную программу?

muryn.official, все верно. Я отписался выше, что не заметил return'а.
Сейчас самая дорогая операция у вас получается Collections.frequency в цикле

Как можно оптимизировать данную программу?

Collections.frequency тоже дорогая операция. Вы каждый раз вынуждены перечитать всю коллекцию, чтобы посчитать количество вхождений. В моем примере всего 1 проход по изначальной коллекции. И второй проход по коллекции сплющенной в мапу

Как можно оптимизировать данную программу?

muryn.official, да, верно. Не заметил.
Хм. Еще, из того, что может замедлять работу - это перекладывания в коллекций.
Вот этот шаг - Set set = new HashSet<>(sweets); явно лишний, если бы copy уже был сетом.

Как можно оптимизировать данную программу?

Точно. Он правильно говорит. Ее можно вынести в AtomicBoolean. А то, что я написал, называется функциональный стиль. Это стримы. Но если для вас это пока сложновато, лучше обойтись простым вынесением сортировки за пределы цикла.

Как можно оптимизировать данную программу?

У вас очень часто происходить сортировка
Collections.sort(sweets);
При большом массиве данных, если условие будет срабатывать часто, то вы будете дергать сортировку попусту.

Как можно оптимизировать данную программу?

muryn.official, вместо

Set<Integer> set = new HashSet<>(sweets);
    copy.addAll(set);
    
    for (int i = 0; i < copy.size(); i++) {
      if (Collections.frequency(sweets, copy.get(i)) >= min) {
        Collections.sort(sweets);
        StringBuilder sb = new StringBuilder();
        
        for (Integer j : sweets)
          sb.append(j + " ");
        
        System.out.print(sb);
        return;
      }
    }

    System.out.print("Oh sh*t");

Как можно оптимизировать данную программу?

Про проверку забыл. Виноват

Как можно оптимизировать данную программу?

Если бы вы еще сказали, что от программы ожидается, было бы совсем здорово )

Почему в интерфейсах не надо указывать модификатор доступа?

Для лямбд )

Лаконичность и правильность кода в Java. В какой момент стоит объявлять переменные в коде?

Явных косков не вижу. Все верно.
Но есть кое-что, что можно улучшить.
1) Если использовать String.format вместо конкатенации строк, код станет чуть более читаемым.
2) Хорошим тоном считается объявлять переменную в контексте использования, а не заранее. Так легче понимать, откуда она взялась, когда читаешь не весь код, а только его фрагмент. Это так же облегчит рефакторинг, например, если ты захочешь разбить этот большой метод на несколько маленьких методов.
3) Любой смысловой блок, который хочется визуально выделить в коде, заслуживает того, чтобы быть перенесенным в отдельный метод с осмысленным названием.
4) Многоразовые переменные - это не очень круто, начнешь это ощущать, когда начнешь рефакторить код.
5) Я бы дал переменной upp более осмысленное название, это бы облегчило чтение кода по фрагментам.

А еще, нам стоит сворачиваться, пока не пришел модератор и не отругал нас за нарушение правил тостера. Поскольку мы увлеклись обсуждением и превратили вопрос в топик форума.

Лаконичность и правильность кода в Java. В какой момент стоит объявлять переменные в коде?

PRAIT, почти любой закрываемый объект предполагает одно закрытие за весь цикл жизни. После закрытия им не пользуются

Лаконичность и правильность кода в Java. В какой момент стоит объявлять переменные в коде?

Либо так

package test;
import java.util.Scanner;
public class Test {
  public static void main(String[] args) {
    Scanner tu = new Scanner(System.in);
    System.out.println("Ваше имя:");
    name = tu.nextLine();
    System.out.println("Вашe отчество:");
    surName = tu.nextLine();
    System.out.println("Какой сейчас год?");
    yearBorn = tu.nextInt();
    System.out.println("В каком году вы родились?");
    yearNow = tu.nextInt();
    System.out.println("Здравствуйте "+name+" "+surName+"!");
    System.out.println("Ваш возраст "+(yearBorn-yearNow)+"");
    tu.close();
  }
}

Либо так

package test;
import java.util.Scanner;
public class Test {
  public static void main(String[] args) {
    try(Scanner tu = new Scanner(System.in)) {
      System.out.println("Ваше имя:");
      name = tu.nextLine();
      System.out.println("Вашe отчество:");
      surName = tu.nextLine();
      System.out.println("Какой сейчас год?");
      yearBorn = tu.nextInt();
      System.out.println("В каком году вы родились?");
      yearNow = tu.nextInt();
      System.out.println("Здравствуйте "+name+" "+surName+"!");
      System.out.println("Ваш возраст "+(yearBorn-yearNow)+"");
    }
  }
}

Лаконичность и правильность кода в Java. В какой момент стоит объявлять переменные в коде?

Да, по хорошему сканер надо закрывать после использования. Вообще все классы, которые имплеменят AutoCloseable и Closeable нужно закрывать после использования

Лаконичность и правильность кода в Java. В какой момент стоит объявлять переменные в коде?

try(Requester requester = new Requester()) {
...
}

Означает, что при выходе из скоупа у объекта вызовется метод close

Как написать generic фабрику?

Еще стоит добавление в мапу вынести в отдельный дженериковый метод, чтобы никто не ошибся указывая класс машин с хендлером.

private static <T extends Car> void putHandler(Map<Class<? extends Car>, Handler> carHandlers, Class<T> car, Handler<T> handler) {
    carHandlers.put(car, handler);
  }