Переписать функцию в ES6

[DarkPhoenix]

const timeEveryOneMinute = () => {
	let Data = new Date();
	let Hour = Data.getHours();
	let Minutes = Data.getMinutes();
		console.log("Текущее время: " + Hour + ":" + Minutes + " ");
		setTimeout(timeEveryOneMinute, 1000 * 60);
}
timeEveryOneMinute();

На курсах просят переписать в стиле ES6.
Прошу Гуру форума помочь.
Спасибо!

[рони]

DarkPhoenix,
вы спросите, чего в супе коде не хватает?

[рони]

DarkPhoenix,

const timeEveryOneMinute = () => {
	const data = new Date(),
	time = new Intl.DateTimeFormat("ru", {
        hour: "2-digit",
        minute: "2-digit"
    }).format(data);
		console.log(`Текущее время: ${time}`);
		setTimeout(timeEveryOneMinute, 1000 * 60);
}
timeEveryOneMinute();

[Aetae]

Оно и так стиле es6.

А в современном стиле оно как-то так:

import moment from 'moment'
import timer from 'some-super-timer'
import logger from 'some-super-logger'

timer.every(60).run(step => logger.log(
    moment().format("Текущее время: HH:mm")
));

(*код вымышлен, все совпадения случайны )

[Malleys]

https://github.com/airbnb/javascript...ng-conventions

[рони]

Malleys,
а можно словами или примером, для наглядности, что не так с кодом?

[Dilettante_Pro]

рони,
Можно предположить, что let Data и let Time противоречат этому:

Use PascalCase only when naming constructors or classes

[рони]

Dilettante_Pro,
поправил #3

[Malleys]

а можно словами или примером, для наглядности, что не так с кодом?

Обычно в JavaScript имена даются также, как в Java, но поскольку автор может не знать Java, или не понять, почему был упомянут Java, то я привёл руководство от Airbnb.

Я думаю, автору интересна последняя версия JavaScript, описанная в EcmaScript2019...

Конкретно пример автора может быть записан так...

(function timeEveryOneMinute() {
	const date = new Date();
	const hours = String(date.getHours()).padStart(2, "0");
	const minutes = String(date.getMinutes()).padStart(2, "0");

	console.log(`Текущее время: ${hours}:${minutes}`);
	setTimeout(timeEveryOneMinute, 1000 * 60);
})();

Пример рони так...

(function timeEveryOneMinute() {
	const date = new Date();
	const timeOptions = {
		hour: "2-digit",
		minute: "2-digit"
	};
	const time = date.toLocaleTimeString("ru", timeOptions);
	
	console.log(`Текущее время: ${time}`);
	setTimeout(timeEveryOneMinute, 1000 * 60);
})();

Пример Aetae возможен, но требует инструментов трансформации кода, сборки, наличие node.js (использование проприетарных импортов)

[Malleys]

console.log(String(new Date()).replace(/^.*?(\d+:\d+).*$/, `Текущее время: $1`));

Poznakomlus, 🙈🙉🙊