Skip to content

ооп#2

Open
ShevchenkoAlexen wants to merge 6 commits into
js-improvement:masterfrom
ShevchenkoAlexen:ООП

Hidden character warning

The head ref may contain hidden characters: "\u041e\u041e\u041f"
Open

ооп#2
ShevchenkoAlexen wants to merge 6 commits into
js-improvement:masterfrom
ShevchenkoAlexen:ООП

Conversation

@ShevchenkoAlexen

Copy link
Copy Markdown

No description provided.

Comment thread index.js

var phoneBook = require('./phoneBook');

var phoneBook = require('./phoneBook');

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ты рекваеришь инстанс, а должен рекваерить класс:

var PhoneBook = require('./phoneBook');
var book = new PhoneBook();
`
Comment thread phoneBook.js
var PhoneBook = function () {


Array.call(this);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ты хочешь отнаследоваться от массива, не нужно, сделай собственный класс

Comment thread phoneBook.js
}


function find(pBook, query) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тебе не нужно выносить эти функции отдельно, помести их внутрь класса

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants