Discussion :

[tidus5]

Yop,

je bosse sur un module qui r�cup�re des infos sur todoist.com

je ne suis pas super � l'aise avec javascript :/

voici le module
https://github.com/cbrooker/MMM-Todoist
je l'ai modifi� au niveau du fetcher.js pour r�cup�rer plus d'infos (priority, datedue...)
je concat�ne mes variables et je push ensuite dans l'affichage - mon probl�me se situe au niveau du tri (j'aimerais tri� par date ou priorit�)
et au niveau de l'affichage (j'aimerais une icone differente par rapport � la Priorit�) mais comment g�rer ca ? je ne comprends pas comment je pourrais g�rer le contenu de ma variable dans MMM-Todoist.js

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
 
if (JSON.parse(body).items[i].priority == '1') {icontodo ='P1 - '}
		else if (JSON.parse(body).items[i].priority == '2') {icontodo ='P2 - '}
		else if (JSON.parse(body).items[i].priority == '3') {icontodo ='P3 - '}
		else {icontodo ='P4 - '};
 
if (JSON.parse(body).items[i].due_date_utc) 
			{
				date1 = new Date();
				date2 = new Date(JSON.parse(body).items[i].due_date_utc);
				diff = dateDiff(date1, date2);
 
				if(diff.day > 0)
					{ 
						dayretour = diff.day + "d ";
					}
				else 
					{
						dayretour = '';
					};
				if(diff.hour > 0)
					{
 
						hourretour = diff.hour + "h";
					}
				else 
					{
						hourretour = '';
					};
				if(diff.hour > 0)
					{
 
						minretour = diff.min + "min";
					}
				else 
					{
						minretour = '';
					};	
 
				if (!dayretour && !hourretour && !minretour) {datedueretour ='TIME IS UP - ';} 		
				else {datedueretour += dayretour + hourretour + minretour + " - ";};
		}
		else {datedueretour =''};
 
		contentretour = JSON.parse(body).items[i].content;
 
		if (JSON.parse(body).items[i].responsible_uid ) {
		if (JSON.parse(body).items[i].responsible_uid == '102xxxx') {
		assignedtodoname ='T - '
		}
 
		else if (JSON.parse(body).items[i].responsible_uid == '102xxxx') {assignedtodoname ='N - '}
		else {assignedtodoname ='Inconnu'};
		} else { assignedtodoname ='' };
 
		//checkedretour = JSON.parse(body).items[i].checked;
		onbalance +=assignedtodoname + icontodo + datedueretour + contentretour;
		items.push(onbalance);

merci merciiii si vous savez me mettre sur une piste

[sekaijin]

J'avoue que j'ai beaucoup de mal � comprendre ton algo

mais il y a au moins une chose � faire c'est d'utiliser une variable pour le r�sultat du parsing
j'ai compt� 10 appels � JSON.parse(body) en clair tu relis et re parse le body 10 fois alors que si tu mets le r�sultat du parsing dans une variable tu ne le fais qu'1 fois.

pour le reste j'ai pas compris ce que �a fait.
A+JYT

[Watilin]

Si j�ai bien compris il y a une boucle for autour de tout �a, que tu ne nous as pas montr�e.
Pour appuyer la suggestion de sekaijin d�utiliser une variable afin d��viter les appels multiples � JSON.parse, je te sugg�re une structure comme ceci :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
var jsonBody = JSON.parse(body);
 
for ( … ) { // la boucle for que tu as déjà
  var currentItem = jsonBody.item[i];
  icontodo = "P" + currentItem.priority + " - ";
 
  if (currentItem.due_date_utc) {
    date1 = new Date();
    date2 = new Date(currentItem.due_date_utc);
    …
  }
 
  … etc.
}

On ne voit pas non plus la d�claration des variables, en particulier onbalance. J�attire ton attention sur le fait que tu y ajoutes des donn�es � chaque it�ration, avec l�op�rateur +=, sans jamais la r�initialiser. � chaque it�ration tu te retrouves donc avec les nouvelles donn�es mais aussi les anciennes, ce qui fait une sorte de pyramide, introduisant de la redondance dans ton tableau.

Un exemple hypoth�tique pour illustrer mon histoire de pyramide :

Code exemple :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
itération 0 --> "abc"
itération 1 --> "abcdef"  (au lieu de "def")
itération 2 --> "abcdefghi"  (au lieu de "ghi")
...

Cela dit, je te sugg�re de pusher un objet litt�ral dans ton tableau, plut�t qu�une cha�ne. Pour trier �a sera bien plus pratique (sinon tu devras faire des substring et �a va �tre super compliqu�).

Fais quelque chose dans ce genre :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
  items.push({
    assignedtodoname : assignedtodoname,
    icontodo         : icontodo,
    contentretour    : contentretour,
    datedueretour    : datedueretour
  });

�a te permettra d�exploiter la magie de la m�thode sort en lui passant une fonction en param�tre :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
items.sort(function (itemA, itemB) {
  if (itemA.datedueretour > itemB.datedueretour) return 1;
  if (itemA.datedueretour < itemB.datedueretour) return -1;
  return 0;
});

Car oui, les op�rateurs de comparaison marchent aussi avec les cha�nes. Toutefois tu peux utiliser des types de variables plus pratiques dans tes objets litt�raux, par exemple faire en sorte que datedueretour soit de type Date et, dans la fonction pass�e � sort, utiliser ta fonction dateDiff.