Skip to content

Instantly share code, notes, and snippets.

@sorenlouv
Last active June 16, 2023 15:26
Show Gist options
  • Star 13 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save sorenlouv/fc897c1629979e669714893df966b1b7 to your computer and use it in GitHub Desktop.
Save sorenlouv/fc897c1629979e669714893df966b1b7 to your computer and use it in GitHub Desktop.
React hook for getting a unique identifier for a component
import { useRef } from 'react';
let uniqueId = 0;
const getUniqueId = () => uniqueId++;
export function useComponentId() {
const idRef = useRef(getUniqueId());
return idRef.current;
}
@akomm
Copy link

akomm commented Feb 25, 2020

@macabeus I am not quite getting. You just posted the same thing, only creating the callback for no reason every time the function is called, saying its simpler?

@macabeus
Copy link

macabeus commented Feb 25, 2020

@AKMM As long I understand, useMemo is called just when the requirements is changed. So, in this expression:

useMemo(() => idCounter++, [prefix])

the idCounter will be increments only when prefix is changed, not every time like you said.

And, yes, this code is simpler. Using just 7 lines you can do everything that the 20 lines does.
And it uses fewer browser's features, such as localStorage.

useMemo fixes the problem that you said about useRef.

@TrevorBurnham
Copy link

TrevorBurnham commented Feb 25, 2020

Going by Dan Abramov's comment here, it sounds like the useRef implementation was on the right track. You can avoid calling getUniqueId() on every render by using a conditional on idRef.current:

let uniqueId = 0;
const getUniqueId = () => uniqueId++;

export function useComponentId() {
  const idRef = useRef(null);
  if (idRef.current === null) {
    idRef.current = getUniqueId()
  }
  return idRef.current;
}

@macabeus
Copy link

But using useRef for everything else that isn't a component is strange, at least for me.
Is more semantic and clear just using useMemo instead of useRef and on new line if (ref.current === null) {, that is almost the same of useMemo.

@akomm
Copy link

akomm commented Feb 26, 2020

@macabeus I think you did not understand my initial post. I posted a use case plus counter and you posted counter, telling its simpler. It does not make any sense. When I subtract the use case from my initial post, then you have posted the same counter I did, except you create the arrow function each time, while I don't - minor disadvantage, but no benefits, so what is the point. The creation of the arrow function has nothing to do with the arrow function being called once in this specific example. Its two different questions.

@TrevorBurnham
Thanks for the link. With the fix, it does make sense. I wanted to either point out an error in the gist (which it is as it seems) or get new insight in something I might not knew there. Got it via side-channel from you :). Conclusion: I guess useMemo is more generic and does argument iteration to check against the dependencies, so checking manually with an simple weak null check and useRef is more efficient. Makes sense :)

@macabeus
Copy link

@akomm Okay, now I understand your point. Thank you for reply =]

@mbelsky
Copy link

mbelsky commented Mar 20, 2020

Hey,

There is an alternative implementation with useState hook:

let uniqueId = 0;

export function useComponentId() {
  const [componentId] = useState(() => uniqueId++);
  return componentId;
}

@akomm
Copy link

akomm commented Mar 24, 2020

@mbelsky You can do that, but your example will increment the ID on each render call, even thought no new ID is required. You can fix it making the initialValue a calback () => uniqueId++. I'd create a meaningful hook at this point :)

Also see: https://twitter.com/dan_abramov/status/1099842565631819776?s=20 regarding useRef & useState

@mbelsky
Copy link

mbelsky commented Mar 24, 2020

@akomm I've tested my solution with this demo: https://codesandbox.io/s/determined-goldstine-qj761 A component rerenders there on click event and uniqueId stays same.

How should I change this demo to see behavior that you described?

@akomm
Copy link

akomm commented Mar 25, 2020

@mbelsky in your example, its the componentId which is the same, not uniqueId.

@mbelsky
Copy link

mbelsky commented Mar 25, 2020

@akomm could you explain please what the difference between them and how I can reproduce the issue that you described above?

but your example will increment the ID on each render call, even thought no new ID is required

I'm still not understanding what wrong with this solution

@akomm
Copy link

akomm commented Mar 25, 2020

just log the uniqueId to console in useComponentId and see if it is the same? You tell uniqueId is the same and post an example where you render the componentId. :)

@mbelsky
Copy link

mbelsky commented Mar 25, 2020

@akomm got it, thanks. I've updated originally post & demo. btw there is another interesting issue: uniqueId-s are odd numbers now. I'll try to find a solution for that

@akomm
Copy link

akomm commented Mar 25, 2020

@mbelsky

Your proposal worked. Its just a question if you want to increment the id forever on each render or not. The solution would be, as I proposed above, to replace uniqueId++ with () => uniqueId++.

@akomm
Copy link

akomm commented Mar 25, 2020

@mbelsky

codesandbox is flaky there, hence your odd numbers. Try it locally, it should work. I used to use codepen, then codesandbox came out and was better. Liked it. But recently it got more and more flaky. A pure bugfeast. Might be there bundler configuration or life reload that messes it up. IF you do some tests, you notice on life reload its rendered twice.

