Commitffb5fa3

committed

Add a section on naming methods

1 parent71a235b commitffb5fa3Copy full SHA for ffb5fa3

File tree

+14

-26

lines changed

+14

-26

lines changed

Lines changed: 14 additions & 26 deletions

Original file line number	Diff line number	Diff line change
`@@ -373,38 +373,26 @@ def email_client(clients: Iterator[Client]) -> None:`
`373`	`373`
`374`	`374`	`Bad:`
`375`	`375`
`376`		-```php
`377`		`-class Email`
`378`		`-{`
`379`		`- //...`
`380`		`-`
`381`		`- public function handle()`
`382`		`- {`
`383`		`- mail($this->to, $this->subject, $this->body);`
`384`		`- }`
`385`		`-}`
	`376`	+```python
	`377`	`+classEmail:`
	`378`	`+defhandle(self) ->None:`
	`379`	`+# Do something...`
`386`	`380`
`387`		`-$message =newEmail(...);`
`388`		`-// What is this? A handle for the message? Are we writingtoa file now?`
`389`		`-$message->handle();`
	`381`	`+message= Email()`
	`382`	`+# What is this supposedtodo again?`
	`383`	`+message.handle()`
`390`	`384`	```
`391`	`385`
`392`	`386`	`Good:`
`393`	`387`
`394`		-```php
`395`		`-class Email`
`396`		`-{`
`397`		`- //...`
`398`		`-`
`399`		`- public function send()`
`400`		`- {`
`401`		`- mail($this->to, $this->subject, $this->body);`
`402`		`- }`
`403`		`-}`
	`388`	+```python
	`389`	`+classEmail:`
	`390`	`+defsend(self) ->None:`
	`391`	`+"""Send this message.`
	`392`	`+"""`
`404`	`393`
`405`		`-$message = new Email(...);`
`406`		`-// Clear and obvious`
`407`		`-$message->send();`
	`394`	`+message= Email()`
	`395`	`+message.send()`
`408`	`396`	```
`409`	`397`
`410`	`398`	`[⬆ back to top](#table-of-contents)`

Comments

(0)