Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

misprints or erros in the book #1

Open
UtmostCreator opened this issue Jan 22, 2022 · 1 comment
Open

misprints or erros in the book #1

UtmostCreator opened this issue Jan 22, 2022 · 1 comment

Comments

@UtmostCreator
Copy link

PHP version errors
like on page 51
void not as of PHP 8 but it is as of PHP 7.1

page 131
if indentation

page 141
the return statement is in one line
must be in the next line

p. 361
listing 10.41
Why "DIR" is in bold?

p. 372
3rd paragraph has a misprint. and the next 3 occurrences, or please replace it inside of a class to lowercase word.
Now, it is like this:
dointerpret()
instead of
doInterpret() - it must be camalCase as it is inside of class

p. 384 | listing 11.22
there is no space after a keyword "class"
it looks like classLogin

p. 408 | listing 11.50
Fatal Errors
method get must return string; null is returned!
setError must not have a return type set to string!

p. 438
__construct()
parentheses is not closed

461,466 (the new line is recommended 1. before private 2. before final

p. 477
listing 12.39
return must be on the new line

p. 501 it would be more efficient if you would use $this->total instead of repeating count($raw)

p. 504 Show to Nikita
I would have had to have been careful

p. 664 1st paragraph (not in the note)
PHPUnit[remove space here]

p. 666 at the very beginning of a class 2 properties in one row. add a new line

p.674 I do not see the reason why we have this(greeting application registry instance):
$reg = ApplicationRegistry::instance()
as we do not use it anywhere in the runComand method

p.675 ob_start()
is it on purpose placed on the same line or did it happen by accident

notice:
class Conf is defined in chapter 4
but then you use it in chapter 12, I was confused where does it come from. before using old classes, please add a reference either in the very beginning of a section or by making a footnote in the footer.

@srikat
Copy link

srikat commented Aug 4, 2023

p48

If I now attempt to set $value to anything other than a float or a Boolean

should be

If I now attempt to set $value to anything other than a string or a Boolean

p74

$discount is declared private

Where is this done?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants