D12425: programmingMaze, code clean-up final revision
Aman Kumar Gupta
noreply at phabricator.kde.org
Sat Apr 21 21:50:34 UTC 2018
amankumargupta created this revision.
amankumargupta added a reviewer: GCompris: Improvements.
amankumargupta added a project: GCompris: Improvements.
Restricted Application added a subscriber: KDE Edu.
Restricted Application added a project: KDE Edu.
amankumargupta requested review of this revision.
REVISION SUMMARY
This diff mainly focuses on generalizing the object creation, destroying and accessing process of the instruction objects while executing the user's code.
Previously, the object creation and destroying code for both didn't follow the same process/behaviour. In this diff, the code have been refactored to have same behaviour and the other changes which are very important for the extension for **programmingMaze loops* activity.
Now we maintain two look-up tables for main and procedure respectively and we create the instructions for each at the starting of the levels and they can be accessed to execute the instruction when needed, and destroyed at the end of the level (which was previously done only for procedure).
This is certainly better, more readible, and most importantly having the code to follow the same behaviour for both.
REPOSITORY
R2 GCompris
REVISION DETAIL
https://phabricator.kde.org/D12425
AFFECTED FILES
src/activities/programmingMaze/Dataset.qml
src/activities/programmingMaze/ProgrammingMaze.qml
src/activities/programmingMaze/instructions/Procedure.qml
src/activities/programmingMaze/programmingMaze.js
To: amankumargupta, #gcompris_improvements
Cc: #kde_edu, #gcompris_improvements, himanshuvishwakarma, harrymecwan, ganeshredcobra, nityanandkumar, echarruau, rahulyadav, narvaez, scagarwal, apol, timotheegiet, hkaelberer, jjazeix, bcoudoin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180421/67b589ed/attachment-0001.html>
More information about the kde-edu
mailing list