[Release] OS-Shop

[thepsyca]

Bonjour à vous,

Je vous présente mon nouveau script : Os-Shop.

Présentation :

L'addon est un shop assez basique, avec un derma propre.
Si celui-si vous plait n'hésitez pas à me le faire savoir et à le partager.
Si vous voulez de nouvelles fonctionnalités dessus, dites le moi.

L'addon fonctionne uniquement avec DarkRP et ses dérivés !

VIDEO EXEMPLE

Fonctionnalités :

- Ajout d'item en vente I-G
- Modification d'item I-G
- Système de language pré-définis
- Braquage des vendeurs !
- Restriction d'item par métier !
- Un derma assez sombre pour ne pas faire mal au yeux !

Il ne requiert que un seul et unique content, celui là.

Installation :

Décompiler l'addon, glisser le fichier garrysmod/addon.

Pour la configuration lua/autorun/nlf_shopconfig.lua

Pour la configuration des langues lua/autorun/nlf_shoplanguage.lua

Vous pouvez modifier l'addon, mais il est interdit de le réupload, partager ou vendre !

N'oubliez pas d'ajouter l'addon de content à votre collection pour ne pas avoir de soucis de content

Lien de téléchargement :
GitHub
Workshop
Workshop - Content

 

[Malcolm]

Un release propre encore une fois Par contre c'est "your" et pas "you'r" et "Item amended" tu peux mettre "Item edited" ?

 

[thepsyca]

Un release propre encore une fois Par contre c'est "your" et pas "you'r" et "Item amended" tu peux mettre "Item edited" ?

Yep après la c'est la version anglais, tu as une version française aussi.
Je ferais la modification tout de même

 

[Firyo89]

Le derma me trouble un peu surtout la porte pour quitter le menu, j'aurai plus mis un croix en haut à droite

 

[thepsyca]

Le derma me trouble un peu surtout la porte pour quitter le menu, j'aurai plus mis un croix en haut à droite

Je voulais sortir de l'ordinaire on va dire

 

[Makss]

Pas mal mais parce que j'aime critiquer gratuitement :
- Pour le nom de la commande dans le chat tu peux mettre quelque chose de plus unique pour éviter le conflit entre 2 commandes, !shopadditem est assez générique : n'importe quelle autre addon pourrait l'utiliser, tandis que !oshopadd est plus complexe

- Comme dit précédemment c'est troublant de ne pas avoir une simple croix en haut à droite, de plus le style que tu as utilisé pour agencer la fenêtre "Add item" est pas très esthétique et fais plus "gros bordel où j'ai mis tout ce que y'avais à mettre" -- totalement subjectif

- Quelque chose de super intéressant : virer le bandeau noir à droite des DTextEntry avec SetDrawLanguageID

Maintenant parlons du code :
- Les majuscules n'ont jamais tué personnes, c'est bien plus lisible de mettre comme nom teamForRob que teamforrob

- Il y a une limite de 2048 network string, il est donc préférable d'en faire un seul puis d'envoyer une variable agissant comme switch, en envoyant par exemple un UInt de longueur 4 ce qui permet de faire comme s'il y'avais 16 network string différent. Si tu veux en savoir plus sur ça :

Spoiler: Oui je le veux

Unsigned int est un entier non négatif, en informatique les nombres sont stockés sous forme binaire, une longueur de 4 indique à Gmod qu'il doit écrire un entier non négatif sur 4 bits ( exemple pour le nombre 5: 0101 ). La différence entre un nombre non signé (unsigned) et un nombre 'normal' c'est que la moitié des possibilités sont utilisé pour faire les nombres négatifs, avec 4 bits le nombre de possibilité est 4 puissance 2 soit 16 donc si l'on prends en Int les nombres iront de -8 à 7 tandis qu'avec Uint ils iront de 0 à 15 inclus.
cf quelqu'un qui sait mieux expliquer moi

- Optimisation SQL certes on pourrait s'en foutre car "c'est pas important" ou "y'a pas grande différence" sauf qu'un peu de ceci puis un peu de cela puis encore un peu de l'autre ça finis par faire une corbeille de cette taille

Spoiler: Damn son

- Il faudrait vraiment que j'applaudisse la personne qui a dit " les noms des hooks et network string seront séparés par :: " je vois ça partout et j'ai aucune idée pourquoi

- Optimise la quantité de données envoyé, un Uint(16) a la même longueur (en positif) qu'un Int(32) tout en étant 2 fois plus petit

- Indente mieux ton code, ceci est beaucoup moins lisible

Spoiler: Ceci

if osshop.antispam then
if pl.timedelay == nil then
pl.timedelay = 0 end
if CurTime() < pl.timedelay then DarkRP.notify(pl, 1, 4, osshop.lang[kla].txt31) return end
pl.timedelay = CurTime() + 5
end
if pl:GetPos():DistToSqr(npc:GetPos())>200 then

Sérieusement, si tu fais des crises d'épilepsies en écrivant ton code vaut mieux attendre que ça passe

que cela

Spoiler: Cela

 if osshop.antispam then if pl.timedelay == nil then pl.timedelay = 0 end if CurTime() < pl.timedelay then DarkRP.notify(pl, 1, 4, osshop.lang[kla].txt31) return end pl.timedelay = CurTime() + 5 end if pl:GetPos():DistToSqr(npc:GetPos()) > 200 then

