Запуск с параметрами

[Xatr]

Нужно прогу загрузить с параметрами:
dvd.exe open
dvd.exe close
Но только не работает. пробовал через Message, вроде слово показывает (open или close).

Пример проги:

.486
.model flat,stdcall
option casemap:none
include \masm32\include\masm32rt.inc
include \masm32\include\winmm.inc
includelib \masm32\lib\winmm.lib
.const
a db 'Set cdaudio door open wait',0
b db 'Set cdaudio door closed wait',0
.code

; главная процедура
main proc
LOCAL file1:dword
LOCAL buffer1[20]:BYTE
mov file1,ptr$(buffer1)
invoke GetCL,1,file1
push file1
pop eax
.if eax==6F70656Eh ;open
call open
.endif
.if eax==636Ch ;close
call close
.endif
ret
main endp

;открыть
open proc
push 0
push 0
push 0
push offset a
call mciSendString
push 0
call ExitProcess
open endp

;закрыть
close proc
push 0
push 0
push 0
push offset b
call mciSendString
push 0
call ExitProcess
close endp


start:
call main
push 0
call ExitProcess
end start

[litrovith]

Xatr, вместо '.if eax==6F70656Eh ;open' нуна '.if eax==6E65706Fh ;nepo' вроде

[rpy3uH]

функция GetCommandLine возвращает путь к нашему экзешнику плюс все параметры. Путь к экзешнику и параметр заключены в кавычки. Осталось только пропарсить строки и отделить путь к программе и сам параметр

[litrovith]

rpy3uH, та не, ТС же пользует готовую функцию масма.
Xatr,

LOCAL buffer1[20]:BYTE
mov file1,ptr$(buffer1)

- ето лишнее, нуна:[

invoke GetCL,1, addr buffer1

(буфер нуна побольше), потом

lea eax, buffer1

в результате в eax вы должны получить указатель на буфер!

[Xatr]

А нельзя ли это всё сделать без процедур?

[rpy3uH]

без каких процедур?

[Xatr]

"main proc"
"open proc"
"close proc"

при помощи lea, тоже работает, но дальше условия не проходит
Изменённый код:

.486
.model flat,stdcall
option casemap:none
include \masm32\include\kernel32.inc
includelib \masm32\lib\kernel32.lib
include \masm32\include\winmm.inc
includelib \masm32\lib\winmm.lib
include \masm32\include\masm32.inc
includelib \masm32\lib\masm32.lib
.const
a db 'Set cdaudio door open wait',0
b db 'Set cdaudio door closed wait',0
.code
main proc
LOCAL buffer1[1]:BYTE
invoke GetCL,1, addr buffer1
lea eax,buffer1

cmp ax,31h  ;open
jne ex1
push 0
push 0
push 0
push offset a
call mciSendString
push 0
call ExitProcess

ex1:
cmp eax,32h  ;close
jne ex
push 0
push 0
push 0
push offset b
call mciSendString
push 0
call ExitProcess

ret
main endp

start:
call main
ex:
push 0
call ExitProcess
end start

dvd open
dvd close
для удобства были изменены на
dvd 1
dvd 2
соответственно

[Ulex]

Ну так и правильно всё.
lea eax,buffer1 - берём адрес строки в eax
cmp ax,31h ;open - сравниваете младшие 16 бит адреса со значением 0031h, а надо сравнивать значение по этому адресу.

.486
.model flat,stdcall
option casemap:none
include \masm32\include\kernel32.inc
includelib \masm32\lib\kernel32.lib
include \masm32\include\winmm.inc
includelib \masm32\lib\winmm.lib
include \masm32\include\masm32.inc
includelib \masm32\lib\masm32.lib
.const
a db 'Set cdaudio door open wait',0
b db 'Set cdaudio door closed wait',0
.code
main proc
LOCAL buffer1[16]:BYTE

invoke GetCL,1, addr buffer1
 
lea eax,buffer1
cmp byte ptr [eax],31h  ;open
jne ex1
push 0
push 0
push 0
push offset a
call mciSendString
push 0
call ExitProcess

ex1:
cmp byte ptr [eax],32h  ;close
jne ex
push 0
push 0
push 0
push offset b
call mciSendString
push 0
call ExitProcess

ret
main endp

start:
call main
ex:
push 0
call ExitProcess
end start

[Xatr]

ай спасибо, работает!
После:

cmp byte ptr[eax],32h
jne f
push 0
push 0
push 0
push offset b
call mciSendString
ret

добавил:

include \masm32\macros\macros.asm
f:
print "Use: dvd option"
print chr$(13,10,13,10)
print "Options:"
print chr$(13,10)
print "o - open DVD-ROM"
print chr$(13,10)
print "c - close DVD-ROM"
ret

Как это можно оптимизировать?
В Tasm когда-то писал:
mov ah,"номер вывода строки на экран"
mov dx,offset mes
int 21h

В masm что-то не проходит.

[Ulex]

А я бы вам не советовал использовать все эти масм-ие макросы и функции типа print и StdOut. Почему??

1. Кому как, а мне например код вида:
.data
xxxx
.code
xxxx
.data
xxxx
.code
глаза как то режет. Не, ну конечно здорово, что masm32 такой замечательный компилятор, что во всей этой каше разберётся и данные попадут всё таки в секцию данных. Но лично я такого бардака в своей программе не потерпел бы, а макрос print именно к такому виду вашу программу и приведёт.
2. Использование сторонних функций (StdOut) скорее всего запутает начинающего программиста и вы можете даже не понять, что вы делаете. Поэтому советую для начала поработать с консолью Win32 чистыми WinApi функциями (GetStdHandle, CloseHandle и т.д.).