Adding Function (Funda) for #170

[thecodacus]

This PR closes issue #154 and Based on #170, this will be the first stage to make it more like a real programming language.

added Functional Programming features,

you can now create funda (functions) to make your code modularized, and not repeat yourself

example

hi bhai
apna funda test(c){
  bol bhai c;
  rakh le bhai "return bhi "+c;
}
bol bhai test("kam kiya bhai");
bye bhai

output

kam kiya bhai
return bhi kam kiya bhai

[netlify]

✅ Deploy Preview for bhailang-preview ready!

Name	Link
🔨 Latest commit	a2b0d83
🔍 Latest deploy log	https://app.netlify.com/sites/bhailang-preview/deploys/63848e08619000000829e06f
😎 Deploy Preview	https://deploy-preview-220--bhailang-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[lgtm-com]

This pull request introduces 6 alerts when merging 74b5630 into fc11311 - view on LGTM.com

new alerts:

• 2 for Illegal invocation
• 2 for Unneeded defensive code
• 1 for Useless conditional
• 1 for Comparison between inconvertible types

[lgtm-com]

This pull request introduces 4 alerts when merging 774d9c1 into fc11311 - view on LGTM.com

new alerts:

• 2 for Unneeded defensive code
• 1 for Useless conditional
• 1 for Comparison between inconvertible types

[lgtm-com]

This pull request introduces 3 alerts when merging fc93884 into fc11311 - view on LGTM.com

new alerts:

• 2 for Unneeded defensive code
• 1 for Comparison between inconvertible types

[lgtm-com]

This pull request introduces 2 alerts when merging 0ffb32e into fc11311 - view on LGTM.com

new alerts:

• 1 for Comparison between inconvertible types
• 1 for Unneeded defensive code

[ankitchouhan1020]

Great work! will check.

[ankitchouhan1020]

hi bhai

apna funda Counter() { 
  bhai ye hai count = 1;
  
  apan funda increment() {
    count += 1;
    rakh le bhai count;
  }
  
  rakh le bhai increment;
}

bhai ye hai tick = Counter();
bol bhai tick();

bye bhai

Does it support closures ? Not working in preview.

[thecodacus]

hi bhai

apna funda Counter() { 
  bhai ye hai count = 1;
  
  apan funda increment() {
    count += 1;
    rakh le bhai count;
  }
  
  rakh le bhai increment;
}

bhai ye hai tick = Counter();
bol bhai tick();

bye bhai

Does it support closures ? Not working in preview.

you have a typo "apan" the keyword is "apna"
also I just fixed few bugs, it should be working now

[lgtm-com]

This pull request introduces 2 alerts when merging 3336243 into 0181ca4 - view on LGTM.com

new alerts:

• 1 for Comparison between inconvertible types
• 1 for Unneeded defensive code

[lgtm-com]

This pull request introduces 2 alerts when merging 3dc5b92 into 0181ca4 - view on LGTM.com

new alerts:

• 1 for Comparison between inconvertible types
• 1 for Unneeded defensive code

[tbhaxor]

Great work @thecodacus.

I will look into this. I was thinking to have duck typing in the functions. But this will also work

[tbhaxor]

Shouldnt we have it outside hi bhai? Like other languages? Hi bhai is the execution entry point

[shankarkrupa]

@ankitchouhan1020 @aniketsingh0104 or @ristri - could one of you review and approve the PR to be merged with the develop branch? This is such a cool and important feature. Eager to see this spreading against after playing around with this a bit in the netlify preview playground.

[aniketsingh0104]

To be reviewed this week.

[aniketsingh0104]

Critical: I see test cases are missing from this change specially from Parser. We won't review any PRs which do not contain proper test cases. Please write proper test cases covering positive and negative scenarios for both PARSER and INTERPRETER.

[aniketsingh0104]

Complete Review would be only possible after the test cases are included.

[thecodacus]

Complete Review would be only possible after the test cases are included.

Thanks for the review till now, I will make the changes and add test cases

[lgtm-com]

This pull request introduces 2 alerts when merging 567b961 into de68714 - view on LGTM.com

new alerts:

• 1 for Comparison between inconvertible types
• 1 for Unneeded defensive code

[thecodacus]

Critical: I see test cases are missing from this change specially from Parser. We won't review any PRs which do not contain proper test cases. Please write proper test cases covering positive and negative scenarios for both PARSER and INTERPRETER.

test cases added

[thecodacus]

took me a while to get the time and make the changes :)
@aniketsingh0104 please check the changes

[thecodacus]

changes implemented

[lgtm-com]

This pull request introduces 2 alerts when merging a2b0d83 into de68714 - view on LGTM.com

new alerts:

• 1 for Comparison between inconvertible types
• 1 for Unneeded defensive code

---

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[Moulick]

Can this be merged, looks very useful bhai