Wednesday, 20 December 2017

Book Review: The Elements of Scrum, by Chris Sims and Hillary Louise Johnson

I got this book after reading the abridged version by the same authors. Like the abridged version this book is well written and the concepts well explained. The book opens with a good comparison of "waterfall" vs "agile" development. Following this is a very enjoyable exposition of the agile values and principles (individuals and interactions over processes and tools etc etc). Following this is the core part of the book: explanations of the roles (product owner, scrum master, team member), ceremonies (sprint planning, daily standup, sprint review, sprint retrospective, backlog grooming, story time) and artifacts (product backlog, sprint backlog, burn charts, task board, definition of done) that make up scrum. Within the core part of the book is a good explanation of what user stories are and how they help with understanding what needs to be built, for who and, importantly, why. And closing out the core part of the book is a discussion on some tried and tested strategies for estimating stories.

I've not read any other books on scrum so I can't compare but I'd be very very amazed to find a better one out there than this one. This book covered absolutely lots of ground whilst remaining compact and, most importantly, very readable.

Below is a small selection of quotes from the book:
"...only by demonstrating working software to your customer can you find out what they really want... agile processes include constant feedback from customers, projects stay relevant and on track..."
"...The main idea behind the agile approach is to deliver business value immediately, in the form of working software, and to continue delivering value in regular increments..."
"...Document as you go, and only as needed..."
"...It’s only human nature to become attached to something you worked hard to produce; if you spend the same amount of time drafting your documentation as Tolstoy spent writing War and Peace, you may find yourself treating your masterpiece as if it were the work product itself..."
"...In order to be successful, a contributor-scrum master must put the needs and success of the team above the desire to directly contribute..."
"...Being a scrum team member isn’t about getting your job done, it’s about getting the job done..."
"...Velocity should never be used as a performance metric: it’s not about demonstrating to management that you’re working fast enough; it’s about gaining predictability of schedule..."

Friday, 29 September 2017

Book Review: Scrum: A Breathtakingly Brief and Agile Introduction, by Chris Sims and Hillary Louise Johnson

Good, concise, to-the-point introduction and summary of the Scrum way of working for teams and companies. The book covers the Roles (Product Owner, Scrum Master, Team Member), the Artifacts (Product Backlog, Sprint Backlog, Burn Charts etc) and the Sprint Cycle elements (Daily Scrum, Story Time, Sprint Review etc) that make up a Scrum workflow.

Book Summary: Clean Code, by Robert C Martin

An absolute must-read for all software engineers and programmers. Especially so if you work with Java or any such programming language. The first half of the book explains the concepts that make up clean code (naming, formatting, testing etc) and lays down the foundation. The second half of the book presents some examples of code which the author works through and refactors to get it to a cleaner state. It all comes together nicely and I've found it to be really impactful in influencing how I think about code.

I've included below a summary of the core messages that I took away from the book.

Foreward

  • Honesty in small things is not a small thing.
  • Cleanliness is next to godliness.
  • Leave the campground cleaner than you found it.

Introduction

  • LeBlanc's law: Later equals never.
  • The fault is not in our stars but in ourselves.
  • A building with broken windows looks like nobody cares about it.
  • One broken window starts the process toward decay.
  • Bad code tries to do too much, it has muddled intent and ambiguity of purpose. Clean code is focused. Each function/class/module exposes a single-minded attitude that remains entirely undistracted, and polluted, by the surrounding details.
  • Readers of our code should perceive us to have been decisive.
  • There is a difference between code that is easy to read and code that is easy to change.
  • When the same thing is done over and over, it's a sign that there is an idea in our mind that is not well represented in the code.
  • Can you imagine working on a project where the code simply got better as time passed? Do you believe that any other option is professional?

