[Résolu] Délimiteurs JS • Forum • Zeste de Savoir

nevgupobo, dimanche 25 août 2019 à 14h33

Bonjour !

J’essaie de prendre le contenu d’une balise script sur la page https://www.funcraft.net/fr/joueurs?q=pseudo, jusque là aucun soucis, c’est relativement simple, il suffit de faire :

router.get('/api/funcraft/rush/:username', async (req, res) => {
  let usernameParameter = req.params.username

  let dom = await JSDOM.fromURL(`https://www.funcraft.net/fr/joueurs?q=${usernameParameter}`)
  console.log(dom.window.document.querySelectorAll('script'))
  function getPage(user) {
    return JSDOM.fromURL(`https://www.funcraft.net/fr/joueurs?q=${user}`)
  }

  async function getUser(user) {
    const dom = await getPage(user)

    let funcraftPgames = dom.window.document.body.querySelectorAll('script')[6].textContent
    res.send(funcraftPgames)
  }

  getUser(usernameParameter)

})

Du coup j’obtiens https://hasteb.in/epomohic.swift

Cependant, j’aimerais juste prendre le contenu des variables, pas le nom (donc pgames, pstats et pstatsTotal

Le truc c’est que je ne sais pas comment faire pour délimiter, en gros faire en sorte que ça commence après var pgames = et que ça termine à la fin de cette variable, et ce pour toutes les variables Si je fais pas ça, je ne pourrai pas JSON.parse car il sera invalide.

Je pourrai très bien faire comme ça : https://github.com/Nevgup/funcraft-api/blob/master/api/routes/index.js Mais le problème c’est que c’est au caractère près, et du coup si une valeur est plus grande chez le joueur A que le joueur B, alors le JSON sera pas valide dans les deux cas

J’espère avoir été clair :v

Merci d’avance

25/08/19 à 14h33

+0 -0

anonyme, dimanche 25 août 2019 à 14h38

Tu peux scoper ton code :

pgames = (function() {
  // JSDOM.fromURL(`https://www.funcraft.net/fr/joueurs?q=${user}`)

  return pgames;
})();

25/08/19 à 14h38

+0 -0

nevgupobo, dimanche 25 août 2019 à 14h44

Tu peux scoper ton code :

pgames = (function() {
  // JSDOM.fromURL(`https://www.funcraft.net/fr/joueurs?q=${user}`)

  return pgames;
})();

A-312

Non car du coup pgames est undefined là C’est pas mon site, c’est celui de https://funcraft.fr

25/08/19 à 14h44

+0 -0

anonyme, dimanche 25 août 2019 à 15h01

J’imagine qu'une Regex ferait le taf sans trop se prendre la tête :

/var\s(.+?)\s?=\s?(.+?);/gi.exec("var test = \"string\";");
> Array(3) [ "var test = \"string\";", "test", "\"string\"" ]

25/08/19 à 15h01

+0 -0

ache, dimanche 25 août 2019 à 15h23
Modifié

Tu peux scoper ton code :
pgames = (function() {
  // JSDOM.fromURL(`https://www.funcraft.net/fr/joueurs?q=${user}`)

  return pgames;
})();
A-312
Non car du coup pgames est undefined là C’est pas mon site, c’est celui de https://funcraft.fr

nevgupobo

Je pense que l’idée de @A-312, c’est d’executer le code avec JSDOM.
C’est ce que j’aurais fais je pense. Même si niveau sécurité, c’est pas fou.

@TAlone: Pas super claire ta regex. Ça ne gère pas les variables justement il me semble.

Tu peux également chercher à transformer ce code en JSON valide. var => {, \s = => "\s": et tu rajoutes une accolade fermante à la fin }.

25/08/19 à 15h23
Modifié

ache.one 🦹 👾 🦊

+0 -0

anonyme, dimanche 25 août 2019 à 16h00

S’il a besoin juste de récupérer les variables statiques, plutôt que d’exécuter du code unsafe, faire tourner une Regex qui va récupérer le contenu est bien plus sûr.

25/08/19 à 16h00

+0 -0

nevgupobo, dimanche 25 août 2019 à 18h10

Tu peux scoper ton code :
pgames = (function() {
  // JSDOM.fromURL(`https://www.funcraft.net/fr/joueurs?q=${user}`)

  return pgames;
})();
A-312
Non car du coup pgames est undefined là C’est pas mon site, c’est celui de https://funcraft.fr
nevgupobo
Je pense que l’idée de @A-312, c’est d’executer le code avec JSDOM.
C’est ce que j’aurais fais je pense. Même si niveau sécurité, c’est pas fou.

@TAlone: Pas super claire ta regex. Ça ne gère pas les variables justement il me semble.

Tu peux également chercher à transformer ce code en JSON valide. var => {, \s = => "\s": et tu rajoutes une accolade fermante à la fin }.

ache

J’ai pas trop compris comment vous voulez que je fasse

S’il a besoin juste de récupérer les variables statiques, plutôt que d’exécuter du code unsafe, faire tourner une Regex qui va récupérer le contenu est bien plus sûr.

TAlone J’ai juste besoin de récupérer ces trois variables

25/08/19 à 18h10

+0 -0

ache, dimanche 25 août 2019 à 18h17

J’ai pas trop compris comment vous voulez que je fasse

nevgupobo

Que ne comprends tu pas ?

Si tu utilises JSDOM, c’est pour exécuter le script non ?

JSDOM

25/08/19 à 18h17

ache.one 🦹 👾 🦊

+0 -0

nevgupobo, dimanche 25 août 2019 à 18h27

J’ai pas trop compris comment vous voulez que je fasse

nevgupobo

Que ne comprends tu pas ?

Si tu utilises JSDOM, c’est pour exécuter le script non ?

JSDOM

ache

je récupère le script*

25/08/19 à 18h27

+0 -0

anonyme, dimanche 25 août 2019 à 22h03

Je pensais à si tu peux faire confiance au site/résultat et selon le risque acceptable.

var machin = JSDOM.fromURL(`https://www.funcraft.net/fr/joueurs?q=${user}`);
res.send(`
pgames = (function() {
  ${machin}

  return pgames;
})();
`);

Sinon pour du code unsafe, je ferais :

let str = $('textarea').val();

str = str.replace(/(var |,)([A-z]*)= (\{|\[)/g, '$1"$2": $3')

str = str.replace(/^var /, '')

str = JSON.stringify(JSON.parse(`{${str}}`), null, 2)

$('textarea').val(str)

Exemple ici : https://jsfiddle.net/13nzoL58/

25/08/19 à 22h03

+0 -0

nevgupobo, dimanche 25 août 2019 à 23h18

Je pensais à si tu peux faire confiance au site/résultat et selon le risque acceptable.
var machin = JSDOM.fromURL(`https://www.funcraft.net/fr/joueurs?q=${user}`);
res.send(`
pgames = (function() {
  ${machin}

  return pgames;
})();
`);
Sinon pour du code unsafe, je ferais :
let str = $('textarea').val();

str = str.replace(/(var |,)([A-z]*)= (\{|\[)/g, '$1"$2": $3')

str = str.replace(/^var /, '')

str = JSON.stringify(JSON.parse(`{${str}}`), null, 2)

$('textarea').val(str)
Exemple ici : https://jsfiddle.net/13nzoL58/

A-312

Je comprends pas le ${machin} et j’aimerais ne pas devoir utiliser jquery :v

25/08/19 à 23h18

+0 -0

anonyme, dimanche 25 août 2019 à 23h26

C’est ton retour que tu obtiens avec ta fonction. Je ne peux pas te répondre précisément, je ne sais pas le retour que tu souhaites avoir

et j’aimerais ne pas devoir utiliser jquery :v

Tant mieux, ce qui t’intéresse c’est :

str = str.replace(/(var |,)([A-z]*)= (\{|\[)/g, '$1"$2": $3')

str = str.replace(/^var /, '')

str = JSON.stringify(JSON.parse(`{${str}}`), null, 2)

25/08/19 à 23h26

+1 -0

nevgupobo, dimanche 25 août 2019 à 23h47
Modifié

C’est ton retour que tu obtiens avec ta fonction. Je ne peux pas te répondre précisément, je ne sais pas le retour que tu souhaites avoir

et j’aimerais ne pas devoir utiliser jquery :v

Tant mieux, ce qui t’intéresse c’est :
str = str.replace(/(var |,)([A-z]*)= (\{|\[)/g, '$1"$2": $3')

str = str.replace(/^var /, '')

str = JSON.stringify(JSON.parse(`{${str}}`), null, 2)

A-312

  funcraftPGames = dom.window.document.body.querySelectorAll('script')[6].textContent
  funcraftPGames = funcraftPGames.replace(/(var |,)([A-z]*)= (\{|\[)/g, '$1"$2": $3')
  funcraftPGames = funcraftPGames.replace(/^var /, '')

J’obtiens ça donc c’est plutôt bon mais je comprends pas pourquoi le : (position 8) pose pb :/ https://hasteb.in/opegunut.swift

25/08/19 à 23h47
Modifié

+0 -0

anonyme, lundi 26 août 2019 à 00h10

Tu as oublié : `{${str}}`

En gros au lieu d’avoir : {"a":1} tu as "a":1

26/08/19 à 00h10

+1 -0

nevgupobo, lundi 26 août 2019 à 00h32
Modifié

Tu as oublié : `{${str}}`

En gros au lieu d’avoir : {"a":1} tu as "a":1

A-312

yes en effet, par contre c’est normal que j’arrive pas à me get les objets alors que c’est du JSON valide ?

Edit: Autant pour moi, j’avais fait

    let funcraftAPI;

    funcraftAPI = dom.window.document.body.querySelectorAll('script')[6].textContent
    funcraftAPI = funcraftAPI.replace(/(var |,)([A-z]*)= (\{|\[)/g, '$1"$2": $3')
    funcraftAPI = funcraftAPI.replace(/^var /, '')
    funcraftAPI = JSON.stringify(JSON.parse(`{${funcraftAPI}}`), null, 2)

    res.send(JSON.parse(funcraftAPI.games))

au lieu de :

    let funcraftAPI;

    funcraftAPI = dom.window.document.body.querySelectorAll('script')[6].textContent
    funcraftAPI = funcraftAPI.replace(/(var |,)([A-z]*)= (\{|\[)/g, '$1"$2": $3')
    funcraftAPI = funcraftAPI.replace(/^var /, '')
    funcraftAPI = JSON.stringify(JSON.parse(`{${funcraftAPI}}`), null, 2)

    res.send(JSON.parse(funcraftAPI).pgames)

Merci à vous, problème résolu !

26/08/19 à 00h32
Modifié

+0 -0

anonyme, lundi 26 août 2019 à 03h58

Et regarde aussi pour utiliser try {} catch (e) {} pour retourner une erreur. C’est indispensable quand le json n’est pas fiable à 100% Dans le catch, tu peux envoyer une page d’erreur.

https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Instructions/try…catch

26/08/19 à 03h58

+1 -0

nevgupobo, lundi 26 août 2019 à 11h59

Yes c’est noté

26/08/19 à 11h59

+0 -0

Délimiteurs JS

Pas encore membre ?