Suggestions, Bugs, Issues with my program, Please Rate

[I_Love_My_Vans]

Hello all, well i have just finished a Diary program, used to store Diary entry's from multiple users. I would really like it if you all could download it, and tell me if any errors occured, anything i could do to make it better, little things like this to make the program itself better.

If you could add your comments below.

I already have my girlfriend testing it, but she knows little about computers, or programming.

Thank You So Much
This will help alot

ILMV aka Shabba


REVISIONS

Revision 1) Add SQL to Add User procedure
Revision 2) Accept and Cancel buttons on User Options form have been disabled. Will be enabled when the user decideds to eithe change a password, or delete a user.

[Jacob Roman]

This program should be moved into the CodeBank: Visual Basic 6 and Before section. Have MartinLiss do that for ya.

[I_Love_My_Vans]

Thank You

*Calls MartinLiss* Bonjour, Could you please move

[wiz126]

ok here is a bug once i click on login while i have username and password balnk i get this:

[I_Love_My_Vans]

Thank you, simple mistake.

Bug Fixed >> New ZIP added to first post

[dee-u]

Trying to download it I get this error message: Invalid Attachment specified. If you followed a valid link, please notify the webmaster.

[I_Love_My_Vans]

Um? i can, try again, if it does not work i will host it myself and link you to it.

[wiz126]

yeap works fined for me

[dee-u]

It works now. Seems fine with me but I havent checked it much.

[Hack]

Moved to the CodeBank.

Per CodeBank attachments policy, which can be read here, the attachment has been edited, and the compiled .exe removed.

[I_Love_My_Vans]

Hey? What has been edited!?

I dont mind sticking to the rules, i sahll not include an EXE next time

[Hack]

Hey? What has been edited!?

Just the attachment.

You an include anything in a CodeBank attachment that you wish just as long as it is source code and not compiled.

[wiz126]

I_Love_My_Vans,

you app looks good after you fixed the bug i had,

just one question? what is the point of the cancel button in the use options form, for some reason it does nothing. i thought it supos to cancel the things i did but when i made a new user and click on cancell it did nothing y is that?

[I_Love_My_Vans]

The cancel button is for use with the Delete User, and Change Password procedures.

To avoid confusion i have disabled the buttons until they are needed. Please check the zip file in the first post for new revision.

[CVMichael]

It looks nice...

But here are a couple of suggestions:
About the database:
1) The Password field you've set the input mask of "Password" to hide the password... that's not good enough, I just deleted the input mask, and I could see the password. To properly do it, I sugest to HASH the password before you insert the record in the database.

2) The tables are linked by username (a text field), not good... every user has to have an ID (numeric field, Long Integer), and you should link the tables by ID's...

About the program:
You should have a way to list all the diary entries you've set, right now, you have to click on each day to see if you have anything there, it is quite a pain to click and check for every day until you find the one you were looking for.

You should not make the Recordset global (only the connection), the recordset should be declared at function/sub level whenever you need to use it.

And in this statement:

Public UniUsername, Sqltxt, Flag, AddUser, OptionFlag, OptionExitFlag As String

Only OptionExitFlag is declared as String type, the rest of the variables are Variant Type...
You should declare like:

Public UniUsername As String, Sqltxt As String, Flag As String, AddUser As String, OptionFlag As String, OptionExitFlag As String

Anyways, those are just quick things I picked up while looking through the code, but overall I like it, it's quite nice...

[I_Love_My_Vans]

Thank you, i will look to make those changes, the only problem i have is i dont know what you mean by HASH the password?

Also, i dont want anyone to see the records within the Entry's Table, do you now how to hide them?

[CVMichael]

Thank you, i will look to make those changes, the only problem i have is i dont know what you mean by HASH the password?

I will make the changed to the project you posted in the first post, and add encryption for the password. I will post back the project with the changes.

Also, i dont want anyone to see the records within the Entry's Table, do you now how to hide them?

Why don't you set a password for the whole database ? that way when you try to open it with Microsoft Access, it will ask for a password.

[I_Love_My_Vans]

Sounds god, excuse my being a div, but i dont know how to do that. I will do a search, i am remembering someone said VB would have to hold that password for access aswell?

Cheers

[CVMichael]

Sounds god, excuse my being a div, but i dont know how to do that. I will do a search, i am remembering someone said VB would have to hold that password for access aswell?

Cheers

You can hardcode the password in the application. I know it's not safe enough, but at least it stops the regular user for opening the DB.

I attached the new project.
I made changes to: frmAddUser, frmLogIn, frmUserOptions
I also added the class clsMD5.cls to the project

[I_Love_My_Vans]

Nice one

Cheers CVMichael, appreciate the help

[Eliminator2009]

Nice one

Cheers CVMichael, appreciate the help

I have run the project it says "Can't find project or library"

Public MyDatabase As Database

and Displays Reference Dialog box

Thanks

[I_Love_My_Vans]

This project is no longer being maintained as I don't program VB6 anymore, feel free to mess about with the code yourself.

Cheers, Ben