🚀 go-pugleaf

MrokU pisze:
> To może tak:
> wkleję tutaj kod, który mi działa, ale zapewne wiele rzeczy jest
> nieoptymalnie, niefajnie, etc.
> Czy ktoś może rzucić na to okiem i powiedzieć co i jak robić lepiej?
> pozdrawiam.

[...]

> ;******	Procedura testowa ***** dodawanie
> 
> sumuj PROC a: DWORD , b: DWORD
> 	pushad
> 	mov eax, a
> 	add b, eax
> 	popad
> 	mov eax, b
> 	ret            ;koniec funkcji
> sumuj ENDP

 Możesz nie pisać PUSHAD i POPAD, bo używasz tylko rejestru EAX, który
nie tylko można dowolnie zmieniać, ale przy wyjściu zawiera wynik.
Podobnie w kilku innych procedurach.

> ;******	Procedura testowa ***** HARDCORE - sinus()
> 
> sinus PROC kat:DQ;, wynik:PTR
> 	pushad
> 	finit
> 	fld	kat			; wczytanie zmiennej do rejestru st0
> 	fsin
> 	fst kat
> 	popad
> 	mov eax, kat
> 	ret            ;koniec funkcji
> sinus ENDP

 Oj, tu niedobrze. Pomijając PUSHAD/POPAD, wykonywanie FINIT może być
dość kosztowne czasowo i raczej powinno się to robić tylko raz, przy
starcie programu (chyba że naprawdę potrzebny jest reset). Większy
problem jest w tym, że wkładasz coś na stos i nie zdejmujesz - po 8
uruchomieniach się przepełni (pewnie dlatego robiłeś FINIT). Zamiast
FST użyj FSTP. Nawet jeśli nigdzie indziej nie używasz koprocesora,
rób tak dla zasady.
 Swoją drogą, chyba "wynik" jest od tego, by otrzymać wynik, a nie "kat"?


> #include <windows.h>
> #include <stdio.h>
> #include <iostream>
> #define PI
> 3.1415926535897932384626433832795028841971693993751058209749445923078164062862089986280348253421170679

 Czemu nie użyjesz stałych "firmowych" (jeśli są dostępne, oczywiście)?

#include <math.h>
#ifndef M_PI
#define M_PI 3.141592653589793238462643383279502884197 blablabla...
#endif


> //---------------------------------------------------------------------------
> 
> typedef int (__stdcall *MYPROC)(int, int);
> typedef float (__stdcall *MYPROC2)(float);
> 
> MYPROC ProcAdd, ProcSub;
> MYPROC2 ProcSin;
> HINSTANCE hinstLib;
> int a, b;
> float d;
> 
> int main(int argc, char* argv[])
> {
> 	hinstLib = LoadLibrary ("DLLint.dll");	//ladowanie biblioteki
> 	if (hinstLib == NULL)			//sprawdzenie, czy moduł faktycznie
> został załadowany
> 		return 1;
> 
> 	ProcAdd = (MYPROC) GetProcAddress(hinstLib, "sumuj");
> 	if (NULL != ProcAdd)
> 		a = (ProcAdd)(9,2);

 Formalnie to pewnie powinno być

		a = (*ProcAdd)(9,2);

 jako że "ProcAdd" jest wskaźnikiem. Ale jak działa, to dobrze.

[...]

> 		d = (ProcSin)(PI / 6);	// sinus 30 stopni

 W tym błędu nie ma, ale jak będziesz potrzebował szybkiego działania,
używaj stałych, bez obliczeń, np.

#define M_PI_6 0.5235987755982988527 /* M_PI / 6 */
		d = (*ProcSin)(M_PI_6);

 aby od razu przekazywać parametr do funkcji, bez zbędnych obliczeń na
stałych (a na przykład dzielenie jest akurat kosztowne).
 Pewnie mądry kompilator i tak by tu wstawił stałą, ale co jeśli nie?

 Tyle ja zauważyłem na pierwszy rzut oka. Może ktoś jeszcze coś dołoży.

-- 
Pozdrawiam/Regards - Bogdan                     (GNU/Linux & FreeDOS)
Kurs asemblera x86 (DOS, GNU/Linux):http://rudy.mif.pg.gda.pl/~bogdro
Grupy dyskusyjne o asm:  pl.comp.lang.asm alt.pl.asm alt.pl.asm.win32
www.JabberPL.org www.TorProject.org Soft (EN): miniurl.pl/bogdro-soft