Игра 12 этап

Этот сайт лучше всего просматривать в современном браузере с включённым JavaScript.

Игра 12 этап

vladozz

ошибка 4. checkInitCompleted должен вызываться из handleKeyPress только один раз за время существования
игры.

Код обработчика нажатия клавиш

export const handleKeyPress = (term, state) => {
    return (name, matches, data) => {
        if (String.fromCharCode(data.code) === 'G' || String.fromCharCode(data.code) === 'g') {
            state.gold++;
        }

        for (let i=0;i<state.producers.length;i++){
            if (String.fromCharCode(data.code) === `${state.producers[i].id}`) {
            state.gold-= state.producers[i].cost;
            state.producers[i].cost=state.producers[i].cost*state.producers[i].growthRate;
            state.producers[i].count++;
            state.productionRate+=state.producers[i].baseProduction;
          }
        }

        if (String.fromCharCode(data.code) === '9') {
          throw new Error ('Закончили упражнение');
      }

      if (state.gold>=10 && state.isInitCompleted === false) {
        checkInitCompleted (term,state);
        state.checkInitCompleted = true;
      }
  }

Coderslang_Master

vladozz с кодом все хорошо, но вижу что не хватает одной закрывающей скобки. До сих пор видишь ту же ошибку?

vladozz

Coderslang_Master

да проблема остается, думал может как-то на блок с ошибкой реагирует (добавил чтоб прерывать программу), удалял и его, история та же. Код выполняется верно, все обработки работают. может где-то в другой части кода проблемы могут быть?

скобки проверил, все на месте.

Coderslang_Master

vladozz возможно. Присылай все файлы, посмотрю. Только оформи красиво. Три обратные кавычки в строке перед началом каждого код сниппета, три в строке после окончания.

Я для примера отредактировал твой начальный пост.

vladozz

Coderslang_Master

functions.js

export const init = (term) => {
    term.clear ();
    term ('Welcome to the mining game! Press \'G\' to start!');
    term.hideCursor();
    term.grabInput();
}


export const updateGold = (term, state) => {
    state.gold+=state.productionRate;
    term.moveTo(25,2);
    term.eraseLineAfter();
    term.bold.yellow(state.gold + '   ');

}

export const checkInitCompleted  = (term, state) => {
    term.moveTo(1,1);
    term.eraseLineAfter();
    term('You can purchase producers by clicking the number button (1, 2, 3, ...)');
    term.moveTo(1,2);
    term ('GOLD:')
    term.moveTo(1,3);
    term ('PRODUCTION RATE: ')
}

constants.js

import terminalKit from 'terminal-kit';
export const term = terminalKit.terminal;
export let config  = {gold: 0, productionRate: 0, isInitCompleted: false ,isProducerListUpdated: true, producers: [
{ id: 1, title: 'Miner', cost: 10, growthRate: 1.13, baseProduction: 0.1, count: 0 },
{ id: 2, title: 'Adventurer', cost: 100, growthRate: 1.17, baseProduction: 1, count: 0 },
{ id: 3, title: 'Professional', cost: 1200, growthRate: 1.14, baseProduction: 9, count: 0 } ]
};

gameEngine.js

import { handleKeyPress,handleStateChange } from './handlers.js';
import {init}  from './functions.js';

export const startMiningGame = (term,state) => {
    init(term);

    term.on('key', handleKeyPress (term, state));
    handleKeyPress (term, state);

    setInterval(handleStateChange(term,state),1000);
}

handlers.js

import { updateGold,checkInitCompleted } from "./functions.js";


export const handleStateChange = (term, state) => () => {
  updateGold(term, state);
  }

export const handleKeyPress = (term, state) => {
    return (name, matches, data) => {
        if (String.fromCharCode(data.code) === 'G' || String.fromCharCode(data.code) === 'g') {
            state.gold++;
        }

        for (let i=0;i<state.producers.length;i++){
            if (String.fromCharCode(data.code) === `${state.producers[i].id}`) {
            state.gold-= state.producers[i].cost;
            state.producers[i].cost=state.producers[i].cost*state.producers[i].growthRate;
            state.producers[i].count++;
            state.productionRate+=state.producers[i].baseProduction;
          }
        }

        if (state.gold>=10 && state.isInitCompleted === false) {
          checkInitCompleted (term,state);
          state.checkInitCompleted = true;
        }
  }
}

Coderslang_Master

vladozz Нашел ошибку. Перенеси переключение на state.isInitCompleted = true в checkInitCompleted.

vladozz

Coderslang_Master

Та же история, не проходит проверку

export const checkInitCompleted  = (term, state) => {
    term.moveTo(1,1);
    term.eraseLineAfter();
    term('You can purchase producers by clicking the number button (1, 2, 3, ...)');
    term.moveTo(1,2);
    term ('GOLD:')
    term.moveTo(1,3);
    term ('PRODUCTION RATE: ')
    state.checkInitCompleted = true;
}

Coderslang_Master

vladozz ты не ту переменную меняешь.

state.isInitCompleted = true

А у тебя checkInitCompleted = true

vladozz

Coderslang_Master

😅 пошел мозг проветрю... бдительность на нуле... тысячу раз пересмотрел. спасибо, прошло

vladozz

Coderslang_Master
едем дальше с игрой вот такая ошибка. В запущеном коде все делается согласно логике при достижении 10 золота отображается только производитель Miner. когда покупаем 1 Miner появляется вторая строка и после покупки второго третья строка. может я что-то из задания не так понял.

export const updateProducerList= (term, state) => {
    state.isProducerListUpdated = true;
    term.moveTo(1,5);
    if (state.producers [0].count === 0){
        term(`Miner: ${state.producers [0].count} | Production per second: ${(state.producers [0].baseProduction*state.producers [0].count).toFixed (1)} | Cost: ${(state.producers [0].cost).toFixed (1)}`);
    } else if (state.producers [0].count > 0 && state.producers [1].count === 0) {
        term(`Miner: ${state.producers [0].count} | Production per second: ${(state.producers [0].baseProduction*state.producers [0].count).toFixed (1)} | Cost: ${(state.producers [0].cost).toFixed (1)}\nAdventurer: ${state.producers [1].count} | Production per second: ${(state.producers [1].baseProduction*state.producers [1].count).toFixed (1)} | Cost: ${(state.producers [1].cost).toFixed (1)}`);
    } else if (state.producers [1].count > 0){
        term(`Miner: ${state.producers [0].count} | Production per second: ${(state.producers [0].baseProduction*state.producers [0].count).toFixed (1)} | Cost: ${(state.producers [0].cost).toFixed (1)}\nAdventurer: ${state.producers [1].count} | Production per second: ${(state.producers [1].baseProduction*state.producers [1].count).toFixed (1)} | Cost: ${(state.producers [1].cost).toFixed (1)}\nProfessional: ${state.producers [2].count} | Production per second: ${(state.producers [2].baseProduction*state.producers [2].count).toFixed (1)} | Cost: ${(state.producers [2].cost).toFixed (1)}` );
    }
}

Coderslang_Master

vladozz ты выводишь не всех производителей. Нужен цикл по всем элементам массива.

vladozz

Coderslang_Master
Не понял вообще от слова совсем. производители выводятся как в логике задания последовательно один за одним по мере покупки. на каждое условие одна строка. Нужно выводить элементы массива через цикл? тогда будет нарушено условие что должна быть одна строка

vladozz

Coderslang_Master

export const updateProducerList= (term, state) => {
    let outString=[];
    let count;
    state.isProducerListUpdated = true;
    term.moveTo(1,5);

    if (state.producers [0].count === 0){
        count=1;
    } else if (state.producers [0].count > 0 && state.producers [1].count === 0) {
        count=2;
    } else count=3;
       for (let i=0;i<count;i++){
        outString+=`${state.producers [i].title}: ${state.producers [i].count} | Production per second: ${(state.producers [i].baseProduction).toFixed (1)} | Cost: ${(state.producers [i].cost).toFixed (1)} \n`;
       }
       term(outString);
}

Coderslang_Master

vladozz главное на что ты должен обратить внимание, это то, что функции updateProducerList не известна длина массива до того, как она начнет выполнение.