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

Coding Style #174

Merged
merged 25 commits into from
Jun 8, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 17 additions & 17 deletions 1-js/03-code-quality/02-coding-style/1-style-errors/solution.md
Original file line number Diff line number Diff line change
@@ -1,29 +1,29 @@

You could note the following:
Bạn có thể lưu ý những điều sau:

```js no-beautify
function pow(x,n) // <- no space between arguments
{ // <- figure bracket on a separate line
let result=1; // <- no spaces before or after =
for(let i=0;i<n;i++) {result*=x;} // <- no spaces
// the contents of { ... } should be on a new line
function pow(x,n) // <- không có khoảng cách giữa các đối số
{ // <- dấu ngoặc đơn trên một dòng riêng biệt
let result=1; // <- không có dấu cách trước hoặc sau dấu =
for(let i=0;i<n;i++) {result*=x;} // <- không có dấu cách
// nội dung của { ... } nên ở một dòng mới
return result;
}

let x=prompt("x?",''), n=prompt("n?",'') // <-- technically possible,
// but better make it 2 lines, also there's no spaces and missing ;
if (n<=0) // <- no spaces inside (n <= 0), and should be extra line above it
{ // <- figure bracket on a separate line
// below - long lines can be split into multiple lines for improved readability
alert(`Power ${n} is not supported, please enter an integer number greater than zero`);
let x=prompt("x?",''), n=prompt("n?",'') // <-- có thể trên lý thuyết,
// nhưng tốt hơn là viết thành 2 dòng, cũng như không có dấu cách và thiếu ;
if (n<=0) // <- không có khoảng trắng bên trong (n <= 0) và phải có thêm dòng phía trên nó
{ // <- dấu ngoặc trên một dòng riêng biệt
// bên dưới - các dòng dài có thể được chia thành nhiều dòng để dễ đọc hơn
alert(`Luỹ thừa ${n} không được hỗ trợ, vui lòng nhập một số nguyên lớn hơn 0`);
}
else // <- could write it on a single line like "} else {"
else // <- có thể viết nó trên một dòng như "} else {"
{
alert(pow(x,n)) // no spaces and missing ;
alert(pow(x,n)) // không có dấu cách và thiếu ;
}
```

The fixed variant:
Biến thể cố định:

```js
function pow(x, n) {
Expand All @@ -40,8 +40,8 @@ let x = prompt("x?", "");
let n = prompt("n?", "");

if (n <= 0) {
alert(`Power ${n} is not supported,
please enter an integer number greater than zero`);
alert(`Luỹ thừa ${n} không được hỗ trợ,
vui lòng nhập một số nguyên lớn hơn 0`);
} else {
alert( pow(x, n) );
}
Expand Down
6 changes: 3 additions & 3 deletions 1-js/03-code-quality/02-coding-style/1-style-errors/task.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ importance: 4

# Bad style

What's wrong with the code style below?
Có gì sai với cách viết code này?

```js no-beautify
function pow(x,n)
Expand All @@ -17,12 +17,12 @@ function pow(x,n)
let x=prompt("x?",''), n=prompt("n?",'')
if (n<=0)
{
alert(`Power ${n} is not supported, please enter an integer number greater than zero`);
alert(`Luỹ thừa ${n} không được hỗ trợ, vui lòng nhập một số nguyên lớn hơn 0`);
}
else
{
alert(pow(x,n))
}
```

Fix it.
Hãy sửa nó.
Loading