Go Back   vb.org Archive > Community Central > vBulletin.org Site Feedback
  #1  
Old 08-02-2005, 12:18 AM
akanevsky akanevsky is offline
 
Join Date: Apr 2005
Posts: 3,972
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
But then a Quality Label can be seen as saying that some hacks are better than others, not that their code style is just more in-line with what a group of people think it should be...
If labels are important to you, then we can explicitly state "Validated Code", while having a link at an article explaining what this is all about.

Quote:
Honestly - If a hack is that important to me as it is so good, and the code is poorly done, I re-code it myself - The only people I can see complaining about coding standards are standard fanatics... I'd say over half the people who visit vBulletin.org and install hacks from here rarely care if the code is sloppy - As long as it doesn't bring their site to a halt and does the job they want, they're happy...
It does bring sites to halt (due to a high server load that results in parsing of an unoptimized code), and I can give you some examples. I just don't think it is necessary. "A buck is made up of cents".

Quote:
Honestly, all I can see this doing is creating bad feeling, a "division" between coders and eventually tearing a rift in the community...
Quote:
Thay way no one would fell slighted or small or discouraged for not knowing the standard and having it pointed out
You are missing one: there is already a division: "Coder"/"Adv. Coder"/"Master Coder". And yet - there is no tearing of the community. There are, instead, coders who are struggling to get a higher status by making more useful hacks that will be installed by a larger amount of people.

Quote:
As with my previous post, it creates a rift - Such a system would effectively be the same as slapping a 2nd-rate sticker onto non-QA approved hacks...
Ah.. I see. And the "Coder" and "Adv.Coder" are like slapping a 2nd-rate and 3rd-rate sticker on coders themselves (according to your point of view)?

Quote:
My style, except for when I get lazy, was ingrained in me while working at software companies where maintainable, understandable code takes precedence over optimized code. So, while your one-line replacements for my code blocks are appropriate style, they aren't what I'm going for.
What I suggested in those posts was my personal preference, rather than a standard. From your perspective, one liners are harder to read, and I agree with you. This easiness-to-read, is, though, exactly what the vBulletin code standard intended for. No need for worries.

However, I'd like to point out that in one of those three posts I pointed out how to remove three (!!) unnecessary queries from the hack. These unnecessary queries are exactly what I'd like to try to avoid.

Quote:
THEN, we can add features and code refinements.
There is no such thing as "THEN". "THEN" the hacks will be already installed by everyone, and there will be not point in refining the code. Besides, you usually lose interest in older hacks and do not feel like returning to those later on. Esspecially if they are large hacks. It will be just physically impossible to revise them.

--
P.S. Hellsatan, please do not quote whole posts, ok? You are making the thread cluttered and hard to read. Please edit your posts and remove the quotes. Thanks.
Closed Thread

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. The time now is 06:00 PM.


Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.
X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.03802 seconds
  • Memory Usage 2,295KB
  • Queries Executed 12 (?)
More Information
Template Usage:
  • (1)SHOWTHREAD
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (1)ad_showthread_beforeqr
  • (7)bbcode_quote
  • (1)footer
  • (1)forumjump
  • (1)forumrules
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (1)navbar
  • (3)navbar_link
  • (120)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)showthread_list
  • (1)spacer_close
  • (1)spacer_open
  • (1)tagbit_wrapper 

Phrase Groups Available:
  • global
  • inlinemod
  • postbit
  • posting
  • reputationlevel
  • showthread
Included Files:
  • ./showthread.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_threadedmode.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
  • showthread_start
  • showthread_getinfo
  • forumjump
  • showthread_post_start
  • showthread_query_postids_threaded
  • showthread_threaded_construct_link
  • showthread_query
  • bbcode_fetch_tags
  • bbcode_create
  • showthread_postbit_create
  • postbit_factory
  • 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
  • tag_fetchbit_complete
  • forumrules
  • navbits
  • navbits_complete
  • showthread_complete