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

Chapter 5 #8

Open
LNUitTutor opened this issue Jul 15, 2024 · 0 comments
Open

Chapter 5 #8

LNUitTutor opened this issue Jul 15, 2024 · 0 comments

Comments

@LNUitTutor
Copy link

  • subsec. 5.1, p. 21 "LNNode >> testHasNextNode" should be "LNNodeTest >> testHasNextNode"
  • subsec. 5.2: confusion with node naming: in the previous subsection was declared "Document the mutators to inform users that the argument passed to name: should be a Symbol..." but on p. 22 in the code snippet "(LNNode new name: 'Mac' ; nextNode: (LNNode new name: 'PC1'))" we see strings. So, what we have to use: symbols or strings?
  • subsec. 5.3 holds the wrong instruction "Create an instance method named printOn: that puts the class name and name variable on the argument aStream." Why we need to put the class name? The test methods do not expect name of the class in the print string.
  • subsec. 5.4, p. 24: LNPacketTest >> testIsAddressedTo - self assert missed.
  • subsec. 5.5, p. 25: parentheses - fugitives. In the first code snipped "yourself) accept: (LNPacket new addresseeName: 'Mac')" should be "yourself )) accept: (LNPacket new addresseeName: 'Mac')". In the second one "LNWorkstation new" should be " ( LNWorkstation new".
  • subsec. 5.7, p. 27: "LNNode class >> simpleLan..." declaration missed of the temporary variables node3, packet . "mac := Workstation new name: 'mac'." should be "mac := LNWorkstation new name: 'mac'."
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

1 participant