Detected the use of variables before they are defined JS-0357
Anti-pattern
Major
3 days ago
—
a year old
'SelectedItemCount' was used before it was defined
272 <FilterDisplay>{t('more.options')}</FilterDisplay>
273 </Group>
274 {selectedItemCount > 0 ? (
275 <SelectedItemCount selectedItemCount={selectedItemCount} />276 ) : (
277 <Icon icon='carbon:chevron-down' height={24} />
278 )}'SelectedItemCount' was used before it was defined
253 <>
254 <Group spacing={8} noWrap>
255 <FilterDisplay>{t('more.options')}</FilterDisplay>
256 {selectedItemCount > 0 ? <SelectedItemCount selectedItemCount={selectedItemCount} /> : null}257 </Group>
258 {selectedItemCount > 0 ? (
259 <Link'SocialMediaDisplay' was used before it was defined
15import { type SocialMediaProps } from './types'
16
17export const SocialMedia = ({ edit, ...props }: SocialMediaProps) =>
18 edit ? <SocialMediaEdit {...props} /> : <SocialMediaDisplay {...props} /> 19
20const SocialMediaDisplay = ({ parentId = '', passedData, locationOnly }: SocialMediaProps) => {
21 const { data } = api.orgSocialMedia.forContactInfo.useQuery('SocialMediaEdit' was used before it was defined
15import { type SocialMediaProps } from './types'
16
17export const SocialMedia = ({ edit, ...props }: SocialMediaProps) =>
18 edit ? <SocialMediaEdit {...props} /> : <SocialMediaDisplay {...props} /> 19
20const SocialMediaDisplay = ({ parentId = '', passedData, locationOnly }: SocialMediaProps) => {
21 const { data } = api.orgSocialMedia.forContactInfo.useQuery('WebsitesEdit' was used before it was defined
17const anyTrue = (...args: boolean[]) => args.some((x) => x)
18
19export const Websites = ({ edit, ...props }: WebsitesProps) =>
20 edit ? <WebsitesEdit {...props} /> : <WebsitesDisplay {...props} /> 21
22const WebsitesDisplay = ({
23 parentId = '',
Description
Variables, functions and types should always be used after they've been defined. This issue will flag any code snippets that use variables or types before definition.
Sometimes, the code will run just fine even when the variable is declared after use. Consider the following example:
const ram: Resource = { type: "memory", limit: 1024 ** 3 }
type Resource = {
type: string;
limit: number;
}
Here, Resource is used in the annotation before it has been defined.
Similarly, it is possible to hoist function declarations and variables declared with the var keyword:
const four = twice(2);
function twice(n: number) {
return n * 2;
}
However, it makes the code harder to follow when variables or types are declared after being used.
Bad Practice
const knight: Radiant = {
order: "SurgeBinder"
strength: 30
}
interface Radiant {
order: string;
strength: number;
}
Recommended
interface Radiant {
order: string;
strength: number;
}
const knight: Radiant = {
order: "SurgeBinder"
strength: 30
}