@mbelsky
Copy link

mbelsky commented Mar 25, 2020

codesandbox is flaky there, hence your odd numbers. Try it locally, it should work

I'll try, thank you!

@VWSCoronaDashboard8
Copy link

For anyone landing here using Typescript, it is generally advised to avoid the use of null. It simplifies types, and once you try to stop using it you realize you really don't need it 95% of the time because the JS already gives you undefined to work with.

So here's a slight adaptation:

let uniqueId = 0;
const getUniqueId = () => uniqueId++;

export function useComponentId() {
  const idRef = useRef<number>();
  if (idRef.current === undefined) {
    idRef.current = getUniqueId();
  }
  return idRef.current;
}

@kmurph73
Copy link

kmurph73 commented Jan 13, 2021

Using this with Strict Mode will give you a different value on the 2nd render. I assume this is okay to use otherwise, and it's just some quirk of Strict Mode that produces differing values? I've used it in production w/o apparent issue...

Edit: looks like it's the result of having a harmless side effect: facebook/react#20826

Trying to move away from this Hook in order to be StrictMode compatible (despite it working 100% fine otherwise)... wish there was a React.useComponentName Hook...

@notthatnathan
Copy link

notthatnathan commented Feb 24, 2021

The useRef version works in my testing, but why not just set the initial value rather than checking undefined/null and assigning? Am I missing something here?

export function useComponentId() {
  const id = useRef(getUniqueId());
  return id.current;
}

(edited, forgot .current in the return)

@akomm
Copy link

akomm commented Feb 27, 2021

@notthatnathan it has actually been explained here.

@notthatnathan
Copy link

Oops, missed it. Thanks.

@notthatnathan
Copy link

FWIW, after generating the id (I used shortid()) as the initial value (as mentioned above), it doesn't change in my testing. I don't believe the undefined check and reassignment is necessary with refs.

Also, for test snapshot support, you'll want a predictable id, maybe using a different prop from the instance.

const id = useRef(process.env.NODE_ENV === 'test' ? `test-${title}` : shortid());

@akomm
Copy link

akomm commented Mar 2, 2021

FWIW, after generating the id (I used shortid()) as the initial value (as mentioned above), it doesn't change in my testing. I don't believe the undefined check and reassignment is necessary with refs.

Why is it not needed and how do the two things relate (I don't know your shortid implementation).

Also, for test snapshot support, you'll want a predictable id, maybe using a different prop from the instance.

const id = useRef(process.env.NODE_ENV === 'test' ? `test-${title}` : shortid());

You add test-related code in a component? And also you change behavior depending on how id is used the outcome might be different in test than prod.

@notthatnathan
Copy link

notthatnathan commented Mar 2, 2021

You add test-related code in a component? And also you change behavior depending on how id is used the outcome might be different in test than prod.

Doesn't change behavior, just changes the id. Otherwise every run of your test, you get snapshot updates (new id on mount), which defeats the purpose of snapshots.

Why is it not needed and how do the two things relate (I don't know your shortid implementation).

I'm not sure I understand the question. I'm just saying that this:

  const idRef = useRef(null);
  if (idRef.current === null) {
    idRef.current = getUniqueId()
  }
  return idRef.current;

and this

  const idRef = useRef(getUniqueId());
  return idRef.current;

both return a unique ID that doesn't change on re-render. Which makes sense, refs wouldn't be useful if the initial value changed. Log from the calling component to see what I mean.

(shortid is just an id-generating package my org uses)

updated, typo in the second code example

@akomm
Copy link

akomm commented Mar 4, 2021

You add test-related code in a component? And also you change behavior depending on how id is used the outcome might be different in test than prod.

Doesn't change behavior, just changes the id. Otherwise every run of your test, you get snapshot updates (new id on mount), which defeats the purpose of snapshots.

It does. You use different ID generation methods. If one of the different methods generate non-unique ID, it will work in one, but fail in the other or lead to different results, depending on how you use the generated ID.

Sorry. I don't get what you are talking about. You've just changed things replying to my initial question. Why is it null instead of undefined? Also you did not answer why the undefined check is not needed. I don't understand the problem.

@bluenote10
Copy link

bluenote10 commented Mar 6, 2021

In cases where I don't need globally unique IDs, but rather IDs unique per component I'm using this hook:

export function useIdGenerator(): () => number {
  const ref = useRef(0);

  function getId() {
    ref.current += 1;
    return ref.current;
  }

  return getId;
}

@meglio
Copy link

meglio commented Jul 9, 2021

On a side note, nanoId is considerably faster than shortId or uniqueId according to their benchmarks.

@Izhaki
Copy link

Izhaki commented Mar 16, 2022

There is an open issue for this on the react repo.

Here is a more robust implementation that also supports SSR.

@airtonix
Copy link

airtonix commented Apr 1, 2022

Also, for test snapshot support, you'll want a predictable id, maybe using a different prop from the instance.

You shouldn't be using snapshots at all. write proper tests.

@mbonaci
Copy link

mbonaci commented Jun 16, 2023

For anyone stumbling upon this in 2023 there's React.useId for this very purpose since React v18.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment