1

I need to solve this school exercise but I don't know how. Here's the text:

I need to make a program that takes information from the user via prompt() dialog. The information that user needs to enter is his name and sex (eg. Marc, m). Based on that information, the program needs to write in alert() dialog the adjectives from the (already given) list that start with the same letter as the letters from the name.

For example: user's name is Marc, and his sex is male. Program needs to use adjectives from the male list (there are two lists with adjectives, for both male and female), and to write them like this in alert dialog:

Mad

Accurate

Reasoning

Calculative

If you read their first letters vertically, they say MARC.

I have adjectives for all the letters in alphabet, for both male and female. Keep in mind that variable names and adjectives are on my native language (Serbian), but it shouldn't be the problem, you will get the point and I will explain the code in the comments.

var pridevi = {
  m: ["atraktivan", "blesav", "ciničan", "čudan", "ćopav", "duhovit", "džangrizav", "đavolast", "elokventan", "fantastičan", "grozan", "halapljiv", "imućan", "jak", "katastrofalan", "lep", "ljubazan", "mudar", "naivan", "njanjav", "otporan", "posesivan", "razigran", "smešan", "šaljiv", "tolerantan", "uobražen", "veseo", "zabrinut", "žut"],
  z: ["atraktivna", "blesava", "cinična", "čudna", "ćopava", "duhovita", "džangrizava", "đavolasta", "elokventna", "fantastična", "grozna", "halapljiva", "imućna", "jaka", "katastrofalna", "lepa", "ljubazna", "mudra",  "naivna", "njanjava", "otporna", "posesivna", "razigrana", "smešna", "šaljiva", "tolerantna", "uobražena", "vesela", "zabrinuta", "žuta"],
} // I stored adjectives in object where property m: stands for male and property f: stands for female adjectives

var unos = prompt("Upišite ime i pol. Npr. Mirko, m"); // prompt format

var ime = unos.toLowerCase().split(", ").shift(); // in this variable I stored name
var pol = unos.toLowerCase().split(", ").pop(); // in this variable I stored sex

// console.log(ime + " " + pol) > mirko m

if (unos === null) {
  alert("Korisnik je odustao."); // if user clicks cancel, this message shows in alert dialog
}
else if (unos === undefined && ime < 0 && pol < 0) {
  alert("Nisu uneseni ispravni podaci."); // if user doesn't write the data in correct form, this message shows in alert dialog
}
else {
  var odgovor = pridevi[pol].find(opis => ime[0] === opis[0]); // here's the main thing that doesn't work as it should. it only shows the adjective of the first letter of the name, but not all of them
  alert(odgovor);
}

Improve this question
5
  • Break the problem down into steps. First figure out how to get all the letters in the name. Then figure out how to find the elements of the array that start with a specific letter. Finally, put that together into a loop that processes all the letters of the name. Commented Sep 27, 2018 at 20:48
  • Who said that I didn't try to solve my problem first? I've tried everything and just don't know. Commented Sep 27, 2018 at 20:58
  • You have to show what you tried. Then we can see where you went wrong and show how to fix it. Commented Sep 27, 2018 at 21:00
  • String.prototype.charCodeAt() could be of use... Commented Sep 27, 2018 at 21:12
  • Okay, now I edited the code and put everything that I know. Hope someone can help now. Commented Sep 27, 2018 at 21:26

2 Answers 2

Reset to default
1

This answer relies on pattern matching, it is fairly brittle in that if someone doesn't follow the defined format it won't work correctly. A few notes:

  • Moved adjective groups into a holding object because accessing dynamic properties is fairly straightforward and requires less if/else chaining to get to the correct array group.
  • defined a few utility arrow functions. If those are unfamiliar, feel free to try them using standard function like so:
    • function notEmpty(val) { return (val !== undefined && val !== null && val.length > 0); }
  • Used Array.prototype.map to reformat the result array using a callback
  • Used array destructuring syntax you could replace that by defining those variables after getting the mapped result (const name = mappedArray[0])
  • in JS you don't need to split a string before accessing its indices so 'string'[0] would give you 's', which was useful in the Array.prototype.find callback function

// adjective group
const adjectives = {
  f: ["accurate", "bodybuilder", "calculative", "decisive" ],
  m: ["abstract", "beautiful", "capable", "delightful" ],
}

// utility function to check that a value exists and isn't empty
const notEmpty = val => (val !== undefined && val !== null && val.length > 0);

// prompt response holder (define a format)
const question = prompt("Write your given name and sex (format: Marc, m.)");

// function to format each entry in the response 
// /\W/g is a regex for stripping non alphanumeric 
const formatEntries = val => val.replace(/\W/g, '').toLowerCase();

