vb.org Archive

vb.org Archive (https://vborg.vbsupport.ru/index.php)
-   vBulletin.org Site Feedback (https://vborg.vbsupport.ru/forumdisplay.php?f=7)
-   -   To enforce a valid coding style in the community! (https://vborg.vbsupport.ru/showthread.php?t=93400)

Boofo 08-02-2005 12:32 PM

Quote:

Originally Posted by KirbyDE
No they don't, and I never said they do.
Just wanted to point out that it is possible, although I guess nobody would ever even think of proofing their Code.
This is only being done for really important software like some parts or Aircraft Firmware.

Kirby, relax. I was just trying to get your goat since you got as late start in here. ;)

Read a few posts down. ;)

The Geek 08-02-2005 12:33 PM

The point is that only mod authors look at the code to modify.

jelsoft encourages people to modify and create macks - therefore the importance of a standard for them. Mod authors (free ones especially) have the right to not give a toss if others want to modify their modifications thereby making the coding style argument moot.

As for enforcing a code style so its easy to debug your own code... who cares as by the time it hits the proverbial shelf, it shouldnt need debugging anymore anyway.

Im not overly bothered to spend an extra day respacing, tabbing and tweaking my code so someone can give me a 'good boy' sticker. I would rather move on to something else that would benefit my users or the vb community.

If it works as intended then who cares where the freaking curley braces are?

akanevsky 08-02-2005 12:34 PM

Quote:

I was only kidding Kirby in that remark. Kirby does excellent work and his stuff always works.
I know :) Esspecially the HowTo's :)

Quote:

But to say that every hack/mod you do ALWAYS works the first time, is not only arrogant, but false. If it hasn't happened yet, it will, you can be sure.
I did not say that it ALWAYS works. Show me where I said "ALWAYS"... Thanks.

The Geek 08-02-2005 12:35 PM

ooo better idea!!!!

Have the 'QA' team simply reformat the code to the correct style!

That accomplishes everyones needs:
1- The QA people have macks that are standardized
2- The authors can move on to far more important issues like making more macks.

Whoot!

yoyoyoyo 08-02-2005 12:36 PM

Quote:

Originally Posted by Dark Visor
Come on, what kind of language is that... You are a moderator. Set a good example...

Quote:

Originally Posted by Dark Visor
Shut the f*ck up and stop trying to say that I said what I didn't say. Thanks.

Pot <--> kettle

akanevsky 08-02-2005 12:36 PM

Quote:

f it works as intended then who cares where the freaking curley braces are?
I do... When the "freaking curley braces" are where they shouldn't be, I cannot read the code and therefore I cannot be sure that you did not put any spyware or viruses into it.

Quote:

Pot <--> kettle
lol yea. If a moderator may, then so may I ...

Quote:

ooo better idea!!!!

Have the 'QA' team simply reformat the code to the correct style!

That accomplishes everyones needs:
1- The QA people have macks that are standardized
2- The authors can move on to far more important issues like making more macks.

Whoot!
That'd be a no, because QA does not have that much time..

Andreas 08-02-2005 12:37 PM

Quote:

Originally Posted by Dark Visor
The coders will. I'd like you to recall the coding style of vBulletin 2 and compare it with vBulletin 3. Which one is easier to modify? This:

PHP Code:

        if ($a) {
            while (
$b $c)
                {
            echo 
"!!!!!";    }
    }
                else { print 
"yo";
    } 

or this

PHP Code:

if ($a
 {
     while (
$b $c)
     {
         echo 
"!!!!!";
     }
 }
 else
 {
     print 
"yo";
 } 


Obviously the 2nd one, as it is easier to read.
However, as already said, you could just use a prettyprinter to transform Code #1.

calorie 08-02-2005 12:38 PM

Quote:

Originally Posted by KirbyDE
The 2nd one complies with vBulletin coding standards.

Yep, it does, but if error reporting is E_ALL only...

The point being that X's standards may not be Y's standards may not be Z's standards. If a hack doesn't pose a security risk, let style be as it may. If a hack does pose a security risk, see this.

yoyoyoyo 08-02-2005 12:38 PM

Quote:

Originally Posted by Dark Visor
I do... When the "freaking curley braces" are where they shouldn't be, I cannot read the code and therefore I cannot be sure that you did not put any spyware or viruses into it.

better notify symantec about the curley brace virus! :D

sabret00the 08-02-2005 12:38 PM

Quote:

Originally Posted by yoyoyoyo
Pot <--> kettle

echo'd thoughts.


All times are GMT. The time now is 05:33 AM.

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.01341 seconds
  • Memory Usage 1,754KB
  • Queries Executed 10 (?)
More Information
Template Usage:
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (2)bbcode_php_printable
  • (12)bbcode_quote_printable
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)pagenav
  • (1)pagenav_curpage
  • (4)pagenav_pagelink
  • (1)pagenav_pagelinkrel
  • (1)post_thanks_navbar_search
  • (1)printthread
  • (10)printthreadbit
  • (1)spacer_close
  • (1)spacer_open 

Phrase Groups Available:
  • global
  • postbit
  • showthread
Included Files:
  • ./printthread.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/class_bbcode_alt.php
  • ./includes/class_bbcode.php
  • ./includes/functions_bigthree.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • printthread_start
  • pagenav_page
  • pagenav_complete
  • bbcode_fetch_tags
  • bbcode_create
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • printthread_post
  • printthread_complete