проблема с гуи

[Antony_wes]

Привет, пишу свое гуи и для настроек понадобилось создать виджет свича, в его render написал пока что только круг, запустил попробовать, но при заходе в гуи вылетает ошибка о делении на ноль, хотя в коде я нигде не делю и краш-лог вообще отсылает на super.render.

package net.lil_requiem.gui;

import net.lil_requiem.gui.custom.Switch;
import net.minecraft.client.font.TextRenderer;
import net.minecraft.client.gui.DrawContext;
import net.minecraft.client.gui.screen.Screen;
import net.minecraft.client.gui.tooltip.Tooltip;
import net.minecraft.client.gui.widget.ButtonWidget;
import net.minecraft.text.Text;
import net.minecraft.util.Colors;
import net.minecraft.util.Identifier;

import java.util.HashMap;
import java.util.Map;

public class ModMainScreen extends Screen {
    //settings
    private boolean isBackgroundImage = true;
    //
    private int guiState = 1;
    private Identifier backgroundImage = new Identifier("textures/gui/background_night.png");
    private final Identifier ON = new Identifier("textures/gui/on.png");
    private final Identifier OFF = new Identifier("textures/gui/off.png");

    public ModMainScreen() {
        super(Text.of("Mod Main Screen"));
    }

    public ButtonWidget button1;
    public ButtonWidget button2;
    public ButtonWidget button3;
    private final Switch switch1 = new Switch((int) (width*0.90), (int) (height*0.15), (int) (width*0.05), (int) (height*0.05), Text.literal("Hello!"));

    @Override
    protected void init() {
        button1 = ButtonWidget.builder(Text.literal("SKILLS"), button -> {
                    guiState = 1;
                })
                .dimensions(width / 2 - 150, 5, 100, 20)
                .build();
        button2 = ButtonWidget.builder(Text.literal("TREE"), button -> {
                    guiState = 2;
                })
                .dimensions(width / 2 -50, 5, 100, 20)
                .build();
        button3 = ButtonWidget.builder(Text.literal("SETTINGS"), button -> {
                    guiState = 3;
                })
                .dimensions(width / 2 + 50, 5, 100, 20)
                .build();

        addDrawableChild(button1);
        addDrawableChild(button2);
        addDrawableChild(button3);
    }

    @Override
    public void render(DrawContext context, int mouseX, int mouseY, float delta) {
        super.render(context, mouseX, mouseY, delta);
        renderBackground(context);
        switch (guiState) {
            case 1:
                clearAndInit();
                renderfirst(context, mouseX, mouseY, delta);
                break;
            case 2:
                clearAndInit();
                rendersecond(context, mouseX, mouseY, delta);
                break;
            case 3:
                clearAndInit();
                renderthird(context, mouseX, mouseY, delta);
                break;
        }

    }

    private void renderfirst(DrawContext context, int mouseX, int mouseY, float delta){
        context.drawCenteredTextWithShadow(textRenderer, Text.literal("SKILLS"), width / 2, height / 2, 0xffffff);
    }
    private void rendersecond(DrawContext context, int mouseX, int mouseY, float delta){
        context.drawCenteredTextWithShadow(textRenderer, Text.literal("TREE"), width / 2, height / 2, 0xffffff);
    }
    private void renderthird(DrawContext context, int mouseX, int mouseY, float delta){
        context.drawTextWithShadow(textRenderer, "Background Image", (int) (width*0.05), (int) (height*0.15), 0xffffff);
        addDrawableChild(switch1);
    }
}

package net.lil_requiem.gui.custom;

import net.minecraft.client.gui.DrawContext;
import net.minecraft.client.gui.screen.narration.NarrationMessageBuilder;
import net.minecraft.client.gui.widget.PressableWidget;
import net.minecraft.text.Text;

import static java.lang.Math.abs;
import static java.lang.Math.pow;

public class Switch extends PressableWidget {
    public static final int DEFAULT_WIDTH = 50;
    public static final int DEFAULT_HEIGHT = 20;
    public static final int DEFAULT_RADIUS = 20;
    public static int X = 0;
    public static int Y = 0;

    private boolean state;
    public Switch(int center_x, int center_y, int k, int l, Text text) {
        super(center_x, center_y, k, l, text);
        X=center_x;
        Y=center_y;
        state = false;
    }

    @Override
    public void onPress() {
        state = !state;
    }

