Commit 1476c8a

committed

Add the time comparison guideline

1 parent 524a1ac commit 1476c8aCopy full SHA for 1476c8a

File tree

+17

-0

lines changed

+17

-0

lines changed

Lines changed: 17 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1808,6 +1808,23 @@ date.all_quarter`
`1808`	`1808`	`date.all_year`
`1809`	`1809`	`----`
`1810`	`1810`
	`1811`	`+=== Time Comparison`
	`1812`	`+`
	`1813`	+Use `past?` and `future?` methods instead of comparing time to the current time.
	`1814`	`+`
	`1815`	`+[source,ruby]`
	`1816`	`+----`
	`1817`	`+# bad`
	`1818`	`+grand_opening_at < Time.current`
	`1819`	`+public_release_at > Time.current`
	`1820`	`+`
	`1821`	`+# good`
	`1822`	`+grand_opening_at.past?`
	`1823`	`+public_release_at.future?`
	`1824`	`+----`
	`1825`	`+`
	`1826`	+NOTE: Rails 6.0 and later also provide `before?` and `after?` methods to compare two arbitrary time objects.
	`1827`	`+`
`1811`	`1828`	`== Duration`
`1812`	`1829`
`1813`	`1830`	`=== Duration Application`

Comments

(0)