Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Meaxy change #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Meaxy change #3

wants to merge 1 commit into from

Conversation

d0p1s4m4
Copy link
Member

No description provided.

@@ -57,6 +57,21 @@ end/

' on aurait pu faire
login_win_handle = window/ "Logon window" logon_pos_x ....
' J'aurais mis quelque chose plus à la C
login_win_handle = window(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça casse la cohérence du langage nah ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je trouve également qu'il y a un truc qui va pas. Pourquoi des parenthèses alors que la structure du langage dans les autres exemples convient ?

C'est bizarre je trouve 🤔.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En vrai non, car dans les Beta recentes une fonction similaire à été fait avec le /f: , c'est juste pour ne pas obliger l'utilisateur d'y aller avec la syntaxe direct et permettre d'y aller avec une fonction comme le /f: , j'avais vu cela pour la fenêtre dans les fichiers CPC

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alors tu as sûrement vu ceci (pour les fenêtres) dans la version Alpha, où effectivement il y avait des parenthèses :

ini/ fenetre(
 ini;nom = "appli"
 ini;titre = "Application test"
 ini;px = "60"
 ini;py = "40"
 ini;tx = "420"
 ini;ty = "263"
 ini;couleur = "255,255,255"
 ini;couleurt = "255,255,255"
 ini;couleurf = "255,255,255"
(ini;icone = "Le dossier et l'icone")
 creer/
ini/ fenetre) 

Après le design de cette syntaxe, je l'aime pas trop :-/

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non pas de ça, mais plutôt de https://github.com/SPinti-Software/CpcdosOS2.1/blob/dev/bin/KRNL/CONFIG/ENV_GUI/WINDOW.CPC ! La fonction gui.create_window(name, title, pos_x, pos_y, siz_x, siz_y, parameters, win_rgb, title_rgb, opacity) Je ne me suis pas basé de la Alpha.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah mais je pensais parce que ça ressemblait un peu à ça. J'avais pas compris l'idée de départ déso x)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

T'inquiète lol

Copy link
Contributor

@chrapati24 chrapati24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Des choses qui valent le coup d'œil mais il faudrait en parler avec d0p1 et keyboard pour savoir leur avis.

Personnellement tu as proposé une idée sympa avec ton include/ myFunc.cpc, mais par contre les parenthèses pour les fenêtres et autres objets je suis pas fan. A voir.


' Aussi pour les events Cpcdos utilise .event tout se qui est function dans un ficher externe et qui devrait etre appeller
include/ "MyFunc.CPC"
Copy link
Contributor

@chrapati24 chrapati24 Aug 27, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On a use/ pour les libs liés au CpcdosC+ (GUI, Network[?], etc.). Mais on a pas forcément pensé à ce qui est l'équivalent d'un header en C (genre #include "myFunction.h"). Mais je sais pas si ça compliquera l'apprentissage parce que quelque part use/ et include/ auront un peu la même fonctionnalité ? Je sais pas ce que vous en pensez les autres 🤔.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le use/ de se que j'ai vu dans le concept est d'utiliser les fonctions à la style import de python si je comprends bien mais le include/ permet d'inclure par exemple events.cpc au fichier app.cpc et d'executé les fonctions créé par le programmeur

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je pense qu'il faut trouver un truc sympas pour utiliser un seul mot clé sur le deux

un peu comme en Python

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi pas émettre deux possibilités
ici comme proposer
use/ time

Ensuite ici
use/ "MyFunc.cpc"

alors on garderais la même commande

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je pensais un trucs comme ça 😛
mais je crois qu'il a même pas besoin des quotes 🤔

car le .cpc indique un fichier Cpc

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mais je pense que sans ça peux aussi posé des problèmes avec la résolution de path (car CpcdosC+ utilise bcp de slash et que sur Posix les path separtors sont aussi des slash a voir avec @d0p1s4m4 🤔)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oui, c'est un peu pour ça que j'avais mis les quotes, vu que le classic CC+ c'est des slashs

Copy link
Contributor

@chrapati24 chrapati24 Sep 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En soi, l'idée de mettre des quotes ne me déplaît pas. Mais après les quotes = chaîne de caractères. Donc il faut trouver un compromis adéquat.

t'sais le mec qui poste son avis genre 1 mois plus tard lol

' Aussi pour les events Cpcdos utilise .event tout se qui est function dans un ficher externe et qui devrait etre appeller
include/ "MyFunc.CPC"
' Alors en mettant cela au debut on evite de reouvrir un fichier a chaque objet graphique
' Cela permetterais de mettre des variables dans un ficher CPC externe du programme principale, comme un "header"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En théorie ouais ! Mais maintenant il faudrait voir ce que ça donne au niveau pratique.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants