Coding Standards - Revision history

Bobbanz: /* Specification */

2023-03-31T05:58:23Z

Specification

Covertcorvid: Added requirements for BeeStation compatibility

2023-03-30T05:48:08Z

Added requirements for BeeStation compatibility

imported>LiamLime: /* Specification */

2015-10-04T04:45:32Z

Specification

imported>MisterPerson: Added the colons are ok exception for expensive stuff like the lighting subsystem

2015-01-15T16:39:46Z

Added the colons are ok exception for expensive stuff like the lighting subsystem

imported>MisterPerson: Removed absolute pathing requirement

2015-01-10T15:18:53Z

Removed absolute pathing requirement

imported>Rockdtben: /* Specification */

2014-06-03T13:57:44Z

Specification

imported>Rockdtben at 13:52, 3 June 2014

2014-06-03T13:52:17Z

imported>Rockdtben at 13:52, 3 June 2014

2014-06-03T13:52:06Z

imported>Rockdtben at 13:34, 3 June 2014

2014-06-03T13:34:52Z

imported>Rockdtben: Created page with "=Specification= As mentioned before, you are expected to follow these specifications in order to make everyone's lives easier, it will also save you and us time, with having ..."

2014-06-03T13:27:54Z

Created page with "=Specification= As mentioned before, you are expected to follow these specifications in order to make everyone's lives easier, it will also save you and us time, with having ..."

New page

=Specification=

As mentioned before, you are expected to follow these specifications in order to make everyone's lives easier, it will also save you and us time, with having to make the changes and us having to tell you what to change. Thank you for reading this section.

* As BYOND's Dream Maker is an object oriented language, code must be object oriented when possible in order to be more flexible when adding content to it. If you are unfamiliar with this concept, it is highly recommended you look it up.

* You must write BYOND code with absolute pathing, like so:

<pre>

/obj/item/weapon/baseball_bat
name = "baseball bat"
desc = "A baseball bat."
var/wooden = 1

/obj/item/weapon/baseball_bat/examine()
if(wooden)
desc = "A wooden baseball bat."
else
desc = "A metal baseball bat."
..()

</pre>

* You must not use colons to override safety checks on an object's variable/function, instead of using proper type casting.

* It is rarely allowed to put type paths in a text format, as there are no compile errors if the type path no longer exists. Here is an example:

<pre>
//Good
var/path_type = /obj/item/weapon/baseball_bat

//Bad
var/path_type = "/obj/item/weapon/baseball_bat"
</pre>

* You must use tabs to indent your code, NOT SPACES.

* Hacky code, such as adding specific checks, is highly discouraged and only allowed when there is no other option. You can avoid hacky code by using object oriented methodologies, such as overriding a function (called procs in DM) or sectioning code into functions and then overriding them as required.

* Duplicated code is 99% of the time never allowed. Copying code from one place to another maybe suitable for small short time projects but /tg/station focuses on the long term and thus discourages this. Instead you can use object orientation, or simply placing repeated code in a function, to obey this specification easily.

* Code should be modular where possible, if you are working on a new class then it is best if you put it in a new file.

* Bloated code may be necessary to add a certain feature, which means there has to be a judgement over whether the feature is worth having or not. You can help make this decision easier by making sure your code is modular.

* You are expected to help maintain the code that you add, meaning if there is a problem then you are likely to be approached in order to fix any issues, runtimes or bugs.

@@ Line 31: / Line 31: @@
 if(isovermap(A)) // NSV13 - special overmap handling
      bar()
 </pre>

@@ Line 29: / Line 29: @@
 * You are expected to help maintain the code that you add, meaning if there is a problem then you are likely to be approached in order to fix any issues, runtimes or bugs.
-* Follow the stylesheet. Changing formatting with \red or other tags is not allowed. You must use span classes, which are defined in interface/stylesheet.dm.
+* Follow the stylesheet. Changing formatting with \red or other tags is not allowed. You must use [[Text Formatting|span classes]], which are defined in interface/stylesheet.dm.
 [[Category:Game Resources]] [[Category:Guides]]
 {{Contribution guides}}

← Older revision		Revision as of 15:18, 10 January 2015
Line 4:		Line 4:

	* As BYOND's Dream Maker is an object oriented language, code must be object oriented when possible in order to be more flexible when adding content to it. If you are unfamiliar with this concept, it is highly recommended you look it up.		* As BYOND's Dream Maker is an object oriented language, code must be object oriented when possible in order to be more flexible when adding content to it. If you are unfamiliar with this concept, it is highly recommended you look it up.
