Crash serveur flan mod

impuuu

engrous je mais quoi en null check

Benjamin Loison

Je prends ça clairement pour une blague… Je t’ai pondu une très belle réponse, si tu n’es pas foutu de répondre correctement et de prendre en considération ce qu’on te conseille… Clairement ne viens pas sur un Forum !

impuuu

voila je vien de faire avec ton code sauf que j ai la ligne if(getDriveableData().parts.get(EnumDriveablePart.core).dead != null)
qui m’indique une erreur

sa me donne sa

if(getDriveableData() != null)
{
    if(getDriveableData().parts != null)
    {
         if(getDriveableData().parts.get(EnumDriveablePart.core) != null)
         {
               if(getDriveableData().parts.get(EnumDriveablePart.core).dead != null)
               {
                   if(!worldObj.isRemote)
                   {
                       for(DriveablePart part : driveableData.parts.values())
                       {
                           if(part.health > 0 && !part.dead)
                               killPart(part);

               }
         }     setDead();
    }
         }
    }
}

}

AymericRed

@‘impuuu’:

qui m’indique une erreur

Et je suppose que nous devons deviner cette erreur ? Quoi comme erreur ?

Benjamin Loison

Je pense que le .dead ne vaut rien, puisque c’est un copier-coller de mon code que j’ai fournis, comme exemple fictif… x)

impuuu

quand je mais mas souris sur l’erreur qui est le if(getDriveableData().parts.get(EnumDriveablePart.core).dead != null) sa me donne
The operator != is undefined for the argument type(s) boolean, null

AymericRed

Retire le != null.

Envoyé de mon RAINBOW LITE 4G en utilisant Tapatalk

Benjamin Loison

Je tiens à souligner que le code est ici : https://github.com/minette2400/FlansMod/blob/1.7.10/src/main/java/com/flansmod/common/driveables/EntityDriveable.java

//If the core was destroyed, kill the driveable
if(getDriveableData().parts.get(EnumDriveablePart.core).dead)
{
if(!worldObj.isRemote)
{
for(DriveablePart part : driveableData.parts.values())
{
if(part.health > 0 && !part.dead)
killPart(part);
}
}
setDead();
}

Et je crois que du coup il n’y a pas besoin du !, mais bon impuu je t’invite quand même à revoir cette page, https://openclassrooms.com/courses/apprenez-a-programmer-en-java/les-variables-et-les-operateurs , c’est une erreur de base :X

AymericRed

Ah oui il n’y a pas besoin du ! j’edit mon message.

Envoyé de mon RAINBOW LITE 4G en utilisant Tapatalk

impuuu

donc je retrire le ! ses tout pas plus est benjamin merci de m avoir aidé puis m avoir conseillé cette page

Benjamin Loison

Testes impuuu et dis nous en des nouvelles (il faut prendre des initiatives)

impuuu

si je retire les ! sa me dit “add to cast Boolean” , "Changer type of ‘dead’ to ‘Object’ " est "Insert ‘!=null’ check "

Benjamin Loison

Alors on ne peut pas deviner ton code après plus de 6 messages… De plus les majuscules, l’orthographe et savoir bien recopier les erreurs et les bases en Java, c’est en option ?

impuuu

je vien de retirer le . dead est le bug est reglé mais sa donne le meme code
regarde un peu mais si ses bon je peut retiré la ligne ou il avait le . dead

code:

 if(getDriveableData().parts != null)
    {
         if(getDriveableData().parts.get(EnumDriveablePart.core) != null)
         {
               if(getDriveableData().parts.get(EnumDriveablePart.core) != null)
               {
                   if(!worldObj.isRemote)
                   {
                       for(DriveablePart part : driveableData.parts.values())
                       {
                           if(part.health > 0 && !part.dead)
                               killPart(part);

               }
         }     setDead();
    }
         }
    }
}

}

Benjamin Loison

Alors impuuu, j’ai été gentil d’essayé de te prévenir, mais supprimer une condition comme ça de si la partie est morte, je n’ai jamais trouvé une idée aussi … ! Tu ne sais simplement pas ce que tu fais et tu écris toujours aussi mal ! Plus d’aide de ma part.

AymericRed

if(getDriveableData().parts.get(EnumDriveablePart.core) != null)

devient :

if(getDriveableData().parts.get(EnumDriveablePart.core).dead)

Si la partie principale (core) est morte, on, fait ce qu’il y a après, c’est à dire tuer les autres parties.
Il ne faut pas modifier au hasard comme ça le code qu’on prend, et trouves-tu logique d’avoir deux if indentiques qui se suivent ?

impuuu

donc tout sa devien


1.   **if**(getDriveableData().parts != **null**)
2.       {
3.           **if**(getDriveableData().parts.**get**(EnumDriveablePart.core) != **null**)
4.           {
5.                 [font=monospace**if**][font=monospace(][font=monospacegetDriveableData][font=monospace().][font=monospaceparts][font=monospace.][font=monospace**get**][font=monospace(][font=monospaceEnumDriveablePart][font=monospace.][font=monospacecore][font=monospace).][font=monospacedead][font=monospace)]
6.                 {
7.                     **if**(!worldObj.isRemote)
8.                     {
9.                         **for**(DriveablePart part : driveableData.parts.values())
10.                         {
11.                             **if**(part.health > 0 && !part.dead)
12.                                 killPart(part);
13.                 }
14.           }     setDead();
15.      }
16.           }
17.      }
18.  }
19.  }

AymericRed

Oui, tu n’as qu’à lancer le jeu pour tester.

impuuu

oui mais ses le crach d un pack est ses un crach serveur

impuuu

du moin l entity taxi vien de un pack