Moparscape - RSPS Community

RuneScape 2 Development => RS Client Hacking => Topic started by: Ambokile on November 04, 2014, 04:09:42 AM

Title: [194 and 317 compared] Refactored "buildMenuForNPC" method
Post by: Ambokile on November 04, 2014, 04:09:42 AM
Another same-purpose method refactored in both the 317 and the 194 clients to show the difference between the two.

194
Code: Java
  1.         publicfinalvoid buildMenuForNPC(EntityDefinition targetNPC, int data1, int data2, int data3){
  2.                 if(menuActionRow <400){
  3.                         String targetNPCDisplayName = targetNPC.name;
  4.                         // packetSize += dummy; // dummy
  5.                         if(targetNPC.combatLevel!=0)
  6.                                 targetNPCDisplayName +=(getCombatLevelDifferenceColour(targetNPC.combatLevel,
  7.                                                 localPlayer.combatLevel)
  8.                                                 +" (level-"+ targetNPC.combatLevel+")");
  9.                         if(itemSelected ==1){
  10.                                 menuActionName[menuActionRow]="Use "+ selectedItemName +" with @[email protected]"
  11.                                                 + targetNPCDisplayName;
  12.                                 menuActionId[menuActionRow]=806;
  13.                                 menuActionData1[menuActionRow]= data1;
  14.                                 menuActionData2[menuActionRow]= data2;
  15.                                 menuActionData3[menuActionRow]= data3;
  16.                                 menuActionRow++;
  17.                         }elseif(spellSelected ==1){
  18.                                 if((selectedSpellUsability & 0x2)==2){
  19.                                         menuActionName[menuActionRow]= selectedSpellTooltip +" @[email protected]"+ targetNPCDisplayName;
  20.                                         menuActionId[menuActionRow]=130;
  21.                                         menuActionData1[menuActionRow]= data1;
  22.                                         menuActionData2[menuActionRow]= data2;
  23.                                         menuActionData3[menuActionRow]= data3;
  24.                                         menuActionRow++;
  25.                                 }
  26.                         }else{
  27.                                 if(targetNPC.actionNames!=null){
  28.                                         for(int a =4; a >=0; a--){
  29.                                                 if(targetNPC.actionNames[a]!=null){
  30.                                                         menuActionName[menuActionRow]=(targetNPC.actionNames[a]
  31.                                                                         +" @[email protected]"+ targetNPCDisplayName);
  32.                                                         if(a ==0)
  33.                                                                 menuActionId[menuActionRow]=710;
  34.                                                         if(a ==1)
  35.                                                                 menuActionId[menuActionRow]=301;
  36.                                                         if(a ==2)
  37.                                                                 menuActionId[menuActionRow]=328;
  38.                                                         if(a ==3)
  39.                                                                 menuActionId[menuActionRow]=498;
  40.                                                         if(a ==4)
  41.                                                                 menuActionId[menuActionRow]=74;
  42.                                                         menuActionData1[menuActionRow]= data1;
  43.                                                         menuActionData2[menuActionRow]= data2;
  44.                                                         menuActionData3[menuActionRow]= data3;
  45.                                                         menuActionRow++;
  46.                                                 }
  47.                                         }
  48.                                 }
  49.                                 menuActionName[menuActionRow]="Examine @[email protected]"+ targetNPCDisplayName;
  50.                                 menuActionId[menuActionRow]=1725;
  51.                                 menuActionData1[menuActionRow]= data1;
  52.                                 menuActionData2[menuActionRow]= data2;
  53.                                 menuActionData3[menuActionRow]= data3;
  54.                                 menuActionRow++;
  55.                         }
  56.                 }
  57.         }

