[S&C]

~~Calcolatrice System~~
Versione 1.1!!!

Informazioni: in game usa /calcolatrice per i comandi..


BUG CONOSCIUTI:
Nessuno

Download:
http://pastebin.com/d8U77MuD


(E figo il comando /addmolsotdiv vero?? xD)

Messaggio modificato da S&C il 25 giugno 2011 - 12:27

[Peppinux aka Peppe_Stasu]

Alcuni consigli per migliorare lo script .

1.

// This is a comment
// uncomment the line below if you want to write a filterscript
//#define FILTERSCRIPT
 

Inutile..
2.

#include <dcmd>

Perché? C'è già il define..

3.Penso che definire

new numero, numero1, numero2, numero3, numero4, numerofinale;

si potevano definire benissimo nei comandi ma non è un problema.

4.Capisco che il comando /calcolatrice non usa dei params ma dopo che hai usato il dcmd per tutto perché non usarlo anche per quello?

5.Penso che /addmolsotdiv sia abbastanza inutile LOLOLOLOL ma se hai avuto lo sfizio di farlo cancellarlo non ha senso..

6.Il dcmd è un sistema ormai vecchio e obsoleto e sarebbe quindi consigliabile usare nuovi sistemi come zcmd o ancora meglio y_commands.

7.La funzione sscanf come la usi? Non c'è né una funzione nel Filterscript né un include . Forse quell'include del dcmd doveva essere per l'sscanf2.

8.Tutti gli altri callbacks sono inutile. Appesantiscono solo la GM..

Spero che con questi consigli tu possa migliorare lo script .

[Phanto90]

Alcune osservazioni:

-Rimuovi i callbacks vuoti.. occupano righe inutili
-Utilizzi sscanf e non è incluso...
-Il dividi inoltre è sbagliato, una divisione spesso prevede cifre decimali dopo la virgola %d è intero.
-Perchè hai messo un limite a 999999 come numero? A questo punto metto -1000000 o qualsiasi altro numero maggiore di -999999 per superare il limite
-gli else non credo nemmeno vadano bene messi in quel modo...
-dcmd è obsoleto e se usi sscanf puoi farne anche a meno
-addmolsotdiv l'ho trovato abbastanza hm superfluo.
-Bug conosciuti: Parecchi D: Come hai fatto a testarlo se nemmeno puoi compilarlo :/

Cerca di correggere al più presto!

Messaggio modificato da Phanto90 il 25 giugno 2011 - 11:42

[S&C]

Corretti, Nuovo Download: http://pastebin.com/d8U77MuD Grazie

[Rapgangsta]

e poi puoi fare più corto tipo

     format(string,sizeof(string),"%d - %d = %d", numero, numero1, (numero - numero1));
     SendClientMessage(playerid, BLU, string);