View Single Post
  #63  
Old 08-02-2005, 10:51 AM
sabret00the's Avatar
sabret00the sabret00the is offline
 
Join Date: Jan 2003
Location: London
Posts: 5,268
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by Revan
I would very much like for you to show me the thread in the PHP forum that reads "Please help me verify this segment of code conforms to vBulletin Coding Standards".
If I ever saw such a thread, I would be happy to clean up the code for the user, and also post a link to the Manual, showing him where to reference for the future.
As for the people saying "If you clean the code for them, they'll never learn.", I disagree. What WILL prohibit learning is no-one they look up to pointing out the flaws in their coding. I could have an user with 5 posts tell me he disagreed with my coding standards (if it weren't for the fact that I already comply to an extensive degree with the vBulletin Coding Standards ), and I would simply ignore him.
no theirs no posts that say is this valid, but their are posts that ask how do you do this? why not show them with valid code?

and with the second thing i was speaking from a personal POV i know that if you tell me my codes wrong despite it working you lose my interest where as if you play with the words and your approach it can work wonders, it's like a woman with cleavage asking for a ride compared to a headmistress looking lass.

why dog on someone because they don't meet your interpretation or what something should be? if theirs no security holes give them time to improve themselves as i assure you for everyone you get to come round to your way of thinking, you'll get another that you caused to give up.

Quote:
Originally Posted by Revan
I propose the following change to this idea:
4 levels of labels for a hack:

1. Valid - The hack complies with the vB Coding Standards both in terms of PHP and Source Code Formatting.
2. Cleared - The hack's PHP is optimised, but it has obvious flaws in Coding Style
3. Warning - Neither the hack's PHP or the hack's Coding Style complies
4. Pending - Pending evaluation (default for new hacks)

If a hacker is determined to become a part of the community, he will accept that he has to learn and not publically release 15 hacks before finally wising up his style.
There could even be a new forum where aspiring hackers can post their "raw" hacks for tips and pointers on how to optimise it and make it comply.

I just feel that I have seen too many horribly coded hacks (one of them is a major hack that is even SOLD, Darkwaltz4 will know what I'm talking about ).
Jelsoft creates out-of-the-box clean and fairly optimised code, they have spent many developer hours making their product more hacker friendly, something they really didn't HAVE do to, but did it anyways. Is it really fair to repay them by allowing any form of garbage (as garbage can come from experienced coders as well as newbies), degrading the performance of their product, then whine to them about how it's running slow? Hardly.
as i said make it automated utilizing PHP and regex's or not at all, that way theirs no people involved to hurt feelings and only the people that want their hacks validated get them validated.
 
X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.01120 seconds
  • Memory Usage 1,771KB
  • Queries Executed 11 (?)
More Information
Template Usage:
  • (1)SHOWTHREAD_SHOWPOST
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (2)bbcode_quote
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)post_thanks_box
  • (1)post_thanks_button
  • (1)post_thanks_javascript
  • (1)post_thanks_navbar_search
  • (1)post_thanks_postbit_info
  • (1)postbit
  • (1)postbit_onlinestatus
  • (1)postbit_wrapper
  • (1)spacer_close
  • (1)spacer_open 

Phrase Groups Available:
  • global
  • postbit
  • reputationlevel
  • showthread
Included Files:
  • ./showpost.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_post_thanks.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_postinfo_query
  • fetch_postinfo
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • showpost_start
  • bbcode_fetch_tags
  • bbcode_create
  • postbit_factory
  • showpost_post
  • postbit_display_start
  • post_thanks_function_post_thanks_off_start
  • post_thanks_function_post_thanks_off_end
  • post_thanks_function_fetch_thanks_start
  • post_thanks_function_fetch_thanks_end
  • post_thanks_function_thanked_already_start
  • post_thanks_function_thanked_already_end
  • fetch_musername
  • postbit_imicons
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • post_thanks_function_can_thank_this_post_start
  • showpost_complete