317
Code: Java
  1.         privatevoid buildMenuForNPC(EntityDefinition targetNPC, int data1,
  2.                         int data3, int data2){
  3.                 if(menuActionRow >=400)
  4.                         return;
  5.                 if(targetNPC.childrenIDs!=null)
  6.                         targetNPC = targetNPC.getChildDefinition();
  7.                 if(targetNPC ==null)
  8.                         return;
  9.                 if(!targetNPC.clickable)
  10.                         return;
  11.                 String targetNPCDisplayName = targetNPC.name;
  12.                 if(targetNPC.combatLevel!=0)
  13.                         targetNPCDisplayName = targetNPCDisplayName
  14.                                         + getCombatLevelDifferenceColour(localPlayer.combatLevel,
  15.                                                         targetNPC.combatLevel)+" (level-"
  16.                                         + targetNPC.combatLevel+")";
  17.                 if(itemSelected ==1){
  18.                         menuActionName[menuActionRow]="Use "+ selectedItemName
  19.                                         +" with @[email protected]"+ targetNPCDisplayName;
  20.                         menuActionId[menuActionRow]=582;
  21.                         menuActionData1[menuActionRow]= data1;
  22.                         menuActionData2[menuActionRow]= data2;
  23.                         menuActionData3[menuActionRow]= data3;
  24.                         menuActionRow++;
  25.                         return;
  26.                 }
  27.                 if(spellSelected ==1){
  28.                         if((selectedSpellUsability &2)==2){
  29.                                 menuActionName[menuActionRow]= selectedSpellTooltip +" @[email protected]"
  30.                                                 + targetNPCDisplayName;
  31.                                 menuActionId[menuActionRow]=413;
  32.                                 menuActionData1[menuActionRow]= data1;
  33.                                 menuActionData2[menuActionRow]= data2;
  34.                                 menuActionData3[menuActionRow]= data3;
  35.                                 menuActionRow++;
  36.                         }
  37.                 }else{
  38.                         if(targetNPC.actions!=null){
  39.                                 for(int a =4; a >=0; a--)
  40.                                         if(targetNPC.actions[a]!=null
  41.                                                         &&!targetNPC.actions[a]
  42.                                                                         .equalsIgnoreCase("attack")){
  43.                                                 menuActionName[menuActionRow]= targetNPC.actions[a]
  44.                                                                 +" @[email protected]"+ targetNPCDisplayName;
  45.                                                 if(a ==0)
  46.                                                         menuActionId[menuActionRow]=20;
  47.                                                 if(a ==1)
  48.                                                         menuActionId[menuActionRow]=412;
  49.                                                 if(a ==2)
  50.                                                         menuActionId[menuActionRow]=225;
  51.                                                 if(a ==3)
  52.                                                         menuActionId[menuActionRow]=965;
  53.                                                 if(a ==4)
  54.                                                         menuActionId[menuActionRow]=478;
  55.                                                 menuActionData1[menuActionRow]= data1;
  56.                                                 menuActionData2[menuActionRow]= data2;
  57.                                                 menuActionData3[menuActionRow]= data3;
  58.                                                 menuActionRow++;
  59.                                         }
  60.  
  61.                         }
  62.                         if(targetNPC.actions!=null){
  63.                                 for(int a =4; a >=0; a--){
  64.                                         if(targetNPC.actions[a]!=null
  65.                                                         && targetNPC.actions[a].equalsIgnoreCase("attack")){
  66.                                                 int modifier =0;
  67.                                                 if(targetNPC.combatLevel> localPlayer.combatLevel)
  68.                                                         modifier =2000;
  69.                                                 menuActionName[menuActionRow]= targetNPC.actions[a]
  70.                                                                 +" @[email protected]"+ targetNPCDisplayName;
  71.                                                 if(a ==0)
  72.                                                         menuActionId[menuActionRow]=20+ modifier;
  73.                                                 if(a ==1)
  74.                                                         menuActionId[menuActionRow]=412+ modifier;
  75.                                                 if(a ==2)
  76.                                                         menuActionId[menuActionRow]=225+ modifier;
  77.                                                 if(a ==3)
  78.                                                         menuActionId[menuActionRow]=965+ modifier;
  79.                                                 if(a ==4)
  80.                                                         menuActionId[menuActionRow]=478+ modifier;
  81.                                                 menuActionData1[menuActionRow]= data1;
  82.                                                 menuActionData2[menuActionRow]= data2;
  83.                                                 menuActionData3[menuActionRow]= data3;
  84.                                                 menuActionRow++;
  85.                                         }
  86.                                 }
  87.                         }
  88.                         menuActionName[menuActionRow]="Examine @[email protected]"+ targetNPCDisplayName
  89.                                         +" @[email protected](@[email protected]"+ targetNPC.id+"@[email protected])";
  90.                         menuActionId[menuActionRow]=1025;
  91.                         menuActionData1[menuActionRow]= data1;
  92.                         menuActionData2[menuActionRow]= data2;
  93.                         menuActionData3[menuActionRow]= data3;
  94.                         menuActionRow++;
  95.                 }
  96.         }

All dummies have been removed from both versions.