fixed solution035 and removed git hook

Author: TipsyPixie

Makefile

@@ -21,7 +21,4 @@ fix: FORCE
 report: FORCE
 	curl -s -d 'repo=github.com%2FTipsyPixie%2Fgo-daily-coding-problem' https://goreportcard.com/checks >/dev/null
 
-hook: FORCE
-	find 'hooks' -maxdepth 1 -type f -exec ln -s '../../{}' -t './.git/hooks' \;
-
 precommit: fix analyze format test

README.md

@@ -12,8 +12,6 @@ Run `make test` to test the solutions.
 make test # test all
 ```
 
-`make hook` to install git hooks in `hooks` directory.
-
 ## Problems
 
 Number | Difficulty | Asked By

hooks/pre-commit

@@ -2,7 +2,7 @@ package problem035
 
 func partialSegregate(letter rune, in []rune) int {
 	leftIndex, rightIndex := 0, len(in)
-	for leftIndex < rightIndex {
+	for leftIndex <= rightIndex {
 		leftLetter := in[leftIndex]
 		if leftLetter == letter {
 			leftIndex++
@@ -14,12 +14,12 @@ func partialSegregate(letter rune, in []rune) int {
 			in[leftIndex], in[rightIndex] = rightLetter, leftLetter
 		}
 	}
-	return leftIndex + 1
+	return leftIndex
 }
 
 func Segregate(letters []rune) {
 	letterTypes := []rune{'R', 'G', 'B'}
-	for typeIndex, startingIndex := 0, 0; typeIndex < len(letterTypes) && startingIndex < len(letters); typeIndex++ {
-		startingIndex = partialSegregate(letterTypes[typeIndex], letters[startingIndex:])
+	for typeIndex, startingIndex := 0, 0; typeIndex < len(letterTypes)-1 && startingIndex < len(letters); typeIndex++ {
+		startingIndex += partialSegregate(letterTypes[typeIndex], letters[startingIndex:])
 	}
 }

problem035/problem035.go

@@ -8,7 +8,7 @@ import (
 func TestSegregate(t *testing.T) {
 	input := []rune{'G', 'B', 'R', 'R', 'B', 'R', 'G', 'R', 'R', 'B', 'G', 'R', 'B'}
 	Segregate(input)
-	if reflect.DeepEqual(input, []rune{'R', 'R', 'R', 'R', 'R', 'R', 'G', 'G', 'G', 'B', 'B', 'B'}) {
+	if !reflect.DeepEqual(input, []rune{'R', 'R', 'R', 'R', 'R', 'R', 'G', 'G', 'G', 'B', 'B', 'B', 'B'}) {
 		t.Fail()
 	}
 }