Discussion :

[dj_f.]

Bonjour tout le monde,
j'ai un problème qui se résume ainsi :
j'ai un programme qui fait plusieurs milliers de lignes de code et j'aimerais l'optimiser, donc je pense qu'il serait judicieux de représenter ce programme ou d'en déduire l'algo (je me trompe ?), mais par contre je ne sais pas comment m'y prendre (graphiquement parlant, par exemple représenter "If aListeLignesEcritureComptable <> NIL" je ne sais pas faire ... ).
J'ai vraiment besoin de conseils.

Merci.

[ToTo13]

Bonjour,

en général il suffit de traduire en français les différentes lignes de codes.
Mais dans ton cas, tu as un peu trop de lignes => dans un premier temps, il te faut écrire des lignes en pseudo code qui résument des blocs entiers (aucuns détails), par exemple : trier le tableau (tu ne mets pas le détail), faire une dichothomie, ...
Lorsque tu auras tout ton programme écrit en pseudo code sur une cinquantaines de lignes, tu pourras réfléchir comment l'optimiser.
Ensuite, tu préciseras certaines parties astuces de calculs, heuristiques, ...).

[dj_f.]

Merci c'est pas mal comme idée j'y avais pensé mais y'a pas mal de lignes de codes !! Je m'y mets on verra bien ce que ça me donnera !

[dj_f.]

Je vais vous mettre une partie du code que je dois "transformer" en algo pour l'optimiser ensuite ...

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
					for aLignePiece in aLigne.GetListeLignePieceJournee() do
						aLignePieceTransfert = GCPT_PieceTransfertLigne(aWks.getObjectByOID(aLignePiece.getDetailPieceOID(),pClassName = GCPT_PieceTransfertLigneBase.CLID_GCPT_PIECETRANSFERTLIGNE, pMode= 3));
						If unHT_ListeLignesParDetailPieceOID.Find(key = aLignePieceTransfert.getDetailPieceOID().TextValue.value) = NIL Then
							unPieceALire	= new();
							unVSDK_OID		= new();
							unVSDK_OID . setOID(pOID = aLignePieceTransfert.getDetailPieceOID());
							unPieceALire . setPieceOID(pPieceOID = unVSDK_OID);
							unTabPieceALirePieceOID . appendrow(unPieceALire);
								If unHT_PieceEmargementDetailHorsEmargement.Find(key = aLignePieceTransfert.getDetailPieceOID().TextValue.value ) = NIL Then
									unHT_PieceEmargementDetailHorsEmargement.Enter(element = unPieceALire , key = aLignePieceTransfert.getDetailPieceOID().TextValue.value);
								End if;
							unHT_ListeLignesParDetailPieceOID.Enter(element=aLigne,key=aLignePieceTransfert.getDetailPieceOID().TextValue.value);
 
						End If;
						unHT_ListeLignesParDetailPieceOIDNonUnique.Enter(element=aLigne,key=aLignePieceTransfert.getDetailPieceOID().TextValue.value);
					end for;

[Jedai]

A ta place la première chose que je ferais serait d'assassiner le type qui m'a laissé ce code... (Si c'est toi c'est plus simple, tu t'as sous la main après tout)

J'ai personnellement un problème avec une variable qui s'appelle unHT_ListeLignesParDetailPieceOIDNonUnique... et les lignes de 400 caractères n'aident pas à la lisibilité non plus.

--
Jedaï

[souviron34]

J'ai personnellement un problème avec une variable qui s'appelle unHT_ListeLignesParDetailPieceOIDNonUnique... et les lignes de 400 caractères n'aident pas à la lisibilité non plus.

et c'est malheureusement une certaine tendance d'un certain nombre de gens et de methodologies

[ram-0000]

Peut être qu'avant d'essayer de transformer ce paquet de lignes de code en algo, tu pourrais tenter une double approche

1/ D'abord décrire en français ce que fait le programme, ensuite décrire les particularités que doit prendre en compte le programme.

2/ Une autre approche qui peut être faite simultanément est de regarder le code et de tenter de repèrer les lignes de code dupliquées, leur rôle et peut être d'en faire une fonction à part afin de fractionner le problème.

Je n'ai pas ton code sous les yeux mais parfois certains programmes ne peuvent pas être réduits sous forme d'algo simples (pour le commun des mortels) car ils manipulent des concepts complexes. Je pense par exemple aux algo MD5 ou RSA qui demandent de solides connaissances mathématiques pour les comprendre. Le plus simple que l'on sache faire dans ce cas est de dire que l'on implémente le code de l'algo décrit dans le RFC xxx.

Transformer "If aListeLignesEcritureComptable <> NIL" en français pourrais se dire en algo "Si le tableau des lignes d'écriture comptables est défini (ou non vide, je ne sais pas)"