Preparing report...

Report for github.com/BrianLusina/gopher-land

A    Great!    Found 56 issues across 135 files

Tweet

gofmt90%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command


go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!


gocyclo100%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

No problems detected. Good job!


golint68%

Golint is a linter for Go source code.

    • gopher-land/tour/interfaces/interface_example.go
    • Line 8: warning: exported type Entry should have comment or be unexported (golint)
    • Line 12: warning: exported type Book should have comment or be unexported (golint)
    • Line 18: warning: exported method Book.Title should have comment or be unexported (golint)
    • Line 22: warning: exported type Movie should have comment or be unexported (golint)
    • Line 28: warning: exported method Movie.Title should have comment or be unexported (golint)
    • Line 32: warning: exported function Display should have comment or be unexported (golint)
    • gopher-land/datastructures/linkedlist/doublylinkedlist/doublylinkedlist.go
    • Line 1: warning: don't use MixedCaps in package name; DoublyLinkedlist should be doublylinkedlist (golint)
    • Line 15: warning: comment on exported type DoublyLinkedList should be of the form "DoublyLinkedList ..." (with optional leading article) (golint)
    • Line 20: warning: exported method DoublyLinkedList.Print should have comment or be unexported (golint)
    • Line 24: warning: exported function CreateList should have comment or be unexported (golint)
    • Line 34: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 41: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 58: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 71: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 86: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 97: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 105: warning: comment on exported method DoublyLinkedList.IsPalindrome should be of the form "IsPalindrome ..." (golint)
    • Line 106: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 139: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 170: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 195: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 233: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 263: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 293: warning: comment on exported method DoublyLinkedList.SwapNodesAtKthAndKPlus1 should be of the form "SwapNodesAtKthAndKPlus1 ..." (golint)
    • Line 297: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 323: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • Line 351: warning: receiver name ll should be consistent with previous receiver name l for DoublyLinkedList (golint)
    • gopher-land/datastructures/linkedlist/singlylinkedlist/singlylinkedlist.go
    • Line 14: warning: comment on exported type SinglyLinkedList should be of the form "SinglyLinkedList ..." (with optional leading article) (golint)
    • Line 19: warning: exported method SinglyLinkedList.Print should have comment or be unexported (golint)
    • Line 23: warning: exported function CreateList should have comment or be unexported (golint)
    • Line 33: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 40: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 57: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 70: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 85: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 96: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 104: warning: comment on exported method SinglyLinkedList.IsPalindrome should be of the form "IsPalindrome ..." (golint)
    • Line 105: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 138: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 169: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 194: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 232: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 262: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 292: warning: comment on exported method SinglyLinkedList.SwapNodesAtKthAndKPlus1 should be of the form "SwapNodesAtKthAndKPlus1 ..." (golint)
    • Line 296: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 322: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • Line 350: warning: receiver name ll should be consistent with previous receiver name l for SinglyLinkedList (golint)
    • gopher-land/datastructures/trees/binarytrees/binary_tree.go
    • Line 15: warning: exported type BinaryTreeIterator should have comment or be unexported (golint)
    • Line 105: warning: receiver name root should be consistent with previous receiver name t for BinaryTreeNode (golint)
    • Line 134: warning: comment on exported method BinaryTreeNode.PostOrderTraversal should be of the form "PostOrderTraversal ..." (golint)
    • Line 194: warning: receiver name root should be consistent with previous receiver name t for BinaryTreeNode (golint)
    • Line 218: warning: receiver name root should be consistent with previous receiver name t for BinaryTreeNode (golint)
    • Line 273: warning: receiver name root should be consistent with previous receiver name t for BinaryTreeNode (golint)
    • Line 299: warning: receiver name root should be consistent with previous receiver name t for BinaryTreeNode (golint)
    • Line 335: warning: receiver name root should be consistent with previous receiver name t for BinaryTreeNode (golint)
    • gopher-land/tour/datatypes/organization/person.go
    • Line 13: warning: comment on exported type TwitterHandler should be of the form "TwitterHandler ..." (with optional leading article) (golint)
    • Line 16: warning: exported type Identifiable should have comment or be unexported (golint)
    • Line 26: warning: exported type Conflict should have comment or be unexported (golint)
    • Line 30: warning: exported type Name should have comment or be unexported (golint)
    • Line 35: warning: exported type Person should have comment or be unexported (golint)
    • Line 42: warning: exported function NewSocialSecurityNumber should have comment or be unexported (golint)
    • Line 87: warning: exported function NewPerson should have comment or be unexported (golint)
    • Line 97: warning: comment on exported method Name.FullName should be of the form "FullName ..." (golint)
    • Line 102: warning: exported method Person.TwitterHandler should have comment or be unexported (golint)
    • Line 106: warning: exported method Person.SetTwitterHandler should have comment or be unexported (golint)
    • Line 116: warning: exported method Person.ID should have comment or be unexported (golint)
    • Line 120: warning: exported method TwitterHandler.RedirectUrl should have comment or be unexported (golint)
    • gopher-land/algorithms/triangle/triangle.go
    • Line 5: warning: exported type Kind should have comment or be unexported (golint)
    • Line 8: warning: exported const Equ should have comment (or a comment on this block) or be unexported (golint)
    • Line 14: warning: exported function KindFromSides should have comment or be unexported (golint)
    • gopher-land/datastructures/lrucache/lrucache.go
    • Line 3: warning: exported type LRUCache should have comment or be unexported (golint)
    • Line 8: warning: exported function Constructor should have comment or be unexported (golint)
    • Line 12: warning: exported method LRUCache.Get should have comment or be unexported (golint)
    • Line 12: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 16: warning: exported method LRUCache.Put should have comment or be unexported (golint)
    • Line 16: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • gopher-land/gomath/fibonacci/fibonacci.go
    • Line 5: warning: comment on exported function FibRecursion should be of the form "FibRecursion ..." (golint)
    • Line 30: warning: comment on exported function FibMemo should be of the form "FibMemo ..." (golint)
    • Line 42: warning: comment on exported function FibGoldenRatio should be of the form "FibGoldenRatio ..." (golint)
    • gopher-land/algorithms/onlinestockspan/stockspanner.go
    • Line 3: warning: exported type StockSpanner should have comment or be unexported (golint)
    • Line 7: warning: exported function Constructor should have comment or be unexported (golint)
    • Line 11: warning: exported method StockSpanner.Next should have comment or be unexported (golint)
    • Line 11: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)

license0%

Checks whether your project has a LICENSE file.


ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


misspell97%

Misspell Finds commonly misspelled English words