Mon JavaScript plante parfois sur cette ligne:
var json = eval('(' + this.responseText + ')');
Les plantages se produisent lorsque l'argument de eval()
n'est pas JSON. Est-il possible de vérifier si la chaîne est au format JSON avant de passer cet appel?
Je ne veux pas utiliser de framework - y a-t-il un moyen de faire en sorte que cela fonctionne simplement avec eval()
? (Il y a une bonne raison, promis.)
Si vous incluez le analyseur JSON de json.org, vous pouvez utiliser la fonction parse () et l'envelopper dans un try/catch, comme ceci:
try
{
var json = JSON.parse(this.responseText);
}
catch(e)
{
alert('invalid json');
}
Quelque chose comme ça ferait probablement ce que vous voulez.
C'est l'alternative jQuery ...
try
{
var jsonObject = jQuery.parseJSON(yourJsonString);
}
catch(e)
{
// handle error
}
Je vous recommande fortement d'utiliser une bibliothèque javascript JSON pour la sérialisation vers et à partir de JSON. eval()
est un risque pour la sécurité qui ne doit jamais être utilisé à moins que vous ne soyez absolument certain que son entrée est désinfectée et sûre.
Avec une bibliothèque JSON en place, encapsulez simplement l'appel à son équivalent parse()
dans un bloc try/catch pour gérer les entrées non JSON:
try
{
var jsonObject = JSON.parse(yourJsonString);
}
catch(e)
{
// handle error
}
Promise
au lieu de Try-catch
:npm install is-json-promise ; //for NodeJS environment.
OU
String.IsJSON = (candidate) =>
new Promise(
(resolve, reject) => resolve(JSON.parse(candidate))
)
;
String.IsJSON(`iam here`)
.then((object) => console.info(object))
.catch((error) => alert('Waww, i cannot be JSON')) ; // promise will run catch
ou
String.IsJSON(`{"welcome":"Hello"}`)
.then((object) => console.info(object)) // promise will run "then"
.catch((error) => alert('Waww, i cannot be JSON')) ;
Cela aide peut-être: Avec ce code, vous pouvez obtenir directement vos données…
<!DOCTYPE html>
<html>
<body>
<h3>Open console, please, to view result!</h3>
<p id="demo"></p>
<script>
var tryJSON = function (test) {
try {
JSON.parse(test);
}
catch(err) {
// maybe you need to escape this… (or not)
test = '"'+test.replace(/\\?"/g,'\\"')+'"';
}
eval('test = '+test);
console.debug('Try json:', test);
};
// test with string…
var test = 'bonjour "mister"';
tryJSON(test);
// test with JSON…
var test = '{"fr-FR": "<p>Ceci est un texte en français !</p>","en-GB": "<p>And here, a text in english!</p>","nl-NL": "","es-ES": ""}';
tryJSON(test);
</script>
</body>
</html>
Pourquoi vous ne pouvez pas simplement vérifier quelle est la réponse? C'est plus efficace.
var result;
if (response.headers['Content-Type'] === 'application/json')
result = JSON.parse(this.responseText);
else
result = this.responseText;
Ci-dessous une fonction, vous pouvez essayer:
String.prototype.isJson = function () {
try {
JSON.parse(this.toString());
return true;
} catch (ex) {
return false;
}
};
Le problème avec l'approche try-catch
est que JSON.parse('123') = 123
et qu'il ne lève pas d'exception. Par conséquent, en plus du try-catch
, nous devons vérifier le type comme suit:
function isJsonStr(str) {
var parsedStr = str;
try {
parsedStr = JSON.parse(str);
} catch (e) {
return false;
}
return typeof parsedStr == 'object'
}
Il existe une petite bibliothèque qui vérifie les types de JavaScript: is.js
is.json({foo: 'bar'});
=> true
// functions are returning as false
is.json(toString);
=> false
is.not.json([]);
=> true
is.all.json({}, 1);
=> false
is.any.json({}, 2);
=> true
// 'all' and 'any' interfaces can also take array parameter
is.all.json([{}, {foo: 'bar'}]);
=> true
En fait is.js est bien plus que cela, quelques mentions honorables:
var obj = document.createElement('div');
is.domNode(obj);
=> true
is.error(new Error());
=> true
is.function(toString);
=> true
is.chrome();
=> true if current browser is chrome