Discussion :

[laurenthino]

Bonjour à tous,

Alors voilà, j'ai un petit problème avec mon code. En effet, je demande à mon programme de vérifier qu'une valeur existe dans ma base de donnée excel, dans la colonne A. Si elle existe, je dois ouvrir une message box, pour dire, "cette donnée existe déjà, souhaitez vous quand même la copier?", j'ai réussi toutes ces étapes, sauf que le programme me sort tout le temps cette alerte même si la donnée n'est pas encore présente dans la base.

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
59
60
61
62
63
64
65
66
67
68
69
70
71
Sub Open_MW()
 
 
Dim MW_number As Variant
Dim wb2 As Workbook
Dim ligne As Long
Dim message As String, Answer As Variant
 
ligne = Range("A1").End(xlDown).Row + 1
 
 
MW_number = InputBox("Please enter the MW number:")
 
'look if the MW is already in the database
With Sheets("Database_MW").Range("A1:A10000")
 
Set c = .find(what:=MW, After:=Range("A2"), LookIn:=xlValues, _
        LookAt:=xlWhole, SearchOrder:=xlByRows, SearchDirection:=xlNext, _
        MatchCase:=True, SearchFormat:=False)
 If c Is Nothing Then 'if the margin worksheet is not in the database
     Set wb2 = Workbooks.Open("C:\Users\Jules.Matuvu\Desktop\temp file\MW" & MW_number) '& ".xls"
        Windows("MW" & MW_number & ".xlsx").Activate
        Range("B2:B7").Select
        Selection.Copy
        Windows("New project.xlsm").Activate
        Sheets("Database_MW").Cells(ligne, 1).Activate
        Selection.PasteSpecial Paste:=xlPasteAll, Operation:=xlNone, SkipBlanks:= _
        False, Transpose:=True
 
        Windows("MW" & MW_number & ".xlsx").Activate
        Range("B11:B15").Select
        Selection.Copy
 
        Windows("New project.xlsm").Activate
        Sheets("Database_MW").Cells(ligne, 7).Activate
        Selection.PasteSpecial Paste:=xlPasteAll, Operation:=xlNone, SkipBlanks:= _
        False, Transpose:=True
 
 
    Else
       message = "MW is already in the database, do you still want to put it?"
    Answer = MsgBox(message, vbYesNo)
    Select Case Answer 'two choices, do you still want to upload the information
        Case vbYes 'if yes, it opens and copy paste
            Set wb2 = Workbooks.Open("C:\Users\Jules.Matuvu\Desktop\temp file\MW" & MW_number) '& ".xls"
            Windows("MW" & MW_number & ".xlsx").Activate
            Range("B2:B7").Select
            Selection.Copy
            Windows("New project.xlsm").Activate
            Sheets("Database_MW").Cells(ligne, 1).Activate
            Selection.PasteSpecial Paste:=xlPasteAll, Operation:=xlNone, SkipBlanks:= _
            False, Transpose:=True
 
            Windows("MW" & MW_number & ".xlsx").Activate
            Range("B11:B15").Select
            Selection.Copy
 
            Windows("New project.xlsm").Activate
            Sheets("Database_MW").Cells(ligne, 7).Activate
            Selection.PasteSpecial Paste:=xlPasteAll, Operation:=xlNone, SkipBlanks:= _
            False, Transpose:=True
 
        Case vbNo 'if not, it exit sub
       Exit Sub
        End Select
 
 End If
 End With
 
 
End Sub

Je suppose que c'est une erreur de placement, mais je n'arrive pas à trouver.

Qqn aurait-il une idée?

Merci de votre aide
Laurent

[mat955]

Bonjour laurenthino,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
Set c = .find(what:=MW, After:=Range("A2"), LookIn:=xlValues, _
        LookAt:=xlWhole, SearchOrder:=xlByRows, SearchDirection:=xlNext, _
        MatchCase:=True, SearchFormat:=False)

Je n'ai pas testé, mais il me semble que tu cherches ce qu'il y a dans la variable MW alors que plus haut tu enregistres la valeur de l'inputbox dans une variable nommée MW_number.

[laurenthino]

Bonjour laurenthino,



Je n'ai pas testé, mais il me semble que tu cherches ce qu'il y a dans la variable MW alors que plus haut tu enregistres la valeur de l'inputbox dans une variable nommée MW_number.

Hello Mat,

Honte à moi Je te remercie. Ca m'apprendra à vouloir renommer mes variables en cours de route. Je te remercie!