Thread: [PI] Dialouge Issue

Results 1 to 9 of 9
  1. #1 [PI] Dialouge Issue 
    Registered Member Sirius_'s Avatar
    Join Date
    Nov 2013
    Age
    25
    Posts
    269
    Thanks given
    77
    Thanks received
    53
    Rep Power
    11
    Karma and Thanks to the helpful users.

    I am new to Java as a whole but am still getting use to how my newest project is packaged and what not and running into a dialogue issue. I have made it so the NPC will say whatever it is suppose to say but when it comes to dialogue actions for example teleporting you but in this instance changing your appearance it is not working for some reason.

    My Code:

    DialogueHandler Class

    Code:
    /*
    	 * Make-Over Make Dialogue
    	 */
    		case 20:
    			sendNpcChat1("Hello, "+ c.playerName +" I can change your appearance!", c.talkingNpc, "Make-Over Mage");
    			c.nextChat = 21;
    			break;
    		case 21:
    			sendNpcChat1("Would you like to change your appearance?", c.talkingNpc, "Make-Over Mage");
    			c.nextChat = 22;
    			break;
    		case 22:
    			sendOption2("Yes!", "No.");
    			c.dialogueAction = 22;
    			break;
    		}
    	}
    ActionHandler Class
    Code:
    /*
    * Make-Over Mage First Option
    */
    		case 599:
    			c.getDH().sendDialogues(20, 599);
    			//c.getPA().showInterface(3559);
    			//c.canChangeAppearance = true;
    /*
    * Make-Over Mage Second Option
    */
    case 599: 		
    			 c.getPA().showInterface(3559);
    			 c.canChangeAppearance = true;

    ClickingButtons Class

    Code:
    case 9157:// barrows tele to tunnels
    			if(c.dialogueAction == 22) {
    				c.getPA().showInterface(3559);
        			c.canChangeAppearance = true;	
        		}
    Spoiler for More Info:

    Project Thread: Project-Sirius

    Reply With Quote  
     

  2. Thankful user:


  3. #2  
    Registered Member
    Join Date
    Sep 2013
    Posts
    880
    Thanks given
    43
    Thanks received
    57
    Rep Power
    58
    Quote Originally Posted by Sirius_ View Post
    Karma and Thanks to the helpful users.

    I am new to Java as a whole but am still getting use to how my newest project is packaged and what not and running into a dialogue issue. I have made it so the NPC will say whatever it is suppose to say but when it comes to dialogue actions for example teleporting you but in this instance changing your appearance it is not working for some reason.

    My Code:

    DialogueHandler Class

    Code:
    /*
    	 * Make-Over Make Dialogue
    	 */
    		case 20:
    			sendNpcChat1("Hello, "+ c.playerName +" I can change your appearance!", c.talkingNpc, "Make-Over Mage");
    			c.nextChat = 21;
    			break;
    		case 21:
    			sendNpcChat1("Would you like to change your appearance?", c.talkingNpc, "Make-Over Mage");
    			c.nextChat = 22;
    			break;
    		case 22:
    			sendOption2("Yes!", "No.");
    			c.dialogueAction = 22;
    			break;
    		}
    	}
    ActionHandler Class
    Code:
    /*
    * Make-Over Mage First Option
    */
    		case 599:
    			c.getDH().sendDialogues(20, 599);
    			//c.getPA().showInterface(3559);
    			//c.canChangeAppearance = true;
    /*
    * Make-Over Mage Second Option
    */
    case 599: 		
    			 c.getPA().showInterface(3559);
    			 c.canChangeAppearance = true;

    ClickingButtons Class

    Code:
    case 9157:// barrows tele to tunnels
    			if(c.dialogueAction == 22) {
    				c.getPA().showInterface(3559);
        			c.canChangeAppearance = true;	
        		}
    Spoiler for More Info:

    Project Thread: Project-Sirius

    post your whole 9157 case
    Reply With Quote  
     

  4. Thankful user:


  5. #3  
    Registered Member
    Zivik's Avatar
    Join Date
    Oct 2007
    Age
    28
    Posts
    4,421
    Thanks given
    891
    Thanks received
    1,527
    Rep Power
    3285
    Try:

    Code:
    case 9157:// barrows tele to tunnels
    			if(c.dialogueAction == 22) {
    				c.getPA().showInterface(3559);
        			c.canChangeAppearance = true;	
                           return;
        		}
    If that doesn't work, I'll need to see your entire case 9157 as said above.
    Reply With Quote  
     

  6. Thankful user:


  7. #4  
    Registered Member Sirius_'s Avatar
    Join Date
    Nov 2013
    Age
    25
    Posts
    269
    Thanks given
    77
    Thanks received
    53
    Rep Power
    11
    Quote Originally Posted by The Reason View Post
    post your whole 9157 case
    Quote Originally Posted by Zivik View Post
    Try:

    Code:
    case 9157:// barrows tele to tunnels
    			if(c.dialogueAction == 22) {
    				c.getPA().showInterface(3559);
        			c.canChangeAppearance = true;	
                           return;
        		}
    If that doesn't work, I'll need to see your entire case 9157 as said above.
    Code:
    case 9157:// barrows tele to tunnels
    			if(c.dialogueAction == 22) {
    				c.getPA().showInterface(3559);
        			c.canChangeAppearance = true;	
        		}
    			if (c.dialogueAction == 1) {
    				int r = 4;
    				// int r = Misc.random(3);
    				switch (r) {
    				case 0:
    					c.getPA().movePlayer(3534, 9677, 0);
    					break;
    
    				case 1:
    					c.getPA().movePlayer(3534, 9712, 0);
    					break;
    
    				case 2:
    					c.getPA().movePlayer(3568, 9712, 0);
    					break;
    
    				case 3:
    					c.getPA().movePlayer(3568, 9677, 0);
    					break;
    				case 4:
    					c.getPA().movePlayer(3551, 9694, 0);
    					break;
    				}
    			} else if (c.dialogueAction == 2) {
    				c.getPA().movePlayer(2507, 4717, 0);
    			} else if (c.dialogueAction == 5) {
    				c.getSlayer().giveTask();
    			} else if (c.dialogueAction == 6) {
    				c.getSlayer().giveTask2();
    			} else if (c.dialogueAction == 7) {
    				c.getPA().startTeleport(3088, 3933, 0, "modern");
    				c.sendMessage("NOTE: You are now in the wilderness...");
    			} else if (c.dialogueAction == 8) {
    				c.getPA().resetBarrows();
    				c.sendMessage("Your barrows have been reset.");
    			}
    			c.dialogueAction = 0;
    			c.getPA().removeAllWindows();
    			break;
    Reply With Quote  
     

  8. #5  
    Registered Member
    Zivik's Avatar
    Join Date
    Oct 2007
    Age
    28
    Posts
    4,421
    Thanks given
    891
    Thanks received
    1,527
    Rep Power
    3285
    Try:

    Code:
    case 9157:// barrows tele to tunnels
    			if(c.dialogueAction == 22) {
    				c.getPA().showInterface(3559);
        			c.canChangeAppearance = true;	
                            return;
        		}
    			if (c.dialogueAction == 1) {
    				int r = 4;
    				// int r = Misc.random(3);
    				switch (r) {
    				case 0:
    					c.getPA().movePlayer(3534, 9677, 0);
    					break;
    
    				case 1:
    					c.getPA().movePlayer(3534, 9712, 0);
    					break;
    
    				case 2:
    					c.getPA().movePlayer(3568, 9712, 0);
    					break;
    
    				case 3:
    					c.getPA().movePlayer(3568, 9677, 0);
    					break;
    				case 4:
    					c.getPA().movePlayer(3551, 9694, 0);
    					break;
    				}
    			} else if (c.dialogueAction == 2) {
    				c.getPA().movePlayer(2507, 4717, 0);
    			} else if (c.dialogueAction == 5) {
    				c.getSlayer().giveTask();
    			} else if (c.dialogueAction == 6) {
    				c.getSlayer().giveTask2();
    			} else if (c.dialogueAction == 7) {
    				c.getPA().startTeleport(3088, 3933, 0, "modern");
    				c.sendMessage("NOTE: You are now in the wilderness...");
    			} else if (c.dialogueAction == 8) {
    				c.getPA().resetBarrows();
    				c.sendMessage("Your barrows have been reset.");
    			}
    			c.dialogueAction = 0;
    			c.getPA().removeAllWindows();
    			break;
    Reply With Quote  
     

  9. Thankful user:


  10. #6  
    Registered Member
    Join Date
    Sep 2013
    Posts
    880
    Thanks given
    43
    Thanks received
    57
    Rep Power
    58
    Quote Originally Posted by Sirius_ View Post
    Code:
    case 9157:// barrows tele to tunnels
    			if(c.dialogueAction == 22) {
    				c.getPA().showInterface(3559);
        			c.canChangeAppearance = true;	
        		}
    			if (c.dialogueAction == 1) {
    				int r = 4;
    				// int r = Misc.random(3);
    				switch (r) {
    				case 0:
    					c.getPA().movePlayer(3534, 9677, 0);
    					break;
    
    				case 1:
    					c.getPA().movePlayer(3534, 9712, 0);
    					break;
    
    				case 2:
    					c.getPA().movePlayer(3568, 9712, 0);
    					break;
    
    				case 3:
    					c.getPA().movePlayer(3568, 9677, 0);
    					break;
    				case 4:
    					c.getPA().movePlayer(3551, 9694, 0);
    					break;
    				}
    			} else if (c.dialogueAction == 2) {
    				c.getPA().movePlayer(2507, 4717, 0);
    			} else if (c.dialogueAction == 5) {
    				c.getSlayer().giveTask();
    			} else if (c.dialogueAction == 6) {
    				c.getSlayer().giveTask2();
    			} else if (c.dialogueAction == 7) {
    				c.getPA().startTeleport(3088, 3933, 0, "modern");
    				c.sendMessage("NOTE: You are now in the wilderness...");
    			} else if (c.dialogueAction == 8) {
    				c.getPA().resetBarrows();
    				c.sendMessage("Your barrows have been reset.");
    			}
    			c.dialogueAction = 0;
    			c.getPA().removeAllWindows();
    			break;
    try this

    Code:
    		case 9157:// barrows tele to tunnels	
    			if (c.dialogueAction == 1) {
    				int r = 4;
    				// int r = Misc.random(3);
    				switch (r) {
    				case 0:
    					c.getPA().movePlayer(3534, 9677, 0);
    					break;
    
    				case 1:
    					c.getPA().movePlayer(3534, 9712, 0);
    					break;
    
    				case 2:
    					c.getPA().movePlayer(3568, 9712, 0);
    					break;
    
    				case 3:
    					c.getPA().movePlayer(3568, 9677, 0);
    					break;
    				case 4:
    					c.getPA().movePlayer(3551, 9694, 0);
    					break;
    				}
    			} else if (c.dialogueAction == 2) {
    				c.getPA().movePlayer(2507, 4717, 0);
    			} else if (c.dialogueAction == 5) {
    				c.getSlayer().giveTask();
    			} else if (c.dialogueAction == 6) {
    				c.getSlayer().giveTask2();
    			} else if (c.dialogueAction == 7) {
    				c.getPA().startTeleport(3088, 3933, 0, "modern");
    				c.sendMessage("NOTE: You are now in the wilderness...");
    			} else if (c.dialogueAction == 8) {
    				c.getPA().resetBarrows();
    				c.sendMessage("Your barrows have been reset.");
    			} else if(c.dialogueAction == 22) {
    				c.getPA().showInterface(3559);
    				c.canChangeAppearance = true;
    			}
    			c.dialogueAction = 0;
    			c.getPA().removeAllWindows();
    			break;
    Reply With Quote  
     

  11. Thankful user:


  12. #7  
    Registered Member Sirius_'s Avatar
    Join Date
    Nov 2013
    Age
    25
    Posts
    269
    Thanks given
    77
    Thanks received
    53
    Rep Power
    11
    Not understanding why its still not working. I tried both of you options but it still just sends the same message saying I click action button 9157.

    Spoiler for Start-Up Server:

    Code:
    Loaded 1024 doors in 431ms.
    Loaded 273 Double doors in 15ms.
    Nov 07, 2013 9:40:31 AM server.RS2Server start
    INFO: Waiting for pending background loading tasks...
    Nov 07, 2013 9:40:31 AM server.RS2Server start
    INFO: Setting up login channels...
    Nov 07, 2013 9:40:31 AM server.RS2Server start
    INFO: Project-Sirius accpeting incoming connections...
    System resources cleaned.
    [REGISTERED]: Mod sirius
    Mod sirius - actionbutton: 9157


    Edit: I guess at least the second click method works however that would look very unprofessional if I had dialogue stating please right click me. Haha.
    Reply With Quote  
     

  13. #8  
    Registered Member
    Zivik's Avatar
    Join Date
    Oct 2007
    Age
    28
    Posts
    4,421
    Thanks given
    891
    Thanks received
    1,527
    Rep Power
    3285
    Try adding a sendmessage in it to see if it's atleast calling it.

    Code:
    			if(c.dialogueAction == 22) {
                                    c.sendMessage("Is this showing up");
    				c.getPA().showInterface(3559);
        			c.canChangeAppearance = true;	
                            return;
        		}
    Reply With Quote  
     

  14. #9  
    Registered Member Sirius_'s Avatar
    Join Date
    Nov 2013
    Age
    25
    Posts
    269
    Thanks given
    77
    Thanks received
    53
    Rep Power
    11
    Problem is solved. I put a break statement instead of a return.
    Reply With Quote  
     


Thread Information
Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)


User Tag List

Similar Threads

  1. [PI] Deadlock Issue
    By Santi Cazorla in forum Help
    Replies: 5
    Last Post: 02-21-2011, 02:29 PM
  2. [PI] StreamLoader Issue
    By RuiN in forum Help
    Replies: 7
    Last Post: 10-09-2010, 09:20 PM
  3. [PI] Skillcape Issue
    By Defiled-X in forum Help
    Replies: 0
    Last Post: 09-23-2010, 03:33 AM
  4. [PI] Client Issue
    By coder nick in forum Help
    Replies: 3
    Last Post: 09-20-2010, 08:14 PM
  5. [PI] Food Issue
    By Lenin in forum Help
    Replies: 0
    Last Post: 08-16-2010, 01:15 PM
Posting Permissions
  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •