Jump to content

Recommended Posts

Posted (edited)

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?

  • newXml.PNG.72fd05b2f10a38776e178dcde7d94e1e.PNG
  • 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:

 

index.PNG.59092a0309d36ec906008d974be7b10b.PNGlist.PNG.3aba01faf612e9401c9108172dac4b11.PNGbronze.PNG.98fff4def7ce2716964d124da6b0ae77.PNGsilver.PNG.efb551ecf4863acb5e50198b40088da5.PNGgold.PNG.50ff4c6f395882f1ac8cfb90c7767fbe.PNGplatinum.PNG.95ba89f3b728ce3908678f6f98622116.PNGdiamond.PNG.e91383d2dfb11e980908b43cfc1433c2.PNGmaster.PNG.8229adbaf057a4b03d6062d0de60a959.PNGgrandmaster.PNG.5da5a0c401b3c1092f41d4f7f2217da6.PNG

 

 

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

dyes.PNG.b4c2dfeffa62d594c128dcc3f997a148.PNG

 

 

code

data/html/mods/donate/

data/xml

 

 

Edited by melron
  • Like 1
  • Thanks 2
  • Upvote 1

Share this post


Link to post
Share on other sites
Posted (edited)
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 by melron

Share this post


Link to post
Share on other sites
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

Share this post


Link to post
Share on other sites

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.

Share this post


Link to post
Share on other sites

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)

Share this post


Link to post
Share on other sites
1 minute ago, melron said:

Improved by elfo . Thanks!

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.

Share this post


Link to post
Share on other sites
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? 

Share this post


Link to post
Share on other sites
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?

Share this post


Link to post
Share on other sites
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 :) :) :)

Share this post


Link to post
Share on other sites

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

Share this post


Link to post
Share on other sites
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

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

  • Recently Browsing   0 members

    No registered users viewing this page.



  • Posts

    • Who can tell me how i can edit npc enchant with different enchant? i mean i have one npc for max enchant +150. This item need to be +0 to enchant it. and i want the secound one to be from 150 (not from +0) to +300.  Now, the problem is it in .JAVA file. How i can duplicate npc to work both? I can change just id? (i try this and doesn;t work). Thank's
    • Some words about your ideas:   You are moving wrong.   Players in 2k18 wont see the details you mentioned. What they need?   30 minutes to get S grade 2 days to get max enchant play until become hero (no worries about olympiad system if its good or not) move to another server. They wont look if your project is aCis or frozen They wont prefer your server instead of 'mine' cause of your olympiad. They wont share your website link because of those ideas.   You can add whatever you want inside, the way you described the oly seems good but do not even add it as a feature or what ever.   Why should you focus on olympiad balance since all those years they learned an unbalanced olympiad?     Do you think that all your effort - money will be a good idea for them?     Some words about the team you looking for:   Do not search for people by doing thins. By saying your idea is just a terrible thing that will brake all the things you have in your head. why? because:   A l2dev wont even read what you wrote up there A wannabe admin that do not know what 'code' means , he will agree but guess what. He wont even give you a good idea for you just because he isnt in your head and he will just want to 'part of the X project' A single person who looking for a guy to open a server is just ... NO... get real...   By saying what you want to do, you are not getting what exactly you need, you are actually seem like a guy who wants 1-2 people to give some money for you.    Just make a classic interlude AT FEATURES (you know what i mean) , add what ever code comes in your mind and then, find a good and trusted guy to be a partner. 
  • Topics

×