David Sommers databyte

Dear Rubyists,

I just lost a contract because of my code in a Rails project.

The specific code in question is related to a "posting a comment" feature. Here are the details:

In this project, "posting a comment" does not simply entail inserting a row into the database. It involves a procedure to yes, insert a row, but also detect its language, check for spam, send emails, and "share" it to Twitter and Facebook. I believe this algorithm should be encapsulated. I do not believe it belongs in a controller or a model. I do not believe Active Record callbacks should be used.

The "senior developer", whom is the stake holder's right hand man, said this:

	@ChristinGorman gave this talk at JavaZone: https://vimeo.com/49484333 It's quite good, short, energetic, enthusiastic,
	intelligent, and completely misses the point.

	While it's true that the code she produces is much better than the original, and is quite easy to understand; it fails one
	critical test. It's not polite.

	Polite code is like a well written newspaper article. It allows you to bail out early. A well written article has a
	headline, a synopsis, and a set of paragraphs that begin with the high level concepts and get more and more detailed as you
	read through the article. At any point you can decide: "I get it! I don't need to read further." Indeed, this is how most
	people read newspapers or magazines. The articles are polite, because they allow you to get out quickly.

	$stack, $draws = [], {}

	def method_missing *args
	return if args[0][/^to_/]
	$stack << args.map { \|a\| a or $stack.pop }
	$draws[$stack.pop(2)[0][0]] = args[1] if args[0] == :<
	end

	class Array
	def +@

	# I love the helpers, they help to make the intention clear.
	# I want to be able to do this. That is, using the validation helpers inside a validator object,
	# rather than having to do manual validation.

	class EmployeeValidator < ActiveModel::Validator
	def validate(record)
	record.validates_presence_of :email
	end
	end

	;; based on core.logic 0.8-alpha2 or core.logic master branch

	(ns sudoku
	(:refer-clojure :exclude [==])
	(:use clojure.core.logic))

	(defn get-square [rows x y]
	(for [x (range x (+ x 3))
	y (range y (+ y 3))]
	(get-in rows [x y])))

	require 'fiddle'
	require 'minitest/autorun'

	class RubyVM
	class InstructionSequence
	address = Fiddle::Handle::DEFAULT['rb_iseq_load']
	func = Fiddle::Function.new(address, [Fiddle::TYPE_VOIDP] * 3, Fiddle::TYPE_VOIDP)

	define_singleton_method(:load) do \|data, parent = nil, opt = nil\|
	func.call(Fiddle.dlwrap(data), parent, opt).to_value

	# Previous one is unsafe
	you should take a look at this: https://github.com/raggi/openssl-osx-ca

	Pry.config.editor = "mvim"

	# Load plugins (only those I whitelist)
	Pry.config.should_load_plugins = false
	Pry.plugins["doc"].activate!

	# Launch Pry with access to the entire Rails stack.
	# If you have Pry in your Gemfile, you can pass: ./script/console --irb=pry instead.
	# If you don't, you can load it through the lines below :)
	rails = File.join Dir.getwd, 'config', 'environment.rb'

	Latency Comparison Numbers (~2012)
	----------------------------------
	L1 cache reference 0.5 ns
	Branch mispredict 5 ns
	L2 cache reference 7 ns 14x L1 cache
	Mutex lock/unlock 25 ns
	Main memory reference 100 ns 20x L2 cache, 200x L1 cache
	Compress 1K bytes with Zippy 3,000 ns 3 us
	Send 1K bytes over 1 Gbps network 10,000 ns 10 us
	Read 4K randomly from SSD* 150,000 ns 150 us ~1GB/sec SSD