Optimize counter inc-by-one

spectator/meter/counter.go

@@ -13,14 +13,14 @@ import (
 //
 // https://netflix.github.io/spectator/en/latest/intro/counter/
 type Counter struct {
-	id              *Id
-	writer          writer.Writer
-	meterTypeSymbol string
+	id           *Id
+	writer       writer.Writer
+	incByOneLine string
 }
 
 // NewCounter generates a new counter, using the provided meter identifier.
 func NewCounter(id *Id, writer writer.Writer) *Counter {
-	return &Counter{id, writer, "c"}
+	return &Counter{id, writer, fmt.Sprintf("c:%s:1", id.spectatordId)}
 }
 
 // MeterId returns the meter identifier.
@@ -30,22 +30,25 @@ func (c *Counter) MeterId() *Id {
 
 // Increment increments the counter.
 func (c *Counter) Increment() {
-	var line = fmt.Sprintf("%s:%s:%d", c.meterTypeSymbol, c.id.spectatordId, 1)
-	c.writer.Write(line)
+	c.writer.Write(c.incByOneLine)
 }
 
 // Add adds an int64 delta to the current measurement.
 func (c *Counter) Add(delta int64) {
+	if delta == 1 {
+		c.writer.Write(c.incByOneLine)
+		return
+	}
 	if delta > 0 {
-		var line = fmt.Sprintf("%s:%s:%d", c.meterTypeSymbol, c.id.spectatordId, delta)
+		var line = fmt.Sprintf("c:%s:%d", c.id.spectatordId, delta)
 		c.writer.Write(line)
 	}
 }
 
 // AddFloat adds a float64 delta to the current measurement.
 func (c *Counter) AddFloat(delta float64) {
 	if delta > 0.0 {
-		var line = fmt.Sprintf("%s:%s:%f", c.meterTypeSymbol, c.id.spectatordId, delta)
+		var line = fmt.Sprintf("c:%s:%f", c.id.spectatordId, delta)
 		c.writer.Write(line)
 	}
 }

spectator/meter/counter_test.go

@@ -23,15 +23,21 @@ func TestCounter_Add(t *testing.T) {
 	id := NewId("add", nil)
 	c := NewCounter(id, &w)
 
-	c.Add(4)
+	c.Add(1)
 
-	expected := "c:add:4"
+	expected := "c:add:1"
 	if w.Lines()[0] != expected {
 		t.Error("Expected ", expected, " got ", w.Lines()[0])
 	}
+	c.Add(4)
+
+	expected = "c:add:4"
+	if w.Lines()[1] != expected {
+		t.Error("Expected ", expected, " got ", w.Lines()[1])
+	}
 
 	c.Add(-1)
-	if len(w.Lines()) != 1 {
+	if len(w.Lines()) != 2 {
 		t.Error("Negative deltas should be ignored")
 	}
 }