11521 sujets

JavaScript, DOM et API Web HTML5

Conseil - système d'estimation en Javascript

Bonjour à tous,
J'ai récemment réalisé un système d’estimation. C'est la première fois que je mettai les mains dans du js (et jquery). Le système ""fonctionne"" mais niveau maintenabilité c'est une horreur. Je suis partie tête baissée, ou plutôt je découvrai le fonctionnement petit à petit et donc pas réfléchis à l'organisation. Du coup en 2eme version je souhaite reprendre tout cela est faire quelque chose qui respire la simplicité. Voici un lien vers ce système
Je suis donc à la recherche de conseils pour la structuration de ce genre de projet.
Merci
Salut

else if($(".radioOriginal:checked").val() == 'pb'){
				$('.radioDef').bind('click', function () {
					$.ajax({
						url: "https://www.plurielcom.com/estimations/tableSupport",
						data: {
							definition: $(this).val()
						},
						dataType: 'html',
						type: 'post',
						success: function (html) {
							// $("#tableAmount").html('<img src="http://www.mediaforma.com/sdz/jquery/ajax-loader.gif">');/*a changer*/
							$("#tableSupport").html(html);
						}
					})
				});
				return
			}else if($(".radioOriginal:checked").val() == 'fichierImg'){
				$('.radioDef').bind('click', function () {
					$.ajax({
						url: "https://www.plurielcom.com/estimations/tableSupport",
						data: {
							definition: $(this).val()
						},
						dataType: 'html',
						type: 'post',
						success: function (html) {
							// $("#tableAmount").html('<img src="http://www.mediaforma.com/sdz/jquery/ajax-loader.gif">');/*a changer*/
							$("#tableSupport").html(html);
						}
					})
				});
				return


Tu as donc plusieurs appel comme ça (pour le moment je regarde que le début de ton fichier js)
Quel est la différence entre les deux requêtes ajax ? je comprend pas à quoi sert tous ces if else if
Modifié par JENCAL (27 Nov 2018 - 09:43)
Pareil pour ça 
else if($(".radioOriginal:checked").val() == 'diapos'){
				return
			}else if($(".radioOriginal:checked").val() == 'otherImg'){
				return
			}else if($(".radioOriginal:checked").val() == 'k7audio'){
				return
			}else if($(".radioOriginal:checked").val() == 'vinyle'){
				return
			}else if($(".radioOriginal:checked").val() == 'miniDisc'){
				return
			}else if($(".radioOriginal:checked").val() == 'bandeAudio'){
				return
			}else if($(".radioOriginal:checked").val() == 'carteMemoire'){
				return
			}else if($(".radioOriginal:checked").val() == 'disqueDur'){
				return
			}else if($(".radioOriginal:checked").val() == 'disqueDurMeca'){
				return


Tu as, en jquery, la possibilité de regarder si un valeur de tableau existe avec la méthod inArray() donc tu peux mettre tous tes valeurs carteMemoire, disqueDur, disqueDurMeca etc.. dans un tableau et faire un seul if un peu comme ça 

var myarray= [ "carteMemoire" , "disqueDur", "disqueDurMeca" ];
if(jQuery.inArray($(".radioOriginal:checked").val(), myarray) !== -1)
{
  // oui, val() est bien dans myarray;
}
else
{
  // non il ne l'ai pas.
}

c'est un exemple bien sûr.
Bonjour,
Merci à toi pour cette réponse. Ce message juste pour m'excuser de ce silence, mais la fin d'année a déplacé les priorités...
Je reviendrai donc lancé comme un frelon dès que du temps ce dégage pour faire la misère à mon code (si on peut appeler ca comme ca Smiley smile )
Merci encore à toi Jencal
Sujet clos
Aller à