workAroundExplorer - Please review my code!

Hi there -
At step 7, my code quits working. I know it’s something to do with how I’m trying to import/export the workAroundModule, but I just can’t figure it out. Any help would be greatly appreciated!


// TODO: Add your import statements here.
import default workAroundModule from './modules/workAroundModule.js'
import { getIndustryAverageSalary, getSalaryAtCompany, getAverageSalaryByCompany, getAverageSalaryByRole } from './modules/workAroundModule.js'
import { getRoles, getCompanies } from './modules/salaryData.js'

// TODO: Get the companies and roles using the salaryData module.
const companies = getCompanies();
const roles = getRoles();

// Create input buttons for every company and role represented in the data.
renderInputButtons(companies, 'company');
renderInputButtons(roles, 'role');

// This function will create a new <section> with radio
// inputs based on the data provided in the labels array.
function renderInputButtons(labels, groupName) {
  const container = document.createElement('section');
  container.setAttribute('id', `${groupName}Inputs`);

  let header = document.createElement('h3');
  header.innerText = `Select a ${groupName}`;

  labels.forEach(label => { // For each label...
    // Create the radio input element.
    let divElement = document.createElement('div');
    divElement.setAttribute('class', 'option');

    let inputElement = document.createElement('input');
    inputElement.setAttribute('type', 'radio');
    inputElement.setAttribute('name', groupName);
    inputElement.setAttribute('value', label);

    // Create a label for that radio input element.
    let labelElement = document.createElement('label');
    labelElement.setAttribute('for', label);
    labelElement.innerText = label;

    // Update the results when the input is selected.
    inputElement.addEventListener('click', updateResults);



function updateResults(){
  // Get the current selected company and role from the radio button inputs.
  const company = document.querySelector("input[name='company']:checked").value;
  const role = document.querySelector("input[name='role']:checked").value;

  // If either the company or role is unselected, return.
  if (!company || !role) { return; }

  // TODO: Use the workAroundModule functions to calculate the needed data.
  const averageSalaryByRole = getAverageSalaryByRole(role);
  const averageSalaryByCompany = getAverageSalaryByCompany(company);
  const salary = getSalaryAtCompany(role, company);
  const industryAverageSalary = getIndustryAverageSalary();

  // Render them to the screen.
  document.getElementById('salarySelected').innerText = `The salary for ${role}s at ${company} is \$${salary}`;
  document.getElementById('salaryAverageByRole').innerText = `The industry average salary for ${role} positions is \$${averageSalaryByRole}`;
  document.getElementById('salaryAverageByCompany').innerText = `The average salary at ${company} is \$${averageSalaryByCompany}`;
  document.getElementById('salaryAverageIndustry').innerText = `The average salary in the Tech industry is \$${industryAverageSalary}`;


// Add your imports here.
import default salaryData from "./salaryData.js"
import { roles, companies } from './salaryData.js'
// Replace the empty array with the appropriate imported function/value
const getAverageSalaryByRole = role => {
  const roleData = getDataByRole(role);
  const salariesOfRole = => obj.salary);
  return calculateAverage(salariesOfRole);

// Replace the empty array with the appropriate imported function/value
const getAverageSalaryByCompany = company => {
  const companyData = getDataByCompany(company);
  const salariesAtCompany = => obj.salary);
  return calculateAverage(salariesAtCompany);

// Replace the empty array with the appropriate imported function/value
const getSalaryAtCompany = (role, company) => {
  const companyData = getDataByCompaany(company);
  const roleAtCompany = companyData.find(obj => obj.role === role);
  return roleAtCompany.salary;

// Replace the empty array with the appropriate imported function/value
const getIndustryAverageSalary = () => {
  const allSalaries = => obj.salary);
  return calculateAverage(allSalaries);

// Helper Function. Do not edit.
// Note: This function does not need to be exported since it is only used by the functions contained within this module.
function calculateAverage(arrayOfNumbers) {
  let total = 0;
  arrayOfNumbers.forEach(number => total += number);
  return (total / arrayOfNumbers.length).toFixed(2);

export { getIndustryAverageSalary, getSalaryAtCompany, getAverageSalaryByCompany, getAverageSalaryByRole };

Thanks! Unfortunately, that didn’t fix the issues.

Ok, on main.js:

  1. import default workAroundModule from './modules/workAroundModule.js you don’t need this line

On workAroundModule.js:

  1. import default salaryData from "./salaryData.js" remove the default keyword
  2. import { roles, companies } from './salaryData.js' should be getDataByRole and getDataByCompany
  3. const companyData = getDataByCompaany(company); fix the typo

On salaryData.js

it should be like this at the end:

export { getRoles, getCompanies, getDataByRole, getDataByCompany };
export default salaryData;

and on index.html don’t forget to add the type=“module”

<script type="module" src="main.js" defer></script>

1 Like

Thanks! This worked!