Plz help me fix my ModPE Script! - MCPE: Mods / Tools - Minecraft: Pocket Edition - Minecraft Forum

#1 Oct 9, 2013

99FireyMan99

Void Walker
Join Date: 12/28/2012
Posts: 1,703
Minecraft: 99FireyMan99
Xbox: Oh plz! No Xbox crap, get Playstation!
PSN: TheLegitBacca123
Member Details

I am making a commands mod, and everything else is working except when I do it a gain! I have done the script the same way as I made the other commands! Place help!

} case 'spawncreeper': {

I did all the functions and all the other things to get it to work, but when I go to use it, it says "syntax error"

What do I need to change?

Click Banner to SUBSCRIBE!!! I'm in the Team Flame Clan!!! Click that little green arrow!! -------------->

#2 Oct 9, 2013

coolgboy

Curse Premium

Stuck at Home
Join Date: 7/31/2013
Posts: 461
Location: Playing Minecraft
Minecraft: LyricLy
Member Details

Well, i cannot help, but no one can if you don't show anyone the code.

Last edited by coolgboy: Oct 9, 2013

If I helped in any way, feel free to drop me an Internet:

Or some Blue XP:

A test to see what number is most thought of when a 1-10 choice is given: strawpoll.me/4012187

Don't think about it, just click on the first one that comes to you!

#3 Oct 9, 2013

99FireyMan99

Void Walker
Join Date: 12/28/2012
Posts: 1,703
Minecraft: 99FireyMan99
Xbox: Oh plz! No Xbox crap, get Playstation!
PSN: TheLegitBacca123
Member Details

Here is the code, plz help!

function procCmd(c){
var p = c.split(" ");
var command = p[0];
switch(command) {
case 'commands': {
clientMessage(" Showing help page ")
} case 'commands': {
clientMessage("\nCommands:n/commands, /help <command>, /explode, /give");
break;
} case 'help': {
}switch(p[1]) {
} case 'help': {
clientMessage("\nShowing help page 1 of 2 (/help <page>)\n /help<page>\n /give <ID> <amount>\n /info\n /heal\n /die");
break;
} case 'help2': {
clientMessage("Showing help page 2 of 2 (/help <page>)\n /godmode\n /coords\n /tp <x,y,z>\n /explode <radius>\n /time set <day or night>");
break;
} case 'give': {
addItemInventory(p[1],p[2]);
clientMessage("Spawned " + p[2] + " of " + p[1] + "!");
break;
} case 'info': {
clientMessage("Made by 99FireyMan99, Version 1.0")
break;
} case 'heal': {
Player.setHealth(20);
clientMessage("You have been healed!")
break;
} case 'die': {
Player.setHealth(0);
break;
} case 'godmode': {
Player.setHealth(32767);
clientMessage("Godmode has been enabled!")
break;
} case 'coords': {
clientMessage("\nCurrent coordinates are:\nx: " + Math.floor(getPlayerX()) + " y: " + Math.floor(getPlayerY() - 1) + " z: " + Math.floor(getPlayerZ()));
break;
} case 'tp': {
setPosition(getPlayerEnt(), parseInt(p[1]), parseInt(p[2]), parseInt(p[3]));
clientMessage("\nTeleported to x: " + parseInt(p[1]) + ", y: " + parseInt(p[2]) + ", z: " + parseInt(p[3]))
break;
} case 'explode': {
explode(getPlayerX(), getPlayerY(), getPlayerZ(), p[1]);
clientMessage("BOOOOM!!!!");
break;
} case 'knockback': {
addItemInventory(267, 1);
clientMessage("Knockback Sword, Thanks to: Darth377");
} case 'time': {
if(p[1] == 'set') {
if(p[2] == 'day' || p[2] == 'daytime') {
Level.setTime(0);
clientMessage("Time set to day!");
break;
} if(p[2] == 'night') {
Level.setTime(8280);
clientMessage("Time set to night!");
break;
} case 'spawncreeper': {
clientMessage("Creeper spawned!");

Click Banner to SUBSCRIBE!!! I'm in the Team Flame Clan!!! Click that little green arrow!! -------------->

#4 Oct 9, 2013

Hexdro

Curse Premium

IRC HACKER
Location: Somewhere
Join Date: 10/4/2012
Posts: 2,016
Location: Adelaide
Minecraft: Hexdro
PSN: Hexdro
Member Details

function procCmd©{
var p = c.split(" ");
var command = p[0];
switch(command) {
case 'commands': {
clientMessage(" Showing help page ")
} case 'commands': {
clientMessage("\nCommands:n/commands, /help     

, /explode, /give");
break;
} case 'help': {
}switch(p[1]) {
} case 'help': {
clientMessage("\nShowing help page 1 of 2 (/help )\n /help\n /give  \n /info\n /heal\n /die");
break;
} case 'help2': {
clientMessage("Showing help page 2 of 2 (/help )\n /godmode\n /coords\n /tp \n /explode \n /time set ");
break;
} case 'give': {
addItemInventory(p[1],p[2]);
clientMessage("Spawned " + p[2] + " of " + p[1] + "!");
break;
} case 'info': {
clientMessage("Made by 99FireyMan99, Version 1.0")
break;
} case 'heal': {
Player.setHealth(20);
clientMessage("You have been healed!")
break;
} case 'die': {
Player.setHealth(0);
break;
} case 'godmode': {
Player.setHealth(32767);
clientMessage("Godmode has been enabled!")
break;
} case 'coords': {
clientMessage("\nCurrent coordinates are:\nx: " + Math.floor(getPlayerX()) + " y: " + Math.floor(getPlayerY() - 1) + " z: " + Math.floor(getPlayerZ()));
break;
} case 'tp': {
setPosition(getPlayerEnt(), parseInt(p[1]), parseInt(p[2]), parseInt(p[3]));
clientMessage("\nTeleported to x: " + parseInt(p[1]) + ", y: " + parseInt(p[2]) + ", z: " + parseInt(p[3]))
break;
} case 'explode': {
explode(getPlayerX(), getPlayerY(), getPlayerZ(), p[1]);
clientMessage("BOOOOM!!!!");
break;
} case 'knockback': {
addItemInventory(267, 1);
clientMessage("Knockback Sword, Thanks to: Darth377");
} case 'time': {
if(p[1] == 'set') {
if(p[2] == 'day' || p[2] == 'daytime') {
Level.setTime(0);
clientMessage("Time set to day!");
break;
} if(p[2] == 'night') {
Level.setTime(8280);
clientMessage("Time set to night!");
break;
} case 'spawncreeper': {
clientMessage("Creeper spawned!");

Do that next time. Makes it easier to see, and therefore easier to fix.

Last edited by Hexdro: Oct 9, 2013

Check out my game! It's an open-world, sandbox text adventure.

Follow @hexdro_

#5 Oct 10, 2013

99FireyMan99

Void Walker
Join Date: 12/28/2012
Posts: 1,703
Minecraft: 99FireyMan99
Xbox: Oh plz! No Xbox crap, get Playstation!
PSN: TheLegitBacca123
Member Details

OK, I will, but what do I need to change?

Click Banner to SUBSCRIBE!!! I'm in the Team Flame Clan!!! Click that little green arrow!! -------------->

#8 Oct 10, 2013

99FireyMan99

Void Walker
Join Date: 12/28/2012
Posts: 1,703
Minecraft: 99FireyMan99
Xbox: Oh plz! No Xbox crap, get Playstation!
PSN: TheLegitBacca123
Member Details

Quote from QyMineAsap

Lol!

Quote from Connor4898

This is your script:

function procCmd©{
var p = c.split(" ");
var command = p[0];
switch(command) {
case 'commands': {
clientMessage(" Showing help page ")
} case 'commands': {
clientMessage("\nCommands:n/commands, /help <command>, /explode, /give");
break;
} case 'help': {
}switch(p[1]) {
} case 'help': {
clientMessage("\nShowing help page 1 of 2 (/help <page>)\n /help<page>\n /give <ID> <amount>\n /info\n /heal\n /die");
break;
} case 'help2': {
clientMessage("Showing help page 2 of 2 (/help <page>)\n /godmode\n /coords\n /tp <x,y,z>\n /explode <radius>\n /time set <day or night>");
break;
} case 'give': {
addItemInventory(p[1],p[2]);
clientMessage("Spawned " + p[2] + " of " + p[1] + "!");
break;
} case 'info': {
clientMessage("Made by 99FireyMan99, Version 1.0")
break;
} case 'heal': {
Player.setHealth(20);
clientMessage("You have been healed!")
break;
} case 'die': {
Player.setHealth(0);
break;
} case 'godmode': {
Player.setHealth(32767);
clientMessage("Godmode has been enabled!")
break;
} case 'coords': {
clientMessage("\nCurrent coordinates are:\nx: " + Math.floor(getPlayerX()) + " y: " + Math.floor(getPlayerY() - 1) + " z: " + Math.floor(getPlayerZ()));
break;
} case 'tp': {
setPosition(getPlayerEnt(), parseInt(p[1]), parseInt(p[2]), parseInt(p[3]));
clientMessage("\nTeleported to x: " + parseInt(p[1]) + ", y: " + parseInt(p[2]) + ", z: " + parseInt(p[3]))
break;
} case 'explode': {
explode(getPlayerX(), getPlayerY(), getPlayerZ(), p[1]);
clientMessage("BOOOOM!!!!");
break;
} case 'knockback': {
addItemInventory(267, 1);
clientMessage("Knockback Sword, Thanks to: Darth377");
} case 'time': {
   if(p[1] == 'set') {
     if(p[2] == 'day' || p[2] == 'daytime') {
       Level.setTime(0);
       clientMessage("Time set to day!");
break;
} if(p[2] == 'night') {
     Level.setTime(8280);
     clientMessage("Time set to night!");
break;
} case 'spawncreeper': {
clientMessage("Creeper spawned!");

Here is the correct code:

function procCmd(c) {
   var p = c.split(" ");
   var command = p[0];
   switch(command) {
      case 'commands': {
         clientMessage("\nCommands:n/commands, /help <command>, /explode, /give");
         break;
      } case 'help': {
         clientMessage("\nShowing help page 1 of 2 (/help <page>)\n /help<page>\n /give <ID> <amount>\n /info\n /heal\n /die");
         break;
      } case 'help2': {
         clientMessage("Showing help page 2 of 2 (/help <page>)\n /godmode\n /coords\n /tp <x,y,z>\n /explode <radius>\n /time set <day or night>");
         break;
      } case 'give': {
         addItemInventory(p[1],p[2]);
         clientMessage("Spawned " + p[2] + " of " + p[1] + "!");
         break;
      } case 'info': {
         clientMessage("Made by 99FireyMan99, Version 1.0")
         break;
      } case 'heal': {
         Player.setHealth(20);
         clientMessage("You have been healed!")
         break;
      } case 'die': {
         Player.setHealth(0);
         break;
      } case 'godmode': {
         Player.setHealth(32767);
         clientMessage("Godmode has been enabled!")
         break;
      } case 'coords': {
         clientMessage("\nCurrent coordinates are:\nx: " + Math.floor(getPlayerX()) + " y: " + Math.floor(getPlayerY() - 1) + " z: " + Math.floor(getPlayerZ()));
         break;
      } case 'tp': {
         setPosition(getPlayerEnt(), parseInt(p[1]), parseInt(p[2]), parseInt(p[3]));
         clientMessage("\nTeleported to x: " + parseInt(p[1]) + ", y: " + parseInt(p[2]) + ", z: " + parseInt(p[3]));
         break;
      } case 'explode': {
         explode(getPlayerX(), getPlayerY(), getPlayerZ(), p[1]);
         clientMessage("BOOOOM!!!!");
      break;
      } case 'knockback': {
         addItemInventory(267, 1);
         clientMessage("Knockback Sword, Thanks to: Darth377");
      } case 'time': {
         if(p[1] == 'set') {
            if(p[2] == 'day' || p[2] == 'daytime') {
               Level.setTime(0);
               clientMessage("Time set to day!");
               break;
            } if(p[2] == 'night') {
               Level.setTime(8280);
               clientMessage("Time set to night!");
            }
         }
         break;
      } case 'spawncreeper': {
         Entity.spawnMob(Player.getX(),Player.getY(),Player.getZ(),33,'mob/creeper.png');
         clientMessage("Creeper spawned!");
         break;
      }
   }
}

What to improve on:
-Indent your code. Doing this will help you find where you missed { or }. It also makes your script much more human-readable
-Match every { with a }. You missed many }, and also you sometimes went crazy with the { and }, as shown here:

} case 'help': {
}switch(p[1]) {
} case 'help': {

function procCmd©{

-The hook procCmd needs some parameters. In your script, it looks like it should be a 'c' (Looks very similar to mine...)
-You had two 'commands' cases, and two 'help' cases. You also had a random switch.
-You missed a ; on the cases tp and info.
-You missed a 'break;' after the knockback case.
-The knockback sword does nothing in this script.

If I helped, please click that little green arrow

Thank you soo much Connor!

Click Banner to SUBSCRIBE!!! I'm in the Team Flame Clan!!! Click that little green arrow!! -------------->