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

Больше деталей #3

Merged
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
1 change: 1 addition & 0 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
<meta charset="utf-8">
<meta name="viewport" content="width=device-width,initial-scale=1">
<script src="./js/functions.js"></script>
<script src="./js/main.js"></script>
<link rel="stylesheet" href="css/normalize.css">
<link rel="stylesheet" href="css/style.css">
<link rel="shortcut icon" href="favicon.ico" type="image/x-icon">
Expand Down
2 changes: 1 addition & 1 deletion js/functions.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

checkPalindrom("топот");

function findNumber(inputString) {

Check warning on line 17 in js/functions.js

View workflow job for this annotation

GitHub Actions / Check

'findNumber' is defined but never used
inputString = inputString.toString();
let result = "";
for (let i of inputString) {
Expand All @@ -26,4 +26,4 @@
return (parseInt(result, 10));
}

console.log(findNumber('22 lalal 1'));

66 changes: 66 additions & 0 deletions js/main.js
Original file line number Diff line number Diff line change
@@ -1,0 +1,66 @@
const POST_COUNT = 25;

Choose a reason for hiding this comment

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

not bad 👍🏻
В следующей практике еще унесется в отдельный файлик и будет красота ☺️

const NAMES = [

Choose a reason for hiding this comment

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

Хорошая практика называть константы CONSTANT_CASE, но вот с массивами есть нюанс. Их содержимое можно изменять, и по сути это не константы. Поэтому тут часто от соглашений в команде зависит, но html Academy тут говорит, что перечисления (массивы и объекты) так называть не стоит. В общем стоит помнить про возможное написание разными способами и уже ориентироваться по месту на проекте)
Мы в команде также используем обычно camelCase для массивов, знаю команды, кто именует CONSTANT_CASE. Оставить можешь так. А вот Enum из Typescript, это другое дело. Он то настоящая константа, его так можно называть) Но это потом, сейчас просто затравочка, скоро все узнаете)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Хорошо, поняла!

"Наташа",
"Алексей",
"Лиза",
"Мирослава",
"Дмитрий",
"Данила",
"Софа",
"Марьяна"
];

const MESSAGES = [
"Всё отлично!",
"В целом всё неплохо. Но не всё.",
"Когда вы делаете фотографию, хорошо бы убирать палец из кадра. В конце концов это просто непрофессионально.",
"Моя бабушка случайно чихнула с фотоаппаратом в руках и у неё получилась фотография лучше.",
"Я поскользнулся на банановой кожуре и уронил фотоаппарат на кота и у меня получилась фотография лучше.",
"Лица у людей на фотке перекошены, как будто их избивают. Как можно было поймать такой неудачный момент?!"
];

const DESCRIPTIONS = [
"на чилле, на раслабоне!",
"Анапа 2024",
"Дача, шашлыки, что еще нужно для счастья?",

Choose a reason for hiding this comment

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

еее, ну только еще быть на раслабоне 😄

"Not pickme",
"В качалке с друзьями",
"Гламур"
];

//функции для рандомных чисел
const getRandomInteger = (a, b) => {
const lower = Math.ceil(Math.min(a, b));
const upper = Math.floor(Math.max(a, b));
const result = Math.random() * (upper - lower + 1) + lower;
return Math.floor(result);
};

const createComment = () => {
const randomNameIndex = getRandomInteger(0, NAMES.length - 1);
return {
id:getRandomInteger(0,30),
avatar:`img/avatar-${ getRandomInteger(1,6) }.svg`,
message: Array.from({length:getRandomInteger(1,2)}, () => MESSAGES[getRandomInteger(0, MESSAGES.length - 1)]).join("\n"),
name: NAMES[randomNameIndex]
};
};

const createComments = () => {
const comments = [];
for (let i = 0;i < getRandomInteger(0,31);i++){
comments.push(createComment());
}
return comments;
};
const createPost = (elemt,index) =>({
id:index,
url:`photos/${ index }.jpg`,
description:DESCRIPTIONS[getRandomInteger(0,DESCRIPTIONS.length - 1)],
likes:getRandomInteger(15,200),
comments:createComments()
});


const posts = Array.from({length:POST_COUNT},createPost);
console.log(posts);

Check warning on line 66 in js/main.js

View workflow job for this annotation

GitHub Actions / Check

Unexpected console statement