.. _Coding:

.. raw:: html

   <script>ODSA.SETTINGS.DISP_MOD_COMP = true;ODSA.SETTINGS.MODULE_NAME = "Coding";ODSA.SETTINGS.MODULE_LONG_NAME = "Clean Code";ODSA.SETTINGS.MODULE_CHAPTER = "Week 3"; ODSA.SETTINGS.BUILD_DATE = "2016-11-29 20:59:56"; ODSA.SETTINGS.BUILD_CMAP = false;JSAV_OPTIONS['lang']='en';JSAV_EXERCISE_OPTIONS['code']='java';</script>


.. |--| unicode:: U+2013   .. en dash
.. |---| unicode:: U+2014  .. em dash, trimming surrounding whitespace
   :trim:


.. This file is part of the OpenDSA eTextbook project. See
.. http://algoviz.org/OpenDSA for more details.
.. Copyright (c) 2012-2013 by the OpenDSA Project Contributors, and
.. distributed under an MIT open source license.

.. avmetadata::
   :author: Cliff Shaffer


==========
Clean Code
==========

Example 1 (1)
~~~~~~~~~~~~~
   * A real bug from some code for an OpenDSA exercise.
     You don't need to know much about the language to recognize
     the bug (once you spot it)::

      <li><var><code>O(n^2)</code></var></li>
      <li><var><code>O(n \log n)</code></var></li>
      <li><var><code>O(\log n)</code><var></li>
      <li><var><code>O(1)</code></var></li>


Example 1 (2)
~~~~~~~~~~~~~
   * A real bug from some code for an OpenDSA exercise.
     You don't need to know much about the language to recognize
     the bug (once you spot it)::

      <li><var><code>O(n^2)</code></var></li>
      <li><var><code>O(n \log n)</code></var></li>
      <li><var><code>O(\log n)</code><var></li>
      <li><var><code>O(1)</code></var></li>

   * When refactored for readability, the bug pops out immediately::

      <li><var><code>O(n^2)     </code></var></li>
      <li><var><code>O(n \log n)</code></var></li>
      <li><var><code>O(\log n)  </code><var></li>
      <li><var><code>O(1)       </code></var></li>


Example 2 (1)
~~~~~~~~~~~~~
   * A real sample of code::

      var nodeOne = graphTwo.addNode("a ", {left: 0, top: 0});
      var nodeTwo = graphTwo.addNode("c", {left: 100, top: 0});
      var nodeThree = graphTwo.addNode("e", {left: 50, top: 50});
      var nodeFour = graphTwo.addNode("b", {left: 0,  top: 100});
      var nodeFive =graphTwo.addNode("d ", {left: 100, top: 100});


Example 2 (2)
~~~~~~~~~~~~~
   * A real sample of code::

      var nodeOne = graphTwo.addNode("a ", {left: 0, top: 0});
      var nodeTwo = graphTwo.addNode("c", {left: 100, top: 0});
      var nodeThree = graphTwo.addNode("e", {left: 50, top: 50});
      var nodeFour = graphTwo.addNode("b", {left: 0,  top: 100});
      var nodeFive =graphTwo.addNode("d ", {left: 100, top: 100});

   * Refactored for readability::

      var nodea = graph2.addNode("a", {left:   0, top:   0});
      var nodeb = graph2.addNode("b", {left:   0, top: 100});
      var nodec = graph2.addNode("c", {left: 100, top:   0});
      var noded = graph2.addNode("d", {left: 100, top: 100});
      var nodee = graph2.addNode("e", {left:  50, top:  50});