x/clean-go-article
1
0
Fork 0
mirror of https://github.com/Pungyeon/clean-go-article.git synced 2024-09-19 22:34:04 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

update readme, fix formatting

Browse source
This commit is contained in:
Avuluri Venkata Sai Reddy 2023-01-29 23:18:21 +05:30 committed by GitHub
parent 3011efee29
commit 7e2ba16f20
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
README.md
View file

@ -814,27 +814,27 @@ There are other scenarios in which it is common to find `nil` values that can ca
```go ```go
type App struct { type App struct {
Cache *KVCache Cache *KVCache
} }
type KVCache struct { type KVCache struct {
mtx sync.RWMutex mtx sync.RWMutex
store map[string]string store map[string]string
} }
func (cache *KVCache) Add(key, value string) { func (cache *KVCache) Add(key, value string) {
cache.mtx.Lock() cache.mtx.Lock()
defer cache.mtx.Unlock() defer cache.mtx.Unlock()
cache.store[key] = value cache.store[key] = value
} }
``` ```
This code is absolutely fine. However, the danger is that our `App` can be initialised incorrectly, without initialising the `Cache` property within. Should the following code be invoked, our application will panic: This code is absolutely fine. However, the danger is that our `App` can be initialised incorrectly, without initialising the `Cache` property within. Should the following code be invoked, our application will panic:
```go ```go
app := App{} app := App{}
app.Cache.Add("panic", "now") app.Cache.Add("panic", "now")
``` ```
The `Cache` property has never been initialised and is therefore a `nil` pointer. Thus, invoking the `Add` method like we did here will cause a panic, with the following message: The `Cache` property has never been initialised and is therefore a `nil` pointer. Thus, invoking the `Add` method like we did here will cause a panic, with the following message:
@ -845,14 +845,14 @@ Instead, we can turn the `Cache` property of our `App` structure into a private
```go ```go
type App struct { type App struct {
cache *KVCache cache *KVCache
} }
func (app *App) Cache() *KVCache { func (app *App) Cache() *KVCache {
if app.cache == nil { if app.cache == nil {
app.cache = NewKVCache() app.cache = NewKVCache()
} }
return app.cache return app.cache
} }
``` ```