CMPUT301F13T02 / ChooseYourAdventure

BSD 3-Clause "New" or "Revised" License
1 stars 7 forks source link

Random decision crashes the app #248

Open klindenbach opened 10 years ago

klindenbach commented 10 years ago

Per demo

jcadek commented 10 years ago

Explain what "per demo" Means? no error output? which branch? this gives me nothing to go off of

klindenbach commented 10 years ago

Sorry, that was the best I could say. We gave a demo to Haiming yesterday, and the app crashed when attempting to add a random decision

jcadek commented 10 years ago

Which Branch? We knew that the non-refactored branch had this problem

KarlParkinson commented 10 years ago

FATAL EXCEPTION: main java.lang.NullPointerException at ca.ualberta.CMPUT301F13T02.chooseyouradventure.PageController.findArrayPosition(PageController.java:250) at ca.ualberta.CMPUT301F13T02.chooseyouradventure.DecisionView.onEditDecisionGUI(DecisionView.java:50) at ca.ualberta.CMPUT301F13T02.chooseyouradventure.DecisionView$4.onClick(DecisionView.java:253) at com.android.internal.app.AlertController$AlertParams$3.onItemClick(AlertController.java:941) at android.widget.AdapterView.performItemClick(AdapterView.java:298) at android.widget.AbsListView.performItemClick(AbsListView.java:1100) at android.widget.AbsListView$PerformClick.run(AbsListView.java:2788) at android.widget.AbsListView$1.run(AbsListView.java:3463) at android.os.Handler.handleCallback(Handler.java:730) at android.os.Handler.dispatchMessage(Handler.java:92) at android.os.Looper.loop(Looper.java:137) at android.app.ActivityThread.main(ActivityThread.java:5103) at java.lang.reflect.Method.invokeNative(Native Method) at java.lang.reflect.Method.invoke(Method.java:525) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:737) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553) at dalvik.system.NativeStart.main(Native Method)

This was from the refactored code.

jcadek commented 10 years ago

Still can't replicate this.