jluc2808 peu le savent et vous ? | bonjour voilà j'ai un code plutôt 'sale' et que je n'arrive pas à optimiser , merci pour des suggestions le but de ce code est de remplir après un ordre ajax une zone d'image qui est dans un autre ordre ajax (celui appelant) la première partie construit une ligne par item et réserve la partie image qui sera instanciée par un appel à la fonction get_Poster() la ligne construite se présente sous la forme :
Code :
- temp_output = "<a href=\"javascript:setId('" + filmography_list[i].type + "', '" + filmography_list[i].videoDataId + "');\"><img class=\"poster"+i+"\" src=\"\" height=\"60px\" width=\"40px\" ></img></a>" + temp_output ;
|
le get_poster(filmography_list[i].type, filmography_list[i].videoDataId, i); va chercher via un autre ordre ajax les infos et mets à jour la zone d'image réservée via updatePosterHtml(Poster_data, i);
Code :
- function updatePosterHtml(Posterdata, i)
- {
- // console.log(Posterdata);
- switch (i)
- {
- case 0:
- YAMJ_DIR0 = {
- // display poster
- 'img.poster0@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR0);
- break;
- case 1:
- YAMJ_DIR1 = {
- // display poster
- 'img.poster1@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR1);
- break;
- case 2:
- YAMJ_DIR2 = {
- // display poster
- 'img.poster2@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR2);
- break;
- case 3:
- YAMJ_DIR3 = {
- // display poster
- 'img.poster3@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR3);
- break;
- case 4:
- YAMJ_DIR4 = {
- // display poster
- 'img.poster4@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR4);
- break;
- case 5:
- YAMJ_DIR5 = {
- // display poster
- 'img.poster5@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR5);
- break;
- case 6:
- YAMJ_DIR6 = {
- // display poster
- 'img.poster6@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR6);
- break;
- case 7:
- YAMJ_DIR7 = {
- // display poster
- 'img.poster7@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR7);
- break;
- case 8:
- YAMJ_DIR8 = {
- // display poster
- 'img.poster8@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR8);
- break;
- case 9:
- YAMJ_DIR9 = {
- // display poster
- 'img.poster9@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR9);
- break;
- case 10:
- YAMJ_DIR10 = {
- // display poster
- 'img.poster10@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR10);
- break;
- case 11:
- YAMJ_DIR11 = {
- // display poster
- 'img.poster10@src': function(arg) {
- // Check to see if there are posters
- return arg.context.baseArtworkUrl + arg.context.result.artwork.POSTER[0].filename;
- }
- }
- $p('.result_filmography').render(Posterdata, YAMJ_DIR11);
- break;
- }
- }
|
le switch / case fonctionne mais je trouve cela pas beau et limite le nombre d'item qui pourra aller jusqu'à 50 |