−
−	* You must write BYOND code with absolute pathing, like so:
−
−	~~<pre>~~
−
−	~~/obj/item/weapon/baseball_bat~~
−	~~name = "baseball bat"~~
−	~~desc = "A baseball bat."~~
−	~~var/wooden = 1~~
−
−	~~/obj/item/weapon/baseball_bat/examine()~~
−	~~if(wooden)~~
−	~~desc = "A wooden baseball bat."~~
−	~~else~~
−	~~desc = "A metal baseball bat."~~
−	~~..()~~
−
−	~~</pre>~~

	* You must not use colons to override safety checks on an object's variable/function, instead of using proper type casting.		* You must not use colons to override safety checks on an object's variable/function, instead of using proper type casting.

@@ Line 48: / Line 48: @@
 * Follow the stylesheet. Changing formatting with \red or other tags is not allowed. You must use span classes, which are defined in interface/stylesheet.dm.
 [[Category:Game Resources]] [[Category:Guides]]
 {{Contribution guides}}

← Older revision		Revision as of 13:52, 3 June 2014
Line 48:		Line 48:

	* Follow the stylesheet. Changing formatting with \red or other tags is not allowed. You must use span classes, which are defined in interface/stylesheet.dm.		* Follow the stylesheet. Changing formatting with \red or other tags is not allowed. You must use span classes, which are defined in interface/stylesheet.dm.
		+
		+	[[Category:Game Resources]] [[Category:Guides]]
		+	{{Contribution guides}}

@@ Line 21: / Line 21: @@
 * Hacky code, such as adding specific checks, is highly discouraged and only allowed when there is no other option. You can avoid hacky code by using object oriented methodologies, such as overriding a function (called procs in DM) or sectioning code into functions and then overriding them as required.
-* Duplicated code is 99% of the time never allowed. Copying code from one place to another maybe suitable for small short time projects but /tg/station focuses on the long term and thus discourages this. Instead you can use object orientation, or simply placing repeated code in a function, to obey this specification easily.
+* Duplicated code is 99% of the time never allowed. Copying code from one place to another maybe suitable for small short time projects but NSV13 focuses on the long term and thus discourages this. Instead you can use object orientation, or simply placing repeated code in a function, to obey this specification easily.
 * Code should be modular where possible, if you are working on a new class then it is best if you put it in a new file.
 * Bloated code may be necessary to add a certain feature, which means there has to be a judgement over whether the feature is worth having or not. You can help make this decision easier by making sure your code is modular.

@@ Line 5: / Line 5: @@
 * As BYOND's Dream Maker is an object oriented language, code must be object oriented when possible in order to be more flexible when adding content to it. If you are unfamiliar with this concept, it is highly recommended you look it up.
-* You must not use colons to override safety checks on an object's variable/function, instead of using proper type casting.
+* You must not use colons to override safety checks on an object's variable/function, instead of using proper type casting, unless the code in question is a known resource hog.
 * It is rarely allowed to put type paths in a text format, as there are no compile errors if the type path no longer exists. Here is an example:

@@ Line 1: / Line 1: @@
 =Specification=
-As mentioned before, you are expected to follow these specifications in order to make everyone's lives easier, it will also save you and us time, with having to make the changes and us having to tell you what to change. Thank you for reading this section.
+You are expected to follow these specifications in order to make everyone's lives easier, it will also save you and us time, with having to make the changes and us having to tell you what to change. Thank you for reading this section.
 * As BYOND's Dream Maker is an object oriented language, code must be object oriented when possible in order to be more flexible when adding content to it. If you are unfamiliar with this concept, it is highly recommended you look it up.

← Older revision		Revision as of 13:34, 3 June 2014
Line 46:		Line 46:

	* You are expected to help maintain the code that you add, meaning if there is a problem then you are likely to be approached in order to fix any issues, runtimes or bugs.		* You are expected to help maintain the code that you add, meaning if there is a problem then you are likely to be approached in order to fix any issues, runtimes or bugs.
		+
		+	* Follow the stylesheet. Changing formatting with \red or other tags is not allowed. You must use span classes, which are defined in interface/stylesheet.dm.