voroskoi/aoc2021 - Change HZ7PK7IJOAVEZNPPQX5DENZIA6K25SQMHJUQEDBNJ7EPZO3TP6IQC

Refactor 04/1: use isMarked() and gridSize

Created by voroskoi on December 4, 2021

HZ7PK7IJOAVEZNPPQX5DENZIA6K25SQMHJUQEDBNJ7EPZO3TP6IQC

Dependencies

[2] SZZRPGSAGNADFKQWMLQEAXW4HRN4EJI27V2VPQN5J6BVZEDMC6EQC

In channels

main

Change contents

Insertion in 04/main.go at line 11 [2.47]
[2.127]
[2.127]
```
const gridSize = 5
```
Replacement in 04/main.go at line 14 [2.47]
B:BD[2.128] → [2.128:148]
```
type Grid [5][5]int
```
[2.128]
[2.148]
```
type Grid [gridSize][gridSize]int
```
Replacement in 04/main.go at line 49 [2.47]
B:BD[2.606] → [2.606:622]
```
		if mark < 5 {
```
[2.606]
[2.622]
```
		if mark < gridSize {
```

Replacement in 04/main.go at line 67 [2.47]

B:BD[2.1085] → [2.1085:1246]

			var found bool
			for m := 0; m <= mark; m++ {
				if b.marks[m] == elem {
					// item found in marks
					found = true
					break
				}
			}
			if !found {

[2.1085]

[2.1246]

			if !b.isMarked(elem, mark) {

Replacement in 04/main.go at line 77 [2.47]

B:BD[2.1398] → [2.1398:1523]

		var found bool
		for i := 0; i <= mark; i++ {
			if b.marks[i] == elem {
				found = true
				break
			}
		}
		if !found {

[2.1398]

[2.1523]

		if !b.isMarked(elem, mark) {

Replacement in 04/main.go at line 86 [2.47]
B:BD[2.1692] → [2.1692:1724]
```
	for row := 0; row < 5; row++ {
```
[2.1692]
[2.1724]
```
	for row := 0; row < gridSize; row++ {
```

Replacement in 04/main.go at line 88 [2.47]

B:BD[2.1761] → [2.1761:1886]

		var found bool
		for i := 0; i <= mark; i++ {
			if b.marks[i] == elem {
				found = true
				break
			}
		}
		if !found {

[2.1761]

[2.1886]

		if !b.isMarked(elem, mark) {

Replacement in 04/main.go at line 92 [2.47]

B:BD[2.1909] → [2.1909:1981]

	log.Println("checkCol:", b.marks[:mark+1], b.grids[gridnum][0:5][col])

[2.1909]

[2.1981]

	log.Println("checkCol:", b.marks[:mark+1], b.grids[gridnum][0:gridSize][col])

Insertion in 04/main.go at line 96 [2.47]

[2.1997]

func (b Bingo) isMarked(elem int, maxMark int) bool {
	for i := 0; i <= maxMark; i++ {
		if b.marks[i] == elem {
			return true
		}
	}
	return false
}

Refactor 04/1: use isMarked() and gridSize

Dependencies

In channels

Change contents

Insertion in 04/main.go at line 11 [2.47]

Replacement in 04/main.go at line 14 [2.47]

Replacement in 04/main.go at line 49 [2.47]

Replacement in 04/main.go at line 67 [2.47]

Replacement in 04/main.go at line 77 [2.47]

Replacement in 04/main.go at line 86 [2.47]

Replacement in 04/main.go at line 88 [2.47]

Replacement in 04/main.go at line 92 [2.47]

Insertion in 04/main.go at line 96 [2.47]