Reputation: 1827
I have the following code:
try {
phpDoc(vscode.window.activeTextEditor);
} catch (err) {
console.error(err);
vscode.window.showErrorMessage(err.message);
}
however err.message
gets the error Object is of type 'unknown'.ts(2571)
on err.
, but I cannot type the object in catch (err: Error)
.
What should I do?
Upvotes: 106
Views: 81364
Reputation: 811
You cannot write a specific annotation for the catch clause variable in typescript, this is because in javascript a catch clause will catch any exception that is thrown, not just exceptions of a specified type.
In typescript, if you want to catch just a specific type of exception, you have to catch whatever is thrown, check if it is the type of exception you want to handle, and if not, throw it again.
meaning: make sure the error that is thrown is an axios error first, before doing anything.
using type assertion
Use AxiosError to cast error
import { AxiosError } from 'axios';
try {
// do some api fetch
} catch (error) {
const err = error as AxiosError
// console.log(err.response?.data)
if (!err?.response) {
console.log("No Server Response");
} else if (err.response?.status === 400) {
console.log("Missing Username or Password");
} else {
console.log("Login Failed");
}
}
check if the error is an axios error first, before doing anything
import axios from "axios"
try {
// do something
} catch (err) {
// check if the error is an axios error
if (axios.isAxiosError(err)) {
// console.log(err.response?.data)
if (!err?.response) {
console.log("No Server Response");
} else if (err.response?.status === 400) {
console.log("Missing Username or Password");
} else {
console.log("Login Failed");
}
}
}
Upvotes: 9
Reputation: 887
If you don't want to change all your code after upgrading your TypeScript but are in strict mode, you can add the following compiler option after the strict
option to overwrite it, as was hinted in Oleg's answer:
tsconfig.json
{
"compilerOptions": {
[...]
"strict": true,
"useUnknownInCatchVariables": false,
[...]
},
},
}
"strict": true,
sets useUnknownInCatchVariables
to true, and then "useUnknownInCatchVariables": false,
overrides that and sets it back to false.
Upvotes: 68
Reputation: 18937
My TypeScript version is under 4.0, and I can't make it work again, then I created an auxiliar function to normalize the errors, like following:
interface INormalizedError {
/**
* Original error.
*/
err: unknown;
/**
* Is error instance?
*/
isError: boolean;
/**
* Error object.
*/
error?: Error;
/**
* Call stack.
*/
stack?: Error['stack'];
/**
* Error message.
*/
message: string;
toString(): string;
}
/**
* Normalize error.
*
* @param err Error instance.
* @returns Normalized error object.
*/
function normalizeError(err: unknown): Readonly<INormalizedError> {
const result: INormalizedError = {
err,
message: '',
isError: false,
toString() {
return this.message;
}
};
if (err instanceof Error) {
result.error = err;
result.message = err.message;
result.stack = err.stack;
result.isError = true;
result.toString = () => err.toString();
} else if (typeof err === 'string') {
result.error = new Error(err);
result.message = err;
result.stack = result.error.stack;
} else {
const aErr = err as any;
if (typeof err === 'object') {
result.message = aErr?.message ? aErr.message : String(aErr);
result.toString = () => {
const m = typeof err.toString === 'function' ? err.toString() : result.message;
return (m === '[object Object]') ? result.message : m;
};
} else if (typeof err === 'function') {
return normalizeError(err());
} else {
result.message = String(`[${typeof err}] ${aErr}`);
}
result.error = new Error(result.message);
result.stack = aErr?.stack ? aErr.stack : result.error.stack;
}
return result;
}
An usage example:
try {
phpDoc(vscode.window.activeTextEditor);
} catch (err) {
const e = normalizeError(err);
console.error(err);
vscode.window.showErrorMessage(e.message);
}
Upvotes: 2
Reputation: 10345
As a supplementary answer to CertainPerformance's one:
Up until TypeScript 4.0, the catch
clause bindings were set to any
thus allowing easy access to the message
property. This is unsafe because it is not guaranteed that what's thrown will be inheriting from the Error
prototype - it just happens that we don't throw anything but errors as best practice:
(() => {
try {
const myErr = { code: 42, reason: "the answer" };
throw myErr; //don't do that in real life
} catch(err) {
console.log(err.message); //undefined
}
})();
TypeScript 4.0 introduced an option for a safer catch
clause by allowing you to annotate the parameter as unknown
, forcing you to either do an explicit type assertion or, even better, to type guard (which makes the clause both compile-time and runtime-safe).
However, to avoid breaking most of the codebases out there, you had to explicitly opt-in for the new behavior:
(() => {
try {
throw new Error("ouch!");
} catch(err: unknown) {
console.log(err.message); //Object is of type 'unknown'
}
})();
TypeScript 4.4 introduced a new compiler option called useUnknownInCatchVariables
that makes this behavior mandatory. It is false
by default, but if you have the strict
option turned on (as you should), it is turned on which is most likely the reason why you got the error in the first place.
Upvotes: 101
Reputation: 370759
It's because anything can be thrown, hence unknown
.
const fn = () => {
throw 'foo';
};
try {
fn();
} catch(e) {
console.log(e);
console.log(e instanceof Error);
console.log(e === 'foo');
}
You'll need to check that the err
actually is an error to narrow it down before accessing the message
property.
try {
phpDoc(vscode.window.activeTextEditor);
} catch (err) {
console.error(err);
if (err instanceof Error) {
vscode.window.showErrorMessage(err.message);
} else {
// do something else with what was thrown, maybe?
// vscode.window.showErrorMessage(String(err));
}
}
Upvotes: 54