- Chose intéressante, si une variable n'est pas égal à nil ou false ( peut-être aussi avec 0 ) elle est TOUJOURS vrai, donc tu peux résumer " if tavar == nil then " en " if not tavar then " ou si tu as la même syntaxe que moi " if !tavar then "

- Acheter des oeufs

- DistToSqr est une valeur TRES grande, un DistToSqr(var) > 200 équivaut à Distance(var) > 14 c'est donc presque toujours vrai ( pis quelle est l'utilité de vérifier qu'une personne est assez loin du NPC ? )

- Totalement subjectivement je préfère ply à pl je comprends mieux que c'est un joueur

- Répare les injections SQL, et non je ne donnerais pas plus d'infos sur ce point

- C'est mieux d'utiliser OnPlayerChat que PlayerSay

- ScrH() * 1

- Les noms des variables c'est bien quand c'est explicite, local xp = vgui.Create("DPanel", CScrool) c'est mal

Oui j'ai dis que des choses négatives mais bon j'étais pas très chaud pour faire une liste de tout ce qui est bien c'est pas comme ça qu'on s'améliore

PS: Si ce commentaire est cité quand vous parlez de critiques constructives je vous suce serre chaleureusement la pince

 

[ZarosOVH]

- Il faudrait vraiment que j'applaudisse la personne qui a dit " les noms des hooks et network string seront séparés par :: " je vois ça partout et j'ai aucune idée pourquoi

C'est la faute de @slownls il a commencer à le faire puis c'est partie en 'omg je fait comme slown'

- Acheter des oeufs

Seulement le mardi.

 

[Makss]

C'est la faute de @slownls il a commencer à le faire puis c'est partie en 'omg je fait comme slown'

* applaudis @slownls *
* lui mets une tape sur l'épaule en susurrant à son oreille la mort vient du ciel *

Seulement le mardi.

Non mardi c'est pates, bref je ne mélange pas vie professionnelle et vie privé

 

[Fleodon]

Bon travail

 

[thepsyca]

Nouvelle MAJ :

Optimisation !
Meilleure indentation du code !
Ajout de restriction par métier suite à la demande Kitsu !
Correction de language suite à la demande de malcolm2003 !
Refonte de certains design ! 1 / 2
La commande de base pour l'ouverture du panel passe à : "!osshopadd"

En espérant que l'addon vous plait

Des idées de fonctionnalités ? Préviens moi sur le topic ou en privé !

Si vous utilisiez l'ancienne version de l'addon, merci d'utiliser la commande "osshop_reloadtable" dans votre console développeur I-G.

La MAJ arrivera sur le WorkShop dans quelques jours, pour être sur de ne pas avoir besoin d'en faire 400 suite à des soucis que je n'aurais pas vu.

 

[Kitsu]

Ajout de restriction par métier !

Merci moi :3

 

[MonsieurLeLama]

Gégé pour la maj

 

[MonsieurLeLama]

Mais je suis sur que Atila te le fait en 20 sec

 

[LaNouvelleFrance]

Bonjour à vous,

Je vous présente mon nouveau script : Os-Shop.

Présentation :

L'addon est un shop assez basique, avec un derma propre.
Si celui-si vous plait n'hésitez pas à me le faire savoir et à le partager.
Si vous voulez de nouvelles fonctionnalités dessus, dites le moi.

L'addon fonctionne uniquement avec DarkRP et ses dérivés !

VIDEO EXEMPLE

Fonctionnalités :

- Ajout d'item en vente I-G
- Modification d'item I-G
- Système de language pré-définis
- Braquage des vendeurs !
- Restriction d'item par métier !
- Un derma assez sombre pour ne pas faire mal au yeux !

Il ne requiert que un seul et unique content, celui là.

Installation :

Décompiler l'addon, glisser le fichier garrysmod/addon.

Pour la configuration lua/autorun/nlf_shopconfig.lua

Pour la configuration des langues lua/autorun/nlf_shoplanguage.lua



N'oubliez pas d'ajouter l'addon de content à votre collection pour ne pas avoir de soucis de content

Lien de téléchargement :
GitHub
Workshop
Workshop - Content

Salut, tout abbord belle addon, puis comment mettre plusieur shop? merci =)

 

[thepsyca]

Salut, tout abbord belle addon, puis comment mettre plusieur shop? merci =)

Bonsoir, il n’est pas possible de faire différents shop sur cet addon

 

[LaNouvelleFrance]

Bonsoir, il n’est pas possible de faire différents shop sur cet addon

ah dommage, et ou puije deactiver les braquage? (Merci d’être si rapide

 

[thepsyca]

ah dommage, et ou puije deactiver les braquage? (Merci d’être si rapide

Ah j’ai rien mis pour le désactiver je viens de voir

 

[LaNouvelleFrance]

Ah j’ai rien mis pour le désactiver je viens de voir

ok bon pg je vais essayer de chercher si ya pas moyen

 

[Offshorp]

J'aime bien, c'est du bon travail

 

[Dolzen]

Salut quand je tape la commande rien ne s'ouvre je n'arrive donc pas a jouter des items. Merci et bonne soirée

 

[NoaGamingFR]

Salut quand je tape la commande rien ne s'ouvre je n'arrive donc pas a jouter des items. Merci et bonne soirée

bruh bruh, tu as des erreurs ? Créer plutôt un sujet ^^'