[SOLVED] help select + menu

[namerpus18]

Good Day Everyone,

Please help me I want to make all cards in my inventory show in the select menu. 

Thank you so much, ~NOOB

prontera,164,200,4	script	selectcard	112,{
getinventorylist();
setarray .@menulist[0],0;
.@j = 0; 

	for (.@i = 0; .@i < @inventorylist_count; ++.@i)
	{
		.@idd = @inventorylist_id[.@i];
		.@type = getiteminfo(.@idd,2); 
			if (.@type == 6)
			{
			
			.@menulist[@j] = .@idd;
			
			.@j++;
	
			dispbottom .@menulist[@j]; 
			}
			
	}
	
	for (.@i = 0; .@i < getarraysize(.@menulist); ++.@i)
		{

		.@menu$ = .@menu$ + getitemname(.@menulist[.@i]);
		.@menu$ = .@menu$ + ":";
		.@part = .@menulist[select(.@menu$)];

		}
		.@part = .@menulist[select(.@menu$)];

end;	
}

 

Edited November 7, 2023 by namerpus18

[Emistry]

getinventorylist;
for (.@i = 0; .@i < @inventorylist_count; .@i++) {
	if (getiteminfo(@inventorylist_id[.@i], ITEMINFO_TYPE) == IT_CARD)
		.@menu$ = .@menu$ + getitemname(@inventorylist_id[.@i]);
	.@menu$ = .@menu$ + ":";
}
.@i = select(.@menu$) - 1;
mes "Your selected card = " + getitemname(@inventorylist_id[.@i]);

you can also do this this way.

[Racaae]

Hi, In the following line there's a missing .(dot) in the variable name:

.@menulist[@j] = .@idd;

To:

.@menulist[.@j] = .@idd;

 

In this part you wrote the same select command line twice:

	for (.@i = 0; .@i < getarraysize(.@menulist); ++.@i)
		{

		.@menu$ = .@menu$ + getitemname(.@menulist[.@i]);
		.@menu$ = .@menu$ + ":";
		.@part = .@menulist[select(.@menu$)];

		}
		.@part = .@menulist[select(.@menu$)];

Remove the first one, and add -1 in the second. Like this:

	for (.@i = 0; .@i < getarraysize(.@menulist); ++.@i)
		{

		.@menu$ = .@menu$ + getitemname(.@menulist[.@i]);
		.@menu$ = .@menu$ + ":";

		}
		.@part = .@menulist[select(.@menu$) - 1];

 

[namerpus18]

38 minutes ago, Racaae said:

Hi, In the following line there's a missing .(dot) in the variable name:

Hi thank you so much, it worked.