can somebody fix this script for me please?

[eboni001]

hello, can somebody know why when i talk to this npc only say: You already claim your free items? i tried with lot of diferents chars but allways the same.

 

prontera,146,159,6	script	Novice Support	893,{

// Name of the NPC
		set .name$, "[Emely]";

    if( #novice = 1){
    mes .name$;
	emotion	e_swt2;
    mes "You already claim your free items.";
    close;
		}
		else
		{
	mes .name$;
	if (Class	<	1)	{
		emotion	e_ho;
		mes "Hello young adventurer and welcome to Kyojin RO";
		mes "I can give you some free items to help you grown fast";
		mes "Do you want it?";
		next;
		switch(select(	"You're so kind. Thank you!:Nah~ Im too pro for this!"	)){
		Case 1:
		mes .name$;
		emotion	e_kis;
		mes "Please take this item and have a nice day~";
		next;
		getitem	5172,1;	//Beret
		getitem	645,25;	//Concentration potion
		getitem	14533,10;	//Field Manual
		getitem	14232,10;	//ygg berry box(10)
		getitem	4142,1;	//Doppelganger Card
		set #novice, 1;
		mes "See you soon~";
		break;
		Case 2:
		mes .name$;
		emotion	e_ag;
		mes "Such a arrogant person.";
		mes "^ff0000@kill "+ strcharinfo(0) +"^000000";
		next;
		mes .name$;
		emotion	e_omg;
		mes "Why my command not work?!";
		mes "Maybe you're lucky!";
			break;
			}
	close;
}
		else
		{
if (Class	<	24)	{
		mes "Congratulations!";
		mes "I see you already reborn.";
		close;
		}
end;
}
}
}

 

[-SkittleNugget-]

Rewrote it all for you.

Would it not be better to auto give freebies to new people rather than clicking an NPC

//============================================
//= Npc that gives freebies to the new
//============================================
//= Rewritten by SkittleNugget (rAthena.org)
//============================================
prontera,146,159,6	script	Novice Support	893,{

    if( #PlayerRecievedFreebies )
	{
		mes "[Emely]";
		emotion	e_swt2;
		mes "You already claimed your free items.";
		close;
	}
		else if ( Class < 1 )
	{
		mes "[Emely]";
		emotion	e_ho;
		mes "Hello young adventurer and welcome to Kyojin RO";
		mes "I can give you some free items to help you grown fast";
		mes "Do you want it?";
		next;
		if ( select("You're so kind. Thank you!:Nah~ Im too pro for this!")==2)
		{
			mes "[Emely]";
			emotion	e_ag;
			mes "Such a arrogant person.";
			mes "^ff0000@kill "+ strcharinfo(0) +"^000000";
			next;
			mes "[Emely]";
			emotion	e_omg;
			mes "Why my command not work?!";
			mes "Maybe you're lucky!";
			close;
		}
		mes "[Emely]";
		emotion	e_kis;
		mes "Please take this item and have a nice day~";
		next;
		getitem	5172,1;	//Beret
		getitem	645,25;	//Concentration potion
		getitem	14533,10;	//Field Manual
		getitem	14232,10;	//ygg berry box(10)
		getitem	4142,1;	//Doppelganger Card
		set #PlayerRecievedFreebies, 1;
		mes "See you soon~";
		close;
	}
}

[Skorm]

prontera,146,159,6	script	Novice Support	893,{

// Name of the NPC

set .name$, "[Emely]";

if(#novice == 1) {

mes .name$;

emotion e_swt2;

mes "You already claim your free items.";

close;

} else {

mes .name$;

if(Class < 1) {

emotion e_ho;

mes "Hello young adventurer and welcome to Kyojin RO";

mes "I can give you some free items to help you grown fast";

mes "Do you want it?";

next;

switch( select("You're so kind. Thank you!:Nah~ Im too pro for this!") ) {

Case 1:

mes .name$;

emotion e_kis;

mes "Please take this item and have a nice day~";

next;

getitem 5172,1; //Beret

getitem 645,25; //Concentration potion

getitem 14533,10; //Field Manual

getitem 14232,10; //ygg berry box(10)

getitem 4142,1; //Doppelganger Card

set #novice, 1;

mes "See you soon~";

break;

Case 2:

mes .name$;

emotion e_ag;

mes "Such a arrogant person.";

mes "^ff0000@kill "+ strcharinfo(0) +"^000000";

next;

mes .name$;

emotion e_omg;

mes "Why my command not work?!";

mes "Maybe you're lucky!";

break;

}

close;

} else {

if(Class < 24) {

mes "Congratulations!";

mes "I see you already reborn.";

close;

}

end;

}

}

}

Tell me if you're having the same problem. Edited July 4, 2013 by Skorm

[Patskie]

This code :

if( #novice = 1) 

 

must be : 

if( #novice == 1)

[eboni001]

Really Thank you guys this is working Awesome   all of you are the best.

Edited July 4, 2013 by eboni001