[C++] Help! :)

[Dr. Livingstone]

I've been working on this reputation NPC. Which I released, but now I want to make it cost an item. Not gold. So lets say I wanted to make it cost 5 items with the intid of 558801... What would I do? Like what would the script look like? This would be a huge help to me, and +Rep if you can help me out.

[Sonic Waffle]

Make it use a quest to get the requirements. Its way more handy...

[Dr. Livingstone]

Hmm... Could you explain more? I was hoping for something people wouldn't have to talk to the NPC twice. Once to get the quest and once to turn in. So I thought this would be better...

[Sonic Waffle]

Well this is the simpliest solution I know of... Unless you know how to code C++ or Lua...
Wait a little bit longer and see if anyone has got a better idea, but this is best imo.

[Dr. Livingstone]

Alright. +Rep for helping me out I'm open to other thoughts too

[Aznex]

something like

if(Plr->GetItemInterface()->GetItemCount(ItemID, Amount))
Plr->SetStanding(Faction, Amount);}
Plr->GetItemInterface()->RemoveItemAmt(ItemID, Amount);

[Kalash47]

I've been working on this reputation NPC. Which I released, but now I want to make it cost an item. Not gold. So lets say I wanted to make it cost 5 items with the intid of 558801... What would I do? Like what would the script look like? This would be a huge help to me, and +Rep if you can help me out.

If this is from a quest, then what you should do is make it multiple quests (for reputations).

There should be, in the DB, and Item Requirement column (look up other quests that have you submit items to the quest giver).

In that item requirement column add an ID of the item, and there should be a related "count" column which allows for you to add how much of that item is necessary to complete the quest.

Upon quest completion there should also be a column for rewarding reputation.

A reputation NPC needn't be coded in C++, it can be done straight from the DB.

I know this for a FACT with MaNGOS, but it should be in ArcEmu as well.

[Aznex]

I think he's trying to maximize his C++ Skills, thats why he's doing it this way

[Dr. Livingstone]

Aznex is right Trying to figure out more C++. And I'm hoping to make something that looks different than what most servers/repacks have for reputation. Gotta spread before I can give you rep Aznex

[Sounddead]

Exactly. Hes not asking people for alternative ways to do it. Hes asking how he can do it in C++. And Aznex is right. Something along those lines.

[Kalash47]

I think he's trying to maximize his C++ Skills, thats why he's doing it this way

Aznex is right Trying to figure out more C++. And I'm hoping to make something that looks different than what most servers/repacks have for reputation. Gotta spread before I can give you rep Aznex

Well you need to keep in mind a good practice in programming. Don't do it one way if there's a faster and easier solution. Find a more challenging project in C++ rather than reinventing the wheel, because you won't be learning much rather than doing it another way.

[Dr. Livingstone]

Alright. I've just been trying to get a grasp of C++ when it comes to these emus. Trying to learn as much as I can. Thanks for the advise +Rep

[Sounddead]

You always gotta start somewhere Kalash. You can't just start tackling big projects if you don't even know what your doing. Starting small and working your way up is how people learn

[Kalash47]

Alright. I've just been trying to get a grasp of C++ when it comes to these emus. Trying to learn as much as I can. Thanks for the advise +Rep

You always gotta start somewhere Kalash. You can't just start tackling big projects if you don't even know what your doing. Starting small and working your way up is how people learn

I understand what your saying but even something like this is trivial. It's like a 4th chapter in every common book, calling functions. You give some arguments functions, and you're done.

If you were doing a function from scratch and doing all the insides & outs, that'd be neat and I'd absolutely encourage for you to do it in C++, but all you are doing is calling implementation.

Little correct in Aznex code, because according to function definition:

	uint32 GetItemCount(uint32 itemid, bool IncBank = false);

There is no "amount argument" it just returns how many of "this" items you have. So that's why we say if it's equal to 5 then blah blah.

if (Plr->GetItemInterface()->GetItemCount(ItemID, false) == Amount)
{
        // Aznex's code
	Plr->SetStanding(Faction, Amount);}
	Plr->GetItemInterface()->RemoveItemAmt(ItemID, Amount); 
}
else
{
	Plr->BroadcastMessage("Error: Items not found in inventory");
}

[Sounddead]

I totally understand your point. Just doesn't seem like hes reading/read a book :P