Skip to content

react-game - #1

Open
kli2m wants to merge 16 commits into
mainfrom
develop
Open

react-game#1
kli2m wants to merge 16 commits into
mainfrom
develop

Conversation

@kli2m

@kli2m kli2m commented Mar 3, 2021

Copy link
Copy Markdown
Owner
  1. The task
  2. Screen -
    image
  3. Demo
  4. Deadline - March 3, 2021 | 23:59
  5. Score: 70 / 100

##Basic scope +30

  • вёрстка, дизайн, UI. Выполняются требования к оформлению приложения +10
  • эффекты анимации. Анимация ходов, перемещения фигур, допустимых и недопустимых ходов, начала и окончания игры, победы и поражения и т.д +10
  • механика игры. Ходы, перемещения фигур, набранные баллы, окончание игры и т.д. подчиняются определённым свойственным игре правилам +10

##Advanced scope +40

  • звуки и музыка. В игре есть звуки и музыка, и есть настройки звуков и музыки: возможность вкл/откл звуки, вкл/откл музыку, регулировать громкость звуков, регулировать громкость музыки +10
  • настройки игры. Есть не меньше трёх настроек разных опций игры( установка сложности игры - изменяет время на ответ, установка уровня игры - сложность слов, смена языка) +10
    статистика. ведётся запись ходов, побед и поражений. Есть таблица рекордов, в которой сохраняются результаты предыдущих 10 и более игр +10
  • сохранение состояния игры при перезагрузке страницы ( переход в главное меню - если набрать Ник из предыдущих сыгранных игр Сохраняются все опции игры: ходы и набранные очки) Состояние игры сохраняется без необходимости кликать на кнопку save +10
  • возможность управления игрой с клавиатуры или не меньше пяти hot keys. Есть список горячих клавиш для управления игрой с клавиатуры + 0

##Hacker scope +0

  • автопроигрывание игры. Есть кнопка Autoplay запускающая игру в автоматическом режиме. Это может быть отдельная партия игры (крестики-нолики, шашки, шахматы, морской бой, судоку и т.д.). Для бесконечных игр (тетрис, Flappy Bird и т.д), игра зацикливается не раньше чем через 30 ходов. Речь не про автозавершение начатой игры, автопроигрывание игры начинает игру с пустого поля или стандартной раскладки и показывает как эта игра сама играет без пользователя +0
  • дополнительная функциональность. Реализована достаточно сложная в реализации дополнительная функциональность, которая улучшает качество игры. В качестве дополнительной функциональности рекомендуется создать и использовать бекенд, например, для хранения счёта игры. Также может быть и другая дополнительная функциональность, например, 3d игра, написана логика для компьютерного противника, есть сетевой режим игры, и т.д +0

Comment thread src/App.js
import imgGit from "./assets/img/github.png";

const App = () => {
localStorage.getItem("statistics")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Из тернарника надо куда-то присваивать значение, Если не надо присвивать то надо использовать if

Comment thread package.json
"antd": "^4.12.3",
"react": "^17.0.1",
"react-beautiful-dnd": "^13.0.0",
"react-dom": "^17.0.1",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Убрать ^

Comment thread src/App.scss
@@ -0,0 +1,67 @@
.letterSolver__wrapper {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Опционально. Можно полностью подогнать под BEM

</div>{" "}
</Text>
<Text type="warning">
{" "}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

так и должно быть {" "}?

height: 50px;
background-color: rgb(58, 218, 106);
box-shadow: 0px 0px 9px 5px rgba(143, 72, 72, 0.75) inset;
-webkit-box-shadow: 0px 0px 9px 5px rgba(143, 72, 72, 0.75) inset;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

-webkit-box- можно и без префиксов

<div>
<ul>
{Object.keys(getStatisticsLocal).map((e, i) => {
return (

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

delete return


const userLocal = getStatisticsLocal[user.name];

if (!userLocal[user.difficultLevel])

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Фигурные скобки


if (!userLocal[user.difficultLevel])
userLocal[user.difficultLevel] = { [user.wordsLevel]: [] };
else {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Фигурные скобки

else {
const diffLevel = userLocal[user.difficultLevel];

if (!diffLevel[user.wordsLevel]) diffLevel[user.wordsLevel] = [];

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Фигурные скобки

Comment thread src/utils/getWordsApi2.js
const getWordsSpeakit2 = async (group) => {
try {
let requests = new Array(30)
.fill("")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

зачем .fill("")?

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