Thread: LoadQuestInterface() Clean Method

Loops Loops Loops!!!

What do you mean? It's a clean one?

You can shorten that method with a nice array and loop

Eh, i prefer this way. Plus, never been bothered to use loops and arrays.

ooh nice :] green yellow and red =]]] thanks again inside

Originally Posted by Inside Sin

I do not take all the credits, just a small part in packaging around 15 - 20 more. Please don't leech.

Complete Method:

Code:

public void LoadQuestInterface() {
    sendQuest("MiniCraft Quests", 640);
    sendQuest("Complete Quest List",663);

    /*Free Quests*/
    /*if(q2stage == -1) {
      sendQuest("@[email protected]'s Assistant", 7333);
    }
    if(q2stage >= 1) {
      sendQuest("@[email protected]'s Assistant", 7333);
    }
    if(q2stage == 0) {
      sendQuest("Cook's Assistant", 7333);
    }*/
    
	//sendQuest("", 7334);
	sendQuest("", 7332);
	sendQuest("", 7333);
	sendQuest("", 7334);
    sendQuest("", 7336);
    sendQuest("", 7383);
    sendQuest("", 7339);
    sendQuest("", 7338);
    sendQuest("", 7340);
    sendQuest("", 7346);
    sendQuest("", 7341);
    sendQuest("", 7342);
    sendQuest("", 7337);
    sendQuest("", 7343);
    sendQuest("", 7335);
    sendQuest("", 7344);
    sendQuest("", 7345);
    sendQuest("", 7347);
    sendQuest("", 7348);
    
    /*Members Quests*/
    sendQuest("", 12772);
	// unknown id
    sendQuest("", 7352);
    sendQuest("", 12129);
    sendQuest("", 8438);
    sendQuest("", 12852);
    sendQuest("", 7354);
    sendQuest("", 7355);
    sendQuest("", 7356);
    sendQuest("", 8679);
    sendQuest("", 7459);
    sendQuest("", 7357);
    sendQuest("", 12836);
    sendQuest("", 7358);
    sendQuest("", 7359);
    sendQuest("", 14169);
    sendQuest("", 10115);
	sendQuest("", 15592);
	sendQuest("", 15235);
    sendQuest("", 14604);
    sendQuest("", 7360);
	sendQuest("", 668);
    sendQuest("", 12282);
	sendQuest("", 6024);
	sendQuest("", 191);
	sendQuest("", 14912);
	sendQuest("", 15499);
	sendQuest("", 15352);
	sendQuest("", 15487);
	sendQuest("", 6987);
	sendQuest("", 18306);
	sendQuest("", 16128);
	sendQuest("", 16149);
	sendQuest("", 6027);
    sendQuest("", 13577);
	sendQuest("", 15841);
    sendQuest("", 17510);
	sendQuest("", 12839);
    sendQuest("", 7361);
	sendQuest("", 673);
    sendQuest("", 11857);
	sendQuest("", 7353);
    sendQuest("", 7362);
    sendQuest("", 7363);
    sendQuest("", 7364);
    sendQuest("", 10135);
    sendQuest("", 4508);
    sendQuest("", 11907);
    sendQuest("", 7365);
    sendQuest("", 7366);
    sendQuest("", 7367);
    sendQuest("", 13389);
    sendQuest("", 7368);
    sendQuest("", 11132);
    sendQuest("", 7369);
    sendQuest("", 12389);
    sendQuest("", 13974);
    sendQuest("", 7370);
    sendQuest("", 8137);
    sendQuest("", 7371);
    sendQuest("", 12345);
    sendQuest("", 7372);
    sendQuest("", 8115);
	// unknown id
    sendQuest("", 8576);
    sendQuest("", 12139);
    sendQuest("", 7373);
    sendQuest("", 7374);
    sendQuest("", 8969);
    sendQuest("", 7375);
    sendQuest("", 7376);
    sendQuest("", 1740);
    sendQuest("", 3278);
    sendQuest("", 7378);
    sendQuest("", 6518);
    sendQuest("", 7379);
    sendQuest("", 7380);
    sendQuest("", 7381);
    sendQuest("", 11858);
	// unknown id
    sendQuest("", 9927);
    sendQuest("", 7349);
    sendQuest("", 7350);
    sendQuest("", 7351);
    sendQuest("", 13356);
    /*END OF ALL QUESTS*/
  }

This is the stupidents thing i have ever seen in my life.. all the quests are not there.. and the quest stage (int) method is retarded..


use an array


public int frames[] = {};//all the frames for the frame126 method
public string text[] = {};//all the string text fro the frame126 method
public int questpoints = {};//each amount qp's you get for each quest 0 being the first quest..

sort it out guys wtf?

also i recommend that you use 0 as not complete 1 as complete and any thing else as yellow because each quest has a different amount of stages

Originally Posted by x19

Loop through it, make the code smaller and neater.

also makes you lag babe lol but that is what i have done

Originally Posted by sigex

This is the stupidents thing i have ever seen in my life.. all the quests are not there.. and the quest stage (int) method is retarded..

Of course all 317 quests are

What client are you using?