From 7dd16892cb2cbef82ddca71889ab072c07465667 Mon Sep 17 00:00:00 2001
From: chrqls <charles.quelos@gmail.com>
Date: Tue, 17 Dec 2013 05:10:14 -0800
Subject: [PATCH] [#14] Removing 'Highest Rating' + Reparing '% correct'

---
 res/layout-large-land/students.xml            | 20 +++++++++---------
 .../smile/ui/fragment/StudentsFragment.java   | 21 ++++++++-----------
 2 files changed, 19 insertions(+), 22 deletions(-)

diff --git a/res/layout-large-land/students.xml b/res/layout-large-land/students.xml
index 2b95c59..e194d00 100644
--- a/res/layout-large-land/students.xml
+++ b/res/layout-large-land/students.xml
@@ -250,16 +250,16 @@
             android:layout_marginTop="5dip"
             android:layout_below="@+id/view_separator_above" />
 
-        <TextView
-            android:id="@+id/tv_top_scorers_rating"
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:text="@string/score"
-            android:textColor="@color/black"
-            android:textSize="16sp"
-            android:layout_marginLeft="10dip"
-            android:layout_marginTop="10dip"
-            android:layout_below="@+id/tv_top_scorers_top" />
+<!--         <TextView -->
+<!--             android:id="@+id/tv_top_scorers_rating" -->
+<!--             android:layout_width="wrap_content" -->
+<!--             android:layout_height="wrap_content" -->
+<!--             android:text="@string/score" -->
+<!--             android:textColor="@color/black" -->
+<!--             android:textSize="16sp" -->
+<!--             android:layout_marginLeft="10dip" -->
+<!--             android:layout_marginTop="10dip" -->
+<!--             android:layout_below="@+id/tv_top_scorers_top" /> -->
 
 <!--         <RatingBar -->
 <!--             android:id="@+id/rb_ratingbar" -->
diff --git a/src/main/java/org/smilec/smile/ui/fragment/StudentsFragment.java b/src/main/java/org/smilec/smile/ui/fragment/StudentsFragment.java
index 70c3516..cfa77fa 100644
--- a/src/main/java/org/smilec/smile/ui/fragment/StudentsFragment.java
+++ b/src/main/java/org/smilec/smile/ui/fragment/StudentsFragment.java
@@ -251,11 +251,11 @@ private String getPercentCompleted() {
 
     	float n = 0;
     	
-    	System.out.println("*****students.size()="+students.size());
+    	System.out.println("**Number of students in session ="+students.size());
     	
     	for(int i=0; i<students.size();i++) {
     		
-    		System.out.println("=====students.get(i).isSolved()="+students.get(i).isSolved());
+    		System.out.println("**The student 'i' already solved the questions ="+students.get(i).isSolved());
     		
     		n += students.get(i).isSolved() ? 1:0;
     	}
@@ -294,10 +294,9 @@ private String getPercentCorrect() {
     			
 	    			nbQuestionsCorrect += answersOfStudent.get(q).equals(q.getAnswer()) ? 1:0;
 	    		} 
-	    		
-	    		// if the student has 70+% of correct answers, we count it 
-	    		nbStudentOver70 += nbQuestionsCorrect*100/questions.size() >= 70 ? 1:0;
 	    	}
+    		// if the student has 70+% of correct answers, we count it 
+    		nbStudentOver70 += nbQuestionsCorrect*100/questions.size() >= 70 ? 1:0;
     	}
 		
     	float percent = !students.isEmpty()? (float) (nbStudentOver70/students.size())*100 : 0;
@@ -314,13 +313,8 @@ private void setTopScorersArea(Results results, Board board) {
                 R.id.tv_top_scorers_top);
 
             String sBestScoredStudentNames = results.getBestScoredStudentNames();
-            JSONArray bestScoredStudentNames = new JSONArray(sBestScoredStudentNames == null ? ""
-                : sBestScoredStudentNames);
-            tvTopScorersTop.setText(getString(R.string.top_scorer) + ": "
-                + bestScoredStudentNames.join(", ").replaceAll("\"", ""));
-
-            TextView tvTopScorersRating = (TextView) getActivity().findViewById(R.id.tv_top_scorers_rating);
-            tvTopScorersRating.setText(getString(R.string.rating) + ": " + results.getWinnerRating());
+            JSONArray bestScoredStudentNames = new JSONArray(sBestScoredStudentNames == null ? "" : sBestScoredStudentNames);
+            tvTopScorersTop.setText(getString(R.string.top_scorer) + ": " + bestScoredStudentNames.join(", ").replaceAll("\"", ""));
             
             TextView tvPercentCompleted = (TextView) getActivity().findViewById(R.id.tv_percent_completed);
             tvPercentCompleted.setText(this.getPercentCompleted());
@@ -329,6 +323,9 @@ private void setTopScorersArea(Results results, Board board) {
             TextView tvPercentCorrect = (TextView) getActivity().findViewById(R.id.tv_percent_correct);
             tvPercentCorrect.setText(this.getPercentCorrect());
             tvPercentCorrect.setVisibility(View.VISIBLE);
+            
+            //TextView tvTopScorersRating = (TextView) getActivity().findViewById(R.id.tv_top_scorers_rating);
+            //tvTopScorersRating.setText(getString(R.string.rating) + ": " + results.getWinnerRating());
 
             // Seems useless >> issues 14 
             //final RatingBar rbRatingBar = (RatingBar) getActivity().findViewById(R.id.rb_ratingbar);