Texture Four en fonction du son status

SCAREX

Normal que ça marche pas…

package com.google.SpyMan.Mechanicalcraft.common.BlockCrusher;

import com.google.SpyMan.Mechanicalcraft.common.MechanicalCraft;
import net.minecraft.block.Block;
import net.minecraft.init.Blocks;
import net.minecraft.init.Items;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;

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

public class CrusherRecipes
{
private static final CrusherRecipes smeltingBase = new CrusherRecipes();
private Map <itemstack[], itemstack="">smeltingList = new HashMap<itemstack[], itemstack="">();
private MechanicalCraft m; // ici la variable n'est jamais initialisée

public CrusherRecipes()
{
this.m.crusherRecipes(); // Et ici tu appelles la variable
}
}

L’instance ne va pas se créer toute seule.

PS : pense vraiment à respecter la convention java</itemstack[],></itemstack[],>

SpyMan

9a ne change pas grand chose ça crash toujours

Pour la code convention alors y’a 24 pages et je ne suis qu’à la 13ème pages

Deleted

Si MechanicalCraft est le nom de ta classe Mod, alors ça ne sert strictement à rien de lui faire une instance puisqu’elle est censée être singleton. Tu as déjà dedans (dans ta classe MechanicalCraft) une variable instance, normalement, si tu as bien suivi le tuto. Alors tu aurais juste à faire MechanicalCraft.instance.crusherRecipes()

SCAREX

9a ?? C’est quoi ça ?

Si tu veux avoir une instance de ton mod, tu peux aussi faire “MechanicalCraft m = MechanicalCraft.INSTANCE;” ou quelque chose dans le genre, ça dépend de ce que tu as mis.

Les conventions sont assez simples :

écriture en lowerCamelCase pour les variables et les fonctions
écriture en UpperCamelCase pour les noms de classes
écriture en SCREAMING_SNAKE_CASE pour les constantes
les noms de packages doivent êtres entièrement en minuscules

Deleted

@SCAREX
Je pense qu’il voulait dire “Ca” dans son début de réponse.
Et au passage ouais, pas besoin de te taper 30 pages de blabla, juste pour 4/5 règles grand maxi (citées au-dessus) de convention Java. Perso pour l’apprendre, je n’ai eu besoin que de 5 min ^^’

SpyMan

Ok ça marche merci
Mais comment je peut changer la texture de front de ma machine car j’ai essayé tout ce que on n’a pu me donner comme code et ça marche pas ou je ne mit prend pas correctement

AymericRed

Sinon si rien ne marche, fais deux blocs (on et off) et imite le code vanilla

SCAREX

C’est à dire ? La texture n’est pas chargée (Il faut regarder dans les logs pour savoir si la texture est trouvée) ?

SpyMan

Non le jeu crash, mais je vais essayé de faire ça dans la fonction updateEntity de mon TileEntity voir ce que cela donne

SCAREX

Envoi le crash report et les classes concernées

SpyMan

J’ai trouvé la solution mais au problèmes textures enfin !
Mais j’ai un petit soucis : Mon block ne se met pas a jour comment faire pour ?
Quand je place ma speed upgrade à l’intérieure ça fonctionne mais quand je la retire ça garde la valeur précédente de workingTimeNeeded


[color=#0187df// Ceci se trouve dans la method updateEntity() de mon TileEntity]
[font=Courier Newif ](contents[4] != null)
{
   if (contents[4].getItem() == MechanicalCraft.speedUpgrade)
   {
       workingTimeNeeded = SpeedUpgrade.getSpeedUp()%(#aaa9a7)[;
]    }
   %(#0187df)[else
]    {
       workingTimeNeeded = workingTimeNeededDefault%(#aaa9a7)[;
]    }
}
else {}

SCAREX

C’est à dire mettre à jour ?

SpyMan

Je veut que la valeur de workingTimeNeeded change mais sans avoir besoin de mettre a jour le block manuellement ça fonctionne dans un sens mais pas dans l’autre:
Quand je la place ça marche mais quand je la retire ça ne redonne pas la valeur par default (workingTimeNeededDefault)
J’ai bien verifier si ces valeurs etaient enregistré dans les données NBT, et ce n’est pas le cas.

BrokenSwing

Normal quand t’enlève ton upgrade, content[4] passe à nul :


if (contents[4] != null)
{
if (contents[4].getItem() == MechanicalCraft.speedUpgrade)
{
workingTimeNeeded = SpeedUpgrade.getSpeedUp();
}
}
else {
workingTimeNeeded = workingTimeNeededDefault;
}

SpyMan

Merci !