[Request] TabGui colors (lest den Threat durch xD)

[MasterOf13FPS]

Hallo erstmal,
unzwar hab ich jetzt ein TabGui fertig gecodet das Problem ist das ich die Farben so haben möchte das wenn das Module an ist die schrift Farbe weiß sein soll und wenn das Module nicht an ist die schrift Farbe grau sein soll. (Nicht in der ArrayList sondern im TabGui versteht sich)

Wenn das Module an ist : http://prntscr.com/ebmjam (Weiß)
Wenn das Module nicht an ist: http://prntscr.com/ebmj2q (Grau)

Edit : Bin ein anfänger im Coden also bitte etwas respekt xD

 

[MasterOf13FPS]

Sieht aus wie die TabGUI "von" Trace. Ersetz das "§f" mit "§7" Dann solls gehen (falls ich dich richtig verstanden habe).

 

[MasterOf13FPS]

ok, ok Nochmal neu:
Also ich möchte wenn ich beim Tabgui auf ein Modul klicke das dies dann die Farbe wechselt beim Click und wenn ich wieder draufklicke sprich es deaktiviere soll es wieder die anfangsfarbe besitzen.
Hoffe das war jetzt verständlich xD

Edit: So das man im Tabgui sieht welche module an sind und welche nicht

 

[MasterOf13FPS]

Und die Anfangsfarbe ist weiß bzw soll weiß sein?

 

[MasterOf13FPS]

jup und wenn man das Module anmacht soll die Farbe auf Grau springen

 

[MasterOf13FPS]

du machst beim rendern des schriftzugs einen aktive bzw on check und wenn eben true ist, dann ne andere frabe

 

[MasterOf13FPS]

http://prntscr.com/ebn4h5 <- so sollte das gehen xd

 

[MasterOf13FPS]

            for (Mod mod : Pechoin.mods.mods.stream().filter(Mod::isActive).collect(Collectors.toList())) {
                mc.fontRendererObj.drawString(mod.getName(), scaledResolution.getScaledWidth() - mc.fontRendererObj.getStringWidth(mod.getName()) - 2, y, mod.isActive() ? -1 :  getRainbow(6000, -15 * y));
                y += 12;
            }

kleines Beispiel mit dem mod.isActive?

 

[MasterOf13FPS]

http://prntscr.com/ebn4h5 <- so sollte das gehen xd

EDIT : http://prntscr.com/ebn79n (so ists besser)

 

[MasterOf13FPS]

            for (Mod mod : Pechoin.mods.mods.stream().filter(Mod::isActive).collect(Collectors.toList())) {
                mc.fontRendererObj.drawString(mod.getName(), scaledResolution.getScaledWidth() - mc.fontRendererObj.getStringWidth(mod.getName()) - 2, y, mod.isActive() ? -1 :  getRainbow(6000, -15 * y));
                y += 12;
            }

kleines Beispiel mit dem mod.isActive?

Ihhh rainbow :^)

 

[MasterOf13FPS]

Thx

 

[MasterOf13FPS]

            for (Mod mod : Pechoin.mods.mods.stream().filter(Mod::isActive).collect(Collectors.toList())) {
                mc.fontRendererObj.drawString(mod.getName(), scaledResolution.getScaledWidth() - mc.fontRendererObj.getStringWidth(mod.getName()) - 2, y, mod.isActive() ? -1 :  getRainbow(6000, -15 * y));
                y += 12;
            }

kleines Beispiel mit dem mod.isActive?

Die Java-8+ Streams API ist eigentlich anders gedacht.
So wie du es gemacht hast, ist es weder elegant, noch performant.
So sollte dein Code wohl eher aussehen:

for (Mod mod : Pechoin.mods.mods) { // Ich empfehle dir da, nicht static zu verwenden.
    if (mod.isActive()) {
        mc.fontRendererObj.drawString(mod.getName(), scaledResolution.getScaledWidth() - mc.fontRendererObj.getStringWidth(mod.getName()) - 2, y, mod.isActive() ? -1 : getRainbow(6000, -15 * y));
        y += 12;
    }
}

 

[MasterOf13FPS]

http://prntscr.com/ebn4h5 <- so sollte das gehen xd

EDIT : http://prntscr.com/ebn79n (so ists besser)

Das wäre da wohl etwas besser:

// Die Konstanten sind hier nur zur anschaulichkeit da, ich würde sie am Ende nicht verwenden.
private static final int COLOR_ENABLED = 0xFF919499;
private static final int COLOR_DISABLED = 0xFFFFFFFF;
private static final int FONT_HEIGHT = 10;

private void renderArrayList(int x) {
  int y = 0;
  for (Module module : Exotic.moduleManager.getModules()) { // Auch da würde ich kein static verwenden.
    FontUtils.drawString(module.getName(), x, y += FONT_HEIGHT, module.isEnabled() ? COLOR_ENABLED : COLOR_DISABLED);
  }
}

 

[MasterOf13FPS]

Die Java-8+ Streams API ist eigentlich anders gedacht.
So wie du es gemacht hast, ist es weder elegant, noch performant.
So sollte dein Code wohl eher aussehen:

danke, mache nur das was intelliJ mir sagt^^ was meinst du mit

Ich empfehle dir da, nicht static zu verwenden.

?

 

[MasterOf13FPS]

danke, mache nur das was intelliJ mir sagt^^ was meinst du mit

IntelliJ "sagt" sicherlich nicht, dass man das so handhaben soll.

?

http://stackoverflow.com/questions/7026507/why-are-static-variables-considered-evil

 

[MasterOf13FPS]

IntelliJ "sagt" sicherlich nicht, dass man das so handhaben soll.

http://stackoverflow.com/questions/7026507/why-are-static-variables-considered-evil

meinst du weil mein Modmanager static ist_

 

[MasterOf13FPS]

meinst du weil mein Modmanager static ist_

ja

 

[MasterOf13FPS]

also dann ne instanz von meiner main klasse nehmen als static oder soll ich das theminecraft nehmen?

 

[MasterOf13FPS]

Eine instanz von deiner "Main" Klasse