reactjs the syntaxError: url is readonly

Question

i have this issue in react i have this function but its not right format of react

check(img) {
        console.log(img,typeof img)
        const url="";
        const arrN = ["15","16","35","36","37","38","39","40","n15","n16","n35","n36","n37","n38","n39","n40"];
        for (var i = 0; i < arrN.length; i++) {
            if (img === arrN[i]) {
                 url = "/blah/allIcons/blah"+img+"_en.png";
            }else{
                 url = "/blah/allIcons/blah"+img+".png";
            }
        }
        return url;
    }

it give me this errror

Module build failed: SyntaxError: "url" is read-only

how i can do it ?

Maybe use have defined url above. just use url = "" instead of const url = ""; — Adnan Umer
– Adnan Umer, Commented Feb 9, 2017 at 12:32
@AdnanUmer Very bad advice. I guess you don't know why declaring variable is important. — dfsq
– dfsq, Commented Feb 9, 2017 at 12:37
const is something that u can't change, use var or let, read the diff between const, var, let. stackoverflow.com/questions/762011/… — Mayank Shukla
– Mayank Shukla, Commented Feb 9, 2017 at 12:52

dfsq · Accepted Answer · 2017-02-09 13:18:31Z

5

If you change url variable then it's should not be declared as a constant. Use let:

check(img) {
  const arrN = ["15", "16", "35", "36", "37", "38", "39", "40", "n15", "n16", "n35", "n36", "n37", "n38", "n39", "n40"];
  let url = "";

  for (var i = 0; i < arrN.length; i++) {
    if (img === arrN[i]) {
      url = "/blah/allIcons/blah" + img + "_en.png";
    } else {
      url = "/blah/allIcons/blah" + img + ".png";
    }
  }
  return url;
}

But you don't seem to need it anyway, as entire for-loop check seems inefficient. Can be optimized like this:

check(img) {
  const arrN = ["15", "16", "35", "36", "37", "38", "39", "40", "n15", "n16", "n35", "n36", "n37", "n38", "n39", "n40"];

  if (arrN.indexOf(img) > -1) { // or if (arrN.includes(img)) {...}
    return "/blah/allIcons/blah" + img + "_en.png";
  }

  return "/blah/allIcons/blah" + img + ".png";
}

edited Feb 9, 2017 at 13:18

answered Feb 9, 2017 at 12:35

dfsq

193k26 gold badges244 silver badges261 bronze badges

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

1 Comment

Mayank Shukla Over a year ago

it should not be declared as a constant :)

IVN · Accepted Answer · 2017-02-10 16:34:43Z

0

Use let url="" instead of const url="".

The value of a constant cannot change through re-assignment, and it can't be redeclared.

So if you declare variable const url="", you can't later say url="/blah/allIcons/blah" + img + "_en.png"

edited Feb 10, 2017 at 16:34

answered Feb 9, 2017 at 12:35

IVN

1,34715 silver badges19 bronze badges

2 Comments

Robert Over a year ago

Could you provide more info and details with your answer? Thank you

IVN Over a year ago

@Robert, I updated my answer with some more info and references, however I will not go into details explaining let and const here, because question is not about it.

Collectives™ on Stack Overflow

reactjs the syntaxError: url is readonly

2 Answers 2

1 Comment

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related