Raised This Month: $32 Target: $400
 8% 

(vault designed for beginners) nvp.inc


Post New Thread Reply   
 
Thread Tools Display Modes
Craxor
Veteran Member
Join Date: Jan 2016
Location: Romania
Old 06-18-2017 , 10:48   Re: (vault designed for beginners) nvp.inc
Reply With Quote #11

Updated, changed the design and the idea, new opinions
__________________
Project: Among Us
Craxor is offline
Send a message via ICQ to Craxor
Bugsy
AMX Mod X Moderator
Join Date: Feb 2005
Location: NJ, USA
Old 06-25-2017 , 14:06   Re: (vault designed for beginners) nvp.inc
Reply With Quote #12

In my opinion, nVault is already relatively user-friendly. This include essentially just formats a user-specific key for different key values which is something that a beginner user should learn out of the gate before creating a plugin. I think a tutorial on strings/formatex() already exists.

Here are a few things I noticed:
  1. This allows only 1 vault per plugin.
  2. You should allow the user to specify the vault name instead of hard-coding it.
  3. In your open function, you have "Problems with Nvault Passwords .vault" as an error message which makes no sense. You should leave this more broad, something like 'Error opening [vault name] vault."
  4. Your get_integer function doesn't really make sense. Why doesn't it just return iData? If your idea is to make it return 0 if no data exists in the vault then you need to use nvault_lookup() instead of nvault_get() since nvault_get() returns 0 if no data exists or if the stored value is 0. nvault_lookup() allows you to determine whether or not the data exists, but in either case, you need to allow a user to store 0 so you need some other return value if no data was found.
__________________
Bugsy is offline
Craxor
Veteran Member
Join Date: Jan 2016
Location: Romania
Old 06-25-2017 , 16:49   Re: (vault designed for beginners) nvp.inc
Reply With Quote #13

ok updated.

I say already it would be more productive for everyone to learn using directly nvault rather than this, but bugsy now 5 years ago i was searching for something so easy like this , newbies think different and they do not implicate so much in doing things.

and no is not eazy for a newbie.
__________________
Project: Among Us
Craxor is offline
Send a message via ICQ to Craxor
Reply


Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT -4. The time now is 08:32.


Powered by vBulletin®
Copyright ©2000 - 2024, vBulletin Solutions, Inc.
Theme made by Freecode