Code Alternative Donation Manager

[melron]

Hello guys,

I had a request to make a donation manager but when i finished the guy decided to not open his server (wtf?) so the npc is now useless to me.

 

What is this?

• A donation manager that selling packages with items and stats like nobl,rec etc.
• Categories can be added easily i will explain later
• Current added categories:
• Bronze,Silver,Gold,Platinum,Diamond,Master,Grandmaster

 

How to configure it?

• 
• Its configurable via xml

 

How i can add items?

• <Package item="57,500"/>

  Will have as a reward 500 adena.

 

How i can add enchanted item?

• <Package item="7575,1" enchant ="15"/>

  Will have as reward draconic bow +15

 

How to add stats like nobl?

• <Package noblesse="true"/>
  <Package autoDyes="true"/>
  <Package rec="50"/>

  It is can handle noblesse,auto dyes (depends on the current class of the player) and recommends.

 

 

Some photos:

 

 

 

I wont upload video ...

I'm not sure about in which aCis rev is coded but it does not matter.

Also i'm not friend with html designs, so don't hit me :D

 

P.s Auto dyes system is setted up by my opinion about dyes. You can easily edit it in DonationDyes.java

 

 

code

data/html/mods/donate/

data/xml

 

 

Edited July 29, 2018 by melron

[Elfo]

Nice one.

 

Here is a slightly refactored version of the code: https://pastebin.com/5wiEC1TC

[Solomun]

Nice job. Thanks for share :)

[melron]

45 minutes ago, .Elfocrash said:

Nice one.

 

Here is a slightly refactored version of the code: https://pastebin.com/5wiEC1TC

nice one elfo . thanks ;) i'll add the changes in some hours

Edited July 29, 2018 by melron

[Psyancy]

Epic share once again! Thanks for sharing buddy!

[melron]

2 hours ago, Solomun said:

Nice job. Thanks for share :)

 

21 minutes ago, Designatix said:

Epic share once again! Thanks for sharing buddy!

thanks guys

[SickJacken]

26 minutes ago, Designatix said:

Epic share once again! Thanks for sharing buddy!

 

[L2J NexuS]

In my opinion you should change the way you add the price for a category.
It should be like this : 

<Donation category="Bronze" price="ItemID,Amount">

Just my opinion tho.

[melron]

Improved by elfo . Thanks!

 

 

6 minutes ago, L2J NexuS said:

In my opinion you should change the way you add the price for a category.
It should be like this : 

<Donation category="Bronze" price="ItemID,Amount">

Just my opinion tho.

 

It can be done by many ways. I had it like this since it was a request. (i prefer your way too)

[Elfo]

a

Edited October 1, 2021 by Elfocrash

[melron]

52 minutes ago, .Elfocrash said:

You didn't really use the meaningful improvements.

 

Few of the things I changed but you chose to ignore:

 

There are things that should be final but aren't and things that aren't final but they should be, classes and methods responsible for things that shouldn't be (such us the find method having an unjustified Supplier for no reason), streams not used in the best way possible, stringbuilders with size limits that don't make sense, a parseInt method name that is deceiving, method summaries that shouldn't be there, string equality checks that start from the value you need to check against instead of the value you are actually checking and a public singleton constructor that shouldn't be public.

for some reasons i didnt update the pastebin with the latest code . i was checking the code idk what happened.. now it's updated with what ever was missed. 

 

Well thats why im sharing things , it is better for me to see post like this instead of good code etc.

 

34 minutes ago, ImBatman said:

Sir he is amateur don't you see? The whole code he made is amateur style. As i see from your coding you seem like a person who has finished a college and worked in few companies and you try to help by not providing solution but advice instead. Thats very nice for you but for his experience and knowledge he need to give priority in thinking and structure rather than limits and wrong steams. 

 

In this case you are even worse. At least he spent some minutes to update my code . what about you? ofc i'm amateur , do you think he does't know that? 

[melron]

17 minutes ago, ImBatman said:

Thanks for calling me worse. I'm glad you felt offended because i called you amateur. Nice moderator respond from a person who wrote a positive comment toward someone

No I didn't felt anything. If you want to be the smart one, prove your words to him in order to learn too your better way. You are saying that the way he is talking isn't correct for amateurs even if he is helping by writing some codes. On the other hand you did nothing except telling him he is wrong. What's the point then?

[L2J NexuS]

3 hours ago, .Elfocrash said:

There are things that should be final but aren't and things that aren't final but they should be

I think this means the same thing :) :) :)

[xxdem]

Didn't read the code, but from the looks of it it seems so bad... Why would someone use this instead of multisells

[melron]

47 minutes ago, xxdem said:

Didn't read the code, but from the looks of it it seems so bad... Why would someone use this instead of multisells

This was my first thought when i heard about it. The guy should had his purpose idk :P