Task 250 stage 16

Nick

Hi, could anybody help with task250 stage 16? I have "3. The purchase of a new producer should change production rate according to the current tickSpeed. Hint: it should become smaller."

I tried to do this as in my handleKeyPress function as:
const productionRate =
(state.producers.baseProduction / 1000) * state.tickSpeed;
state.productionRate += productionRate;

But it isn't passed... No idea what's wrong 🙁

Full handlers.js bellow

import {
  updateGold,
  checkInitCompleted,
  updateProducerList,
  formatNumber,
} from './functions.js';

export const handleKeyPress = (term, state) => {
  return (name, matches, data) => {
    const key = String.fromCharCode(data.code);
    if (key === 'G' || key === 'g') {
      state.gold++;
    } else {
      for (let i = 0; i < state.producers.length; i++) {
        if (key === String(state.producers[i].id - 1)) {
          state.gold -= state.producers[i].cost;
          state.producers[i].cost *= state.producers[i].growthRate;
          state.producers[i].count++;
          const productionRate =
            (state.producers[i].baseProduction / 1000) * state.tickSpeed;
          state.productionRate += productionRate;
          term.moveTo(25, 3);
          const speed = (state.productionRate * 1000) / state.tickSpeed;
          term.green(formatNumber(speed));
          term.green(formatNumber(state.gold));
        }
      }
    }

    if (!state.isInitCompleted) {
      for (let i = 0; i < state.producers.length; i++) {
        if (state.gold >= state.producers[i].cost) {
          state.isInitCompleted = true;
          checkInitCompleted(term, state);
        }
      }
    }

    if (!state.isProducerListUpdated) {
      updateProducerList(term, state);
    }
  };
};

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

http://prntscr.com/1vb81qv

Coderslang_Master

This looks suspicious. I don't see a reason to do -1 here:

String(state.producers[i].id - 1)

P.S. Also take a look at how your code looks now. I applied some basic markdown rules to it.

Nick

Coderslang_Master
The reason is next. We will purchase producers by clicking 1,2,3 but they have 0,1,2 index in producers array.

Thank you! Code looks better now with markdown.

Coderslang_Master

Take a look at the initial description in stage 6:
producers is an array of objects.
Each object represents an individual producer that has the following fields:

id: a unique number which is used to identify a producer
...

This means that the only thing that uniquely identifies the producer is the id field and not its position in the array.

Nick

Changed with your fix - it works now. Thank you! 🙂

Apostolos

Nick

What was the fix? I get the same error.