Found unused variables in TypeScript code JS-0356
Performance
Major
5 days ago
—
a year old
'color' is assigned a value but never used
4
5export const Tooltip = {
6 utility1: (theme) => {
7 const { color, ...fontProps } = theme.other.utilityFonts.utility1 8 return {
9 tooltip: {
10 ...fontProps,'setTertiaryList' is assigned a value but never used
112 const { t } = useTranslation(['country', 'gov-dist'])
113 // const [primaryList, setPrimaryList] = useState<GeoList[] | undefined>()
114 const [secondaryList, setSecondaryList] = useState<GeoList['districts'] | undefined>()
115 const [tertiaryList, setTertiaryList] = useState<116 NonNullable<GeoList['districts']>[number]['subDistricts'] | undefined
117 >()
118 const [primarySearch, setPrimarySearch] = useState<string | null>(null)'setSecondaryList' is assigned a value but never used
111 // const { control } = useFormContext<FormSchema>()
112 const { t } = useTranslation(['country', 'gov-dist'])
113 // const [primaryList, setPrimaryList] = useState<GeoList[] | undefined>()
114 const [secondaryList, setSecondaryList] = useState<GeoList['districts'] | undefined>()115 const [tertiaryList, setTertiaryList] = useState<
116 NonNullable<GeoList['districts']>[number]['subDistricts'] | undefined
117 >()'requireLanguage' is assigned a value but never used
5
6export const SelectionItem = forwardRef<HTMLDivElement, SelectionItemProps>(
7 ({ icon, label, ...others }, ref) => {
8 const { requireBoolean, requireGeo, requireData, requireLanguage, requireText, ...props } = others 9 return (
10 <div ref={ref} {...props}>
11 <Group'requireData' is assigned a value but never used
5
6export const SelectionItem = forwardRef<HTMLDivElement, SelectionItemProps>(
7 ({ icon, label, ...others }, ref) => {
8 const { requireBoolean, requireGeo, requireData, requireLanguage, requireText, ...props } = others 9 return (
10 <div ref={ref} {...props}>
11 <Group
Description
Unused variables are generally considered a code smell and should be avoided.
Removing unused references - It prevents unused modules from being loaded at runtime, improving performance, and preventing the compiler from loading metadata that will never be used. - It prevents conflicts that may occur when trying to reference another variable.
NOTE: If you have intentionally left a variable unused, we suggest you to prefix the variable name with a _ to prevent them from being flagged by DeepSource.
Bad Practice
import fs from 'fs' // <- unused
import { readFileSync } from 'fs'
const text = readFileSync('declaration_of_independence.txt', 'utf-8')
console.log(text)
Recommended
import { readFileSync } from 'fs'
const text = readFileSync('declaration_of_independence.txt', 'utf-8')
console.log(text)