commit d00d3aa498a11db7fc62a538e0e60a5d61891986 parent 5fe8726117c0d605d46f9520138c66cc773923b0 Author: Jack Mordaunt <jackmordaunt@gmail.com> Date: Sun, 8 Nov 2020 15:40:01 +0800 refactor: make break condition clear Diffstat:
| M | cmd/kanban/main.go | | | 2 | +- |
| M | kanban.go | | | 4 | ++++ |
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/cmd/kanban/main.go b/cmd/kanban/main.go @@ -400,7 +400,7 @@ func (ui *UI) Refocus(d Direction) { ui.FocusedTicket.Stage = len(stages) - 1 } } - if len(stages[ui.FocusedTicket.Stage].Tickets) > 0 { + if stage := stages[ui.FocusedTicket.Stage]; !stage.Empty() { break } } diff --git a/kanban.go b/kanban.go @@ -19,6 +19,10 @@ type Stage struct { Tickets []Ticket } +func (s Stage) Empty() bool { + return len(s.Tickets) == 0 +} + // Ticket in a stage. type Ticket struct { ID int