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

Add Conditions #705

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add Conditions #705

wants to merge 2 commits into from

Conversation

amir-s
Copy link

@amir-s amir-s commented Jul 9, 2024

Because without conditionals, a programming language would be like a software engineer who only ever responds to emails with 'Yes'—great for positivity, terrible for project management.

@guigui0246
Copy link

It's missing the elif

@amir-s
Copy link
Author

amir-s commented Jul 9, 2024

It's missing the elif

Good catch! But I don't think elif should be part of it. elif is exactly file in reverse. To prove that, here is a little script:

when (elif.split("").reverse().join("") = file) {
    print(can not use elif)!
}

Adding that keyword could be confusing since elif could be used to reverse a file in the future versions, and DreamBerd should remain future compatible.

@guigui0246
Copy link

It's missing the elif

Good catch! But I don't think elif should be part of it. elif is exactly file in reverse. To prove that, here is a little script:

when (elif.split("").reverse().join("") = file) {
    print(can not use elif)!
}

Adding that keyword could be confusing since elif could be used to reverse a file in the future versions, and DreamBerd should remain future compatible.

Then it should be elifaton because it would make notafile indicating it's not the reverse of a file

@amir-s
Copy link
Author

amir-s commented Jul 11, 2024

@guigui0246 That actually makes perfect sense. I'll amend the commit and include the context in it as well.

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.

2 participants