Move any output-related logic to printers (and vice-versa)

[pouriyajamshidi]

Move any output-related logic to printers (and vice-versa)

[ravsii]

Also, it's the same the other way around, as I recently found this while investigating #83 (comment)

tcping/statsprinter.go

Lines 201 to 205 in 2839563

	func (p *statsPlanePrinter) printTotalDownTime(now time.Time) {
	latestDowntimeDuration := time.Since(p.startOfDowntime)
	calculatedDowntime := calcTime(latestDowntimeDuration)
	colorYellow("No response received for %s\n", calculatedDowntime)
	}

The printer calculates the downtime the way it wants, while, ideally, it should be something close to this (pseudocode):

func (p *statsPlanePrinter) printTotalDownTime() {
	colorYellow("no response received for %s\n", p.downtimeDuration);
}

The JSON printer is guilty of that as well.

[pouriyajamshidi]

I agree. it also helps making the code more readable.