Skip to content

Commit 6321a5a

Browse files
committed
Use utf8 for readFile and minor optimizations
1 parent 587ba93 commit 6321a5a

File tree

10 files changed

+25
-30
lines changed

10 files changed

+25
-30
lines changed

JavaScript/1-callbacks.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@
22

33
const fs = require('fs');
44

5-
fs.readFile('file1.txt', (err, data) => {
5+
fs.readFile('file1.txt', 'utf8', (err, data) => {
66
console.log(err || data.toString());
7-
fs.readFile('file2.txt', (err, data) => {
7+
fs.readFile('file2.txt', 'utf8', (err, data) => {
88
console.log(err || data.toString());
9-
fs.readFile('file3.txt', (err, data) => {
9+
fs.readFile('file3.txt', 'utf8', (err, data) => {
1010
console.log(err || data.toString());
1111
});
1212
});

JavaScript/2-promise.js

+9-9
Original file line numberDiff line numberDiff line change
@@ -2,21 +2,21 @@
22

33
const fs = require('fs');
44

5-
const readFile = (filename) => new Promise((resolve, reject) => {
6-
fs.readFile(filename, (err, data) => {
7-
if (err) reject(err);
8-
else resolve(data.toString());
9-
});
10-
});
5+
const readFile = (filename, encoding) =>
6+
new Promise((resolve, reject) =>
7+
fs.readFile(filename, encoding, (err, data) => {
8+
if (err) reject(err);
9+
else resolve(data.toString());
10+
}));
1111

12-
readFile('file1.txt')
12+
readFile('file1.txt', 'utf8')
1313
.then(data => {
1414
console.log(data);
15-
return readFile('file2.txt');
15+
return readFile('file2.txt', 'utf8');
1616
})
1717
.then(data => {
1818
console.log(data);
19-
return readFile('file3.txt');
19+
return readFile('file3.txt', 'utf8');
2020
})
2121
.then(data => {
2222
console.log(data);

JavaScript/3-promisify.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,14 @@ const promisify = require('./promisify');
55

66
const readFile = promisify(fs.readFile);
77

8-
readFile('file1.txt')
8+
readFile('file1.txt', 'utf8')
99
.then(data => {
1010
console.log(data.toString());
11-
return readFile('file2.txt');
11+
return readFile('file2.txt', 'utf8');
1212
})
1313
.then(data => {
1414
console.log(data.toString());
15-
return readFile('file3.txt');
15+
return readFile('file3.txt', 'utf8');
1616
})
1717
.then(data => {
1818
console.log(data.toString());

JavaScript/4-then.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const { promisify } = require('util');
55

66
const readFile = promisify(fs.readFile);
77

8-
const readTextFile = filename => readFile(filename)
8+
const readTextFile = filename => readFile(filename, 'utf8')
99
.then(data => data.toString());
1010

1111
readTextFile('file1.txt')

JavaScript/5-catch.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const { promisify } = require('util');
55

66
const readFile = promisify(fs.readFile);
77

8-
const readTextFile = filename => readFile(filename)
8+
const readTextFile = filename => readFile(filename, 'utf8')
99
.then(data => data.toString());
1010

1111
readTextFile('file1.txt')

JavaScript/6-reject.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const { promisify } = require('util');
55

66
const readFile = promisify(fs.readFile);
77

8-
const readTextFile = filename => readFile(filename)
8+
const readTextFile = filename => readFile(filename, 'utf8')
99
.then(data => data.toString());
1010

1111
readTextFile('file1.txt')

JavaScript/a-files.js

+2-3
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,8 @@ const { promisify } = require('util');
66

77
const readFile = promisify(fs.readFile);
88

9-
function readTextFile(filename) {
10-
return readFile(filename).then(buffer => buffer.toString());
11-
}
9+
const readTextFile = filename => readFile(filename)
10+
.then(buffer => buffer.toString());
1211

1312
async(function* () {
1413
console.log(yield readTextFile('file1.txt'));

JavaScript/async-await/files.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const { promisify } = require('util');
66
const readFile = promisify(fs.readFile);
77

88
const readTextFile = async (filename) => {
9-
let data = await readFile(filename);
9+
let data = await readFile(filename, 'utf8');
1010
return data.toString();
1111
};
1212

JavaScript/b-errors.js

+2-4
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,8 @@ const async = require('./async');
66

77
const readFile = promisify(fs.readFile);
88

9-
function readTextFile(filename) {
10-
return readFile(filename)
11-
.then(buffer => buffer.toString());
12-
}
9+
const readTextFile = filename => readFile(filename, 'utf8')
10+
.then(buffer => buffer.toString());
1311

1412
async(function* () {
1513
console.log(yield readTextFile('file1.txt'));

JavaScript/c-catch.js

+2-4
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,8 @@ const async = require('./async');
66

77
const readFile = promisify(fs.readFile);
88

9-
function readTextFile(filename) {
10-
return readFile(filename)
11-
.then(buffer => buffer.toString());
12-
}
9+
const readTextFile = filename => readFile(filename, 'utf8')
10+
.then(buffer => buffer.toString());
1311

1412
async(function* () {
1513
console.log(yield readTextFile('file1.txt'));

0 commit comments

Comments
 (0)