Skip to content

Катюшкина #5

Open
NastyaKatyushkina wants to merge 4 commits intoRTF-Angular-2021:masterfrom
NastyaKatyushkina:katyushkina
Open

Катюшкина #5
NastyaKatyushkina wants to merge 4 commits intoRTF-Angular-2021:masterfrom
NastyaKatyushkina:katyushkina

Conversation

@NastyaKatyushkina
Copy link

No description provided.

const x = {m: 7};

console.log(getProperty(x, "m")); No newline at end of file
console.log(getProperty(x, "m"));

Choose a reason for hiding this comment

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

3 бала

email = newmail;
console.log("email valid")
}
throw "Invalid email"

Choose a reason for hiding this comment

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

Всегда будет эксепшен, когда устанавливают e-mail в поле.

class Example {
public email: string = "";
}
function check_mail(target: object, propertyKey: string | symbol): any{

Choose a reason for hiding this comment

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

Декоратор не будет работать.

public exec(): string
{
return 'Result of the addition operation ${this.a} + ${this.b} = ${super.exec()}'
}

Choose a reason for hiding this comment

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

Это не паттерн декоратор

@m-abrosimov
Copy link

итого 3 бала

@m-abrosimov
Copy link

+3 балла

}
return discriptor;
}

Choose a reason for hiding this comment

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

2 балла

return 'Result of the addition operation ${this.a} + ${this.b} = ${super.exec()}'
return 'Result of the addition operation ${this.a} + ${this.b} = ${this.calculators.exec()}'
}
} No newline at end of file

Choose a reason for hiding this comment

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

1 балл

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

Successfully merging this pull request may close these issues.

2 participants