Naming

  • You should name a variable using the same care with which you name a first-born child.
  • Choosing good names takes time but saves more than it takes.
  • Don't be afraid to spend time choosing a name.
  • Hunting for a good name often results in a favourable restructuring of the code.
  • If we cannot derive a concise name for a class or function, then it's likely too large. The more ambiguous the class or function name, the more likely it has too many responsibilities.
  • Take care with your names and change them when you find better ones.
  • One difference between a smart programmer and a professional programmer is that the professional understands that clarity is king.

Functions

  • The first rule of functions is that they should be small. The second rule of functions is that they should be smaller than that.
  • Functions should do one thing. They should do it well. They should do it only.
  • You know you are working on clean code when each routine turns out to be pretty much what you expected.
  • The ideal number of arguments for a function is zero. Next comes one, followed closely by two. Three arguments should be avoided where possible.
  • Passing a boolean into a function is a terrible practice. It proclaims that the function does more than one thing: one thing if the flag is true and another if the flag is false.
  • Output arguments should be avoided. If your function must change the state of something, have it change the state of its owning object.

Comments

  • Don't comment bad code – rewrite it.
  • The proper use of comments is to compensate for our failure to express ourself in code.
  • Inaccurate comments are far worse than no comments at all. They delude and mislead.

Formatting

  • When people look under the hood, we want them to be impressed with the neatness, consistency, and attention to detail that they perceive. We want them to be struck by the orderliness. We want them to perceive that professionals have been at work.
  • We want the software to have a consistent style. We don't want it to appear to have been written by a bunch of disagreeing individuals.

Objects

  • There is a reason that we keep our variables private [and not exposed through getters and setters]. We don't want anyone else to depend on them. We want to keep the freedom to change their type or implementation on a whim or an impulse.
  • Law of Demeter: a module should not know about the innards of the objects it manipulates.

Error handling

  • Error handling should be complete. This goes to the discipline of paying attention to details.

Tests

  • Code, without tests, is not clean. No matter how elegant or readable it is.
  • Don't rely on your intuition. Look for every boundary condition and write a test for it.
  • Without a test suite we cannot ensure that changes in one part of our system do not break other parts of the system.
  • If you have tests, you do not fear making changes to the code. Without tests every change is a possible bug.
  • Tests enable change.
  • Having dirty tests is equivalent to, if not worse than, having no tests.
  • The dirtier the tests, the harder they are to change.
  • Test code is just as important as production code. It is not a second-class citizen. It requires thought, design, and care. It must be kept as clean as production code.
  • There are things that you might never do in a production environment that are perfectly fine in a test environment. Usually they involve issues of memory or CPU efficiency. But they never involve issues of cleanliness.
  • The number of asserts in a test ought to be minimised.
  • Minimise the number of asserts per concept and test just one concept per test function.
  • Tests should be F.I.R.S.T: Fast, Independent, Repeatable, Self-Validating, Timely.
  • Testing does not guarantee correctness. However, good testing can minimise risk.

Classes

  • The first rule of classes is that they should be small. The second rule of classes is that they should be smaller than that. 
  • Smaller is the primary rule when it comes to designing classes.
  • We should be able to write a brief description of the class in about 25 words, without using the words "if", "and", "or, or "but".
  • A class or module should have one, and only one, reason to change.
  • Classes should have one responsibility – one reason to change.
  • When classes lose cohesion, split them.

Emergence

  • A system that is comprehensively tested and passes all of its tests all of the time is a testable system.
  • Systems that aren't testable aren't verifiable. A system that cannot be verified should never be deployed.
  • Making our systems testable pushes us toward a design where our classes are small and single purpose.
  • Tight coupling makes it difficult to write tests.

Saturday, 29 April 2017

IntelliJ: Live Template for generating an empty JUnit test method

Here's an IntelliJ / Android Studio Live Template for generating an empty JUnit test method:

@org.junit.Test
public void $testMethodName$() {
    // Given.
    
    // When.
    
    // Then.
}

Set the "Applicability" of the Live Template to be "Java declarations" and set the "Expression" associated to the $testMethodName$ variable to be methodName().

Set the "Abbreviation" and "Description" for the Live Template as you wish and that's it... you're good to go!