Forward from: Хабр
Нашёл ошибки в моде Create и проверил их в игре
У Create больше 206 миллионов скачиваний только на CurseForge, но статический анализ всё равно находит в коде занятные места. Например, Ctrl + V для координат не работает, если после запятых стоят пробелы: replaceAll вызывается, но его результат просто выбрасывается.
В другом файле Math.max тоже вызывается впустую — и так было с самой первой версии 2022 года. За четыре года и несколько портов эта строка ничего не делала и никому не мешала.
А дальше уже начинаются проверки в самой игре. Замена одной недостижимой проверки сначала не даёт предметам бесконечно ходить по кругу между механическими крафтерами, но стоит изменить конструкцию — и простого исправления уже недостаточно. В другом месте обращение к null заканчивается stacktrace и обычной вагонеткой вместо штуковины.
То есть предупреждение анализатора ещё нужно довести до реального поведения — и иногда по дороге выясняется совсем не то, что ожидалось в начале.
У Create больше 206 миллионов скачиваний только на CurseForge, но статический анализ всё равно находит в коде занятные места. Например, Ctrl + V для координат не работает, если после запятых стоят пробелы: replaceAll вызывается, но его результат просто выбрасывается.
В другом файле Math.max тоже вызывается впустую — и так было с самой первой версии 2022 года. За четыре года и несколько портов эта строка ничего не делала и никому не мешала.
А дальше уже начинаются проверки в самой игре. Замена одной недостижимой проверки сначала не даёт предметам бесконечно ходить по кругу между механическими крафтерами, но стоит изменить конструкцию — и простого исправления уже недостаточно. В другом месте обращение к null заканчивается stacktrace и обычной вагонеткой вместо штуковины.
То есть предупреждение анализатора ещё нужно довести до реального поведения — и иногда по дороге выясняется совсем не то, что ожидалось в начале.