View Single Post
Old 12-14-2007, 02:37 PM   #4 (permalink)
Haris
The Frequenter
Prolific Welcomer Upcoming Programmer 
 
Join Date: Sep 2007
Posts: 360
Thanks: 24
Haris is on a distinguished road
Default

Quote:
Originally Posted by bdm View Post
Good thinking bmicallef.

Quick things off the top of my head:
- There is no status table.
- Since you're using tinyint as your datatype for user_rank in your users table, you should do the same the the ID in your users_rank table. Tinyint should be more than enough, especdially if you make it unsigned since it'll go up to 255.
- In your comments table, isn't the comments_comments column a little redundant?
  • I don't need a status table. Not necessary. I will have only two status, hide or show i.e 0 or 1.
  • I'll change them. Between, can you explain difference between signed and unsigned? :)
  • It should be comment_comment. I know it sounds confusing. I have two choices, either stay with it due to the table prefix standard in other tables or just change it to comment. The first option sounds better to me but I'll consider a second opinion.
__________________
Necessity is the mother of invention.

My blog
Haris is offline  
Reply With Quote