    @Override
    public void render(DrawContext context, int mouseX, int mouseY, float delta) {
        super.render(context, mouseX, mouseY, delta);
        for(int i=X-DEFAULT_RADIUS; i<X+DEFAULT_RADIUS; i++) {
            for(int j=Y+DEFAULT_RADIUS; j>Y-DEFAULT_RADIUS; j--) {
                if(pow(abs(X-i), 2)+pow(abs(Y-j), 2)<pow(DEFAULT_RADIUS, 2)) {
                    if (state) context.fill(i, j, i, j, 0x009fff);
                    else context.fill(i, j, i, j, 0x009f40);
                }
            }
        }
    }

    @Override
    protected void appendClickableNarrations(NarrationMessageBuilder builder) {

    }
}

 

[Heathcliff]

 

[_SupCM_]

1) Зачем ты в квадрат возводишь модуль? Не пофиг ли, если число одно и то же будет? Напомню: pow достаточно дорогая операция из-за особенностей своей реализации (это касается и C++, и Java в т.ч.), так что в случае куба или квадрата лучше просто перемножать число на себя: (X-i) * (X-i).
2) Рассматривай случай i = j = 0, тогда ошибок быть не должно.

 

[Antony_wes]

окей, я сделал все, как ты сказал. Для того, что бы не делать 4 раза разность, я сделал 2 переменные. так же рассмотрел случай, если i==j==0, майн все равно крашит с той же ошибкой деления на ноль.

@Override
    public void render(DrawContext context, int mouseX, int mouseY, float delta) {
        super.render(context, mouseX, mouseY, delta);
        for(int i=X-DEFAULT_RADIUS; i<X+DEFAULT_RADIUS; i++) {
            for(int j=Y+DEFAULT_RADIUS; j>Y-DEFAULT_RADIUS; j--) {
                if (i!=0 && j!=0) {
                    int a, b;
                    a = X-i;
                    b = Y-j;
                    if((a*a)+(b*b)<(DEFAULT_RADIUS*DEFAULT_RADIUS)) {
                        if (state) context.fill(i, j, i, j, 0x009fff);
                        else context.fill(i, j, i, j,0x009f40);
                    }
                }
            }
        }
    }

Если честно я не особо понял, зачем проверять условие с равностью нуля и i, j, потому что деления с участием этих переменных нет и не было.

 

[_SupCM_]

Деление происходит внутри context#fill, потому и вылетает divide by zero.
Смотри внутр. реализацию. и где там происходит деление на 0.

 

[Antony_wes]

Окей, спасибо большое, буду искать

 

[Antony_wes]

Вообще ничего не нашел, все сводится к Math.fma(), в котором деления так же нет.

@IntrinsicCandidate
    public static float fma(float a, float b, float c) {
        if (Float.isFinite(a) && Float.isFinite(b) && Float.isFinite(c)) {
            if (a == 0.0 || b == 0.0) {
                return a * b + c; // Handled signed zero cases
            } else {
                return (new BigDecimal((double)a * (double)b) // Exact multiply
                        .add(new BigDecimal((double)c)))      // Exact sum
                    .floatValue();                            // One rounding
                                                              // to a float value
            }
        } else {
            // At least one of a,b, and c is non-finite. The result
            // will be non-finite as well and will be the same
            // non-finite value under double as float arithmetic.
            return (float)fma((double)a, (double)b, (double)c);
        }
    }

 

[Antony_wes]

так, я походил по краш логу и понял, что дело в private final Switch switch1 = new Switch((int) (width*0.90), (int) (height*0.15), (int) (width*0.05), (int) (height*0.05), Text.literal("Hello!"));. посчитал эти значения в калькуляторе и заменил их на private final Switch switch1 = new Switch(1728, 162, 96, 52, Text.literal("Hello!")); . Теперь майн не вылетает, но и сам круг не рисуется, можете подсказать, что не так с рисовкой?

 

[_SupCM_]

Смотри в DrawContext#createDivider. Где-то там у тебя возникает деление на 0. В частности, обрати внимание на MathHelper#ceilDiv (который ссылается на floorDiv; в последнем на 2 параметр приходит 0 => возникает деление на 0).
Среды на Fabric 1.20.1 у меня нет, а на Forge 1.20.1 такого класса (DrawContext) нет, потому подсказать могу лишь немного.

 

[Antony_wes]

окей, спасибо большое. Проблему с делением я решил, а с тем, что не рисуется я решу как нибудь сам.