// use format to split and handle your response
const [name, adjectiveGroup] = question.split(", ", 2)
  .map(formatEntries)

// if both variables are set then try and find matching adjective
if (notEmpty(name) && notEmpty(adjectiveGroup)) {
    const answer = adjectives[adjectiveGroup].find(adj => name[0] === adj[0])  
  
    console.log(answer);
}

Improve this answer
Sign up to request clarification or add additional context in comments.

7 Comments

Thank you so much for your answer and effort. I definitely made some progress thanks to your help. I will edit my question with the new code you gave me so you could see what's not working now.
looking at your update I would say you should look at map (in the links above) and try and figure out how you would use it to apply a change to every letter in the name.
I would also move your unos === null check above where you are splitting your code to avoid an Error like Uncaught TypeError: Cannot read property 'split' of null
Yeah, I will move unos === null above, it makes sense. When it comes to map, i have no idea how it works, we still haven't studied it yet so there should be another solution. :(
I managed to solve the problem, take a look at my answer and thank you so much! :D
|
0

After days of trying, I finally managed to solve this problem. I have to say that @D Lowther helped me with few lines, such as to store male and female adjectives in one object instead of two arrays and to split the entry from prompt on this way var [name, adjectiveGroup] = question.split(", ", 2). These were really useful.

You can check the program immediately by clicking on Run.

// in this object I stored all the adjectives, both male and female, because on my language they are various depending on the sex
var pridevi = {
  m: ["атрактиван", "блесав", "циничан", "чудан", "ћопав", "духовит", "џангризав", "ђаволаст", "елоквентан", "фантастичан", "грозан", "халапљив", "имућан", "јак", "катастрофалан", "леп", "љубазан", "мудар", "наиван", "њањав", "отпоран", "посесиван", "разигран", "смешан", "шаљив", "толерантан", "уображен", "весео", "забринут", "жут"],
  z: ["атрактивна", "блесава", "цинична", "чудна", "ћопава", "духовита", "џангризава", "ђаволаста", "елоквентна", "фантастична", "грозна", "халапљива", "имућна", "јака", "катастрофална", "лепа", "љубазна", "мудра",  "наивна", "њањава", "отпорна", "посесивна", "разиграна", "смешна", "шаљива", "толерантна", "уображена", "весела", "забринута", "жута"]
}

// prompt form
var unos = prompt("Упишите име и пол. Нпр. Мирко, м.");

// here I separated the prompt entry into array of two values: name and sex
var [ime, pol] = unos.toLowerCase().split(", ", 2);

// here I stored my native language because that language is required in prompt entry
var cirilica = /[\u0400-\u04FF]/;

// here I later stored the final array
var ispis = [];

// I made a function that converts the first letter of all values in array to uppercase
function velikoSlovo(a) {
  for (var i = 0; i < a.length; i++) {
    a[i] = a[i][0].toUpperCase() + a[i].substr(1);
  }
  return a.join("\n");
}

// if user clicks Cancel, the notification in alert window shows up
if (unos === null) {
  alert("Корисник је одустао.");
}

// here I tested various things: if the prompt entry is empty || if there's either name or sex missing || if it's not on my native language - the notification in alert window shows up
else if (unos.trim() === "" || (unos.split(", ").length < 2) || cirilica.test(unos) === false) {
  alert("Нису унесени исправни подаци.");
}

// if there are other values for sex instead of "m/f" (male/female), the notification in alert window shows up
else if ((unos.split(", ", 2)[1] !== "м") && (unos.split(", ", 2)[1] !== "ж")) {
  alert("Нису унесени исправни подаци.");  
}

// if prompt entry is correct (eg. Marc, m), I tested if the sex is male "м" or female "ж". Based on that, I used loop to compare the first letter of all the adjectives in array with all the letters of the name and then stored all the adjectives that match in the final array that I initialized before. Finally, I showed the final result in alert window and used the function to capitalize each word in string
else {
  if (pol === "м") {
    for (var i = 0; i < ime.length; i++) {
      for (var e = 0; e < pridevi.m.length; e++) {
        if (ime.charAt(i) === pridevi.m[e].charAt(0)) {
          ispis.push(pridevi.m[e]);
        }
      }
    }
    alert(velikoSlovo(ispis));
  }  
  
  else if (pol === "ж") {
    for (var i = 0; i < ime.length; i++) {
      for (var e = 0; e < pridevi.z.length; e++) {
        if (ime.charAt(i) === pridevi.z[e].charAt(0)) {
          ispis.push(pridevi.z[e]);
        }
      }
    }
    alert(velikoSlovo(ispis));  
  }     
}

Improve this answer

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.