bonjour, voici mon soucis
 
je dois faire une requête insert dans une base access. Soucis, elle ne fonctionne pas. Ce qui est surprenant, c'est que VB ne me donne aucun message d'erreur, je constate kiste après coup que la table reste désespérément vide.
malheureusement celle-ci est très longue, la table comportant 130 champs.  
 
voici la requête, juste pour vous donner un aperçu    

 
ma question : quelles sont les bonnes méthodes de travail lorsqu'il s'agit de gérer des requetes de telles longueurs ? c'est très long à écrire, et encore plus chiant à débugguer quand il y a un problème comme dans mon cas.
 
Merci d'avance.

Bonjour,
 
J'ai déjà fait des requêtes plus longues que celle-là.
Donc, je ne crois pas que ce soit ça qui pose le problème.
Il faudrait débugguer en récupérant cette longue chaine, par exemple avec un MsgBox(sql), et en la testant comme étant une requête autonome.
Une autre manière de débugguer serait de faire des essais avec une chaine dont on aurait enlevé certains morceaux (par exemple les dates et heures qui posent souvent des problèmes).

salut,
 
j'affiche évidemment la chaine dans une msgbox, et à sa lecture, je ne vois rien qui cloche...
 
je tente de la récupérer dans le presse papier la chaine avec l'outil Clipboard, mais ça ne marche pas. Je souhaiterais la tester directement dans Access, qui fournit des messages d'erreurs plus explicites.

il faudrait prendre l'habitude de ne pas débugger à la MessageBox, mais de logger dans un fichier texte les messages d'erreur...

@Harkonnen
Pourquoi ne pas utiliser MsgBox si ça peut rendre service de temps en temps ?
(Et ce n'est pas MessageBox quand on utilise le VB, vous n'avez pas dû passer autant de centaines d'heures que moi sur le VB.)  
 
S'il vous plait, que vous soyez modérateur, ou non, arrêtez de critiquer les interventions de ceux qui répondent !
Cela donne une mauvaise ambiance à ce forum, comme cela a déjà été signalé plusieurs fois.
On n'est pas dans une cour de récréation, où les gros bras font la loi. On a le droit de s'exprimer pour donner une solution, même si ce n'est peut-être pas la solution optimum.
Dans d'autres sections de ce forum (par exemple Windows, Mobiles, etc), il y a beaucoup moins de personnes qui critiquent les autres, et l'ambiance y est plus agréable.
 
Quoi qu'il en soit, pour ma part, je continuerai d'essayer d'aider tous ceux qui posent des questions, malgré les critiques.
 
@clements42
C'est vrai qu'il est très dommage que MsgBox ne permette pas le copier/coller.
 
Mais déjà, cela montre la chaîne s'affiche entièrement, qu'il n'y a pas la limite sur la longueur à laquelle vous pensiez.
 
Regardez bien, c'est obligatoirement un petit problème à l'intérieur de cette chaîne de caractères.
 
Ou bien, tester en prenant une chaine moins longue. Vous verrez qu'un qu'une petite chaîne passe. Puis, vous ajoutez quelques champs, etc. jusqu'à finir par trouver quel morceau de la requête pose un problème.

wowowow, du calme hein, mon intervention n'était nullement ici pour critiquer
Je dis que débugger à la MessageBox (fonction native de Win32, qui est appelée par le MsgBox du VB) est très mal parce que :
- on en prend vite l'habitude, et du coup on en met partout. Et après on a le client qui nous appelle en disant "c'est quoi ce message bizarroïde qui apparaît quand je clique sur mon joli bouton ?". Tu l'auras compris, à force de mettre des MessageBox, on a vite fait d'en laisser une ou deux en production. Alors que le fichier de trace est bien plus discret et surtout non intrusif pour le client.
- les infos de la MessageBox ne sont pas pérennes : elle disparaît une fois qu'on a cliqué sur OK, et pour peu qu'on ait cliqué trop rapidement ou par erreur sans lire le message, c'est tout le processus de debug qui est à refaire. Et si le bug est difficilement reproductible, c'est autant de temps perdu pour récupérer des infos qui seraient figées dans le marbre avec une trace fichier.
- pour récupérer les infos de la MessageBox, c'est soit par capture d'écran, soit par recopie manuelle du message. Pas de copier/coller possible. Vachement pratique .
- la MessageBox est modale. Elle prend directement le focus une fois qu'elle apparait, et on ne remarque pas forcément de suite qu'elle est là. Donc si tu as continué à utiliser ton clavier pendant que la MessageBox était à l'écran, t'es bon pour tout retaper.
 
Bref le debug à la MessageBox est tout, sauf pro. A éviter autant que possible. Surtout avec la facilité de mise en place de fichiers de trace.

Bon, alors deux propositions pour réduire ton code :
1) il semble que tu vas chercher des valeur dans Excel. Alors au lieu de mettre cells(x,y), qui qui est lourd et rallonge le code, ne peux-tu pas donner directement un nom explicit à tes cellules dans Excel ?
2) tu utilises sans compter la fonction Replace(Cells(x, y), ",", "." ), ne peux-tu pas faire une fonction dédiée avec un nom plus simple et qui te donnerait comme code appellant : Rpl(MaCellule), MaCellule étant le nom que tu as donné à ta cellule dans Excel.

Et Harkonnen ne fait peut etre pas de VB mais il a raison. On ne passe pas par un msgbox pour debuguer. On utilise debug.print, là on récupere notre chaine, on peut copier/coller etc. Et comme son nom l'indique, c'est fait pour. Et c'est la meme chose qu'un trace.
 
clements42, si tu nous montrais ton tableau Excel, je suis sur que d'une ca serait beaucoup plus parlant, et de deux... hum... beaucoup plus parlant
Ensuite, comme le souligne otobox tu utilises la fonction Replace qui ne sert au final qu'a remplacer une virgule par un point. Je ne vois pas à quoi ca sert si tu as définis la virgule comme separateur décimal. Si tel est le cas, Excel ou Access même combat et pas besoin de faire tout ca...
Ensuite tu fais des Int() sur tout ou presque. Es-tu sur que tes cellules contiennent des nombres ? Mais meme souci, je ne vois pas à quoi ca sert etant donné que tu fais un replace() du separateur décimal avant...
 
Edit : ensuite pour simplifier la requete, je ferais tout simplement une boucle qui boucle ( ) sur le numéro de ligne et le numéro de colonne.
Ensuite tu parles de 130 champs, alors que tes données sources ne sont que sur 4 colonnes...

Sans compter que si une conversion virgule->point est nécessaire, elle pourrait être lancée sur tous les champs avant de faire la requête, ainsi plus la peine d'en tenir compte pour la suite.