cafe-grader-web Commit - r383:31aa87b1b1e9

cafe-grader-web

Commit `r383:31aa87b1b1e9` public

parent | child

Description:

saves submitted output, checks time outs; updated submission front-end flows

Commit status:

[Not Reviewed]

References:

default

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r383:31aa87b1b1e9 - Thu, 29 Jan 2015 03:09:12 - 8 files changed: 116 inserted, 20 deleted

db/migrate/20150129021221_add_output_to_submissions.rb created 644 ¶ +5

@@ -0,0 +1,5
	1	+ class AddOutputToSubmissions < ActiveRecord::Migration
	2	+ def change
	3	+ add_column :submissions, :output, :text
	4	+ end
	5	+ end

app/assets/javascripts/toicontest.js ¶ +43 -3

@@ -1,7 +1,9
1	var TOIContest = {	1	var TOIContest = {
2	NO_TIMEOUT: -1,	2	NO_TIMEOUT: -1,
		3	+ SUBMISSION_TIMEOUT: 300,
3		4
4	timeOuts: {},	5	timeOuts: {},
		6	+ timeStarted: 0,
5		7
6	problemSelectClick: function() {	8	problemSelectClick: function() {
7	$$(".submission-submit-divs").each(function(item) {	9	$$(".submission-submit-divs").each(function(item) {
@@ -14,8 +16,18
14	$("submission_submit_div_" + problem_id + "_id").show();	16	$("submission_submit_div_" + problem_id + "_id").show();
15	},	17	},
16		18
17	- confirmDownload: function() {	19	+ confirmDownload: function(pid) {
18	- return confirm("แน่ใจ?");	20	+ result = confirm("คุณแน่ใจที่จะส่งข้อนี้หรือไม่?\nเมื่อคุณดาวน์โหลดข้อมูลชุดทดสอบแล้ว คุณจะต้องส่งข้อมูลส่งออกและโปรแกรมภายในเวลา 5 นาที");
		21	+ if ( result ) {
		22	+ if ( TOIContest.timeOuts[ pid ] == TOIContest.NO_TIMEOUT ) {
		23	+ TOIContest.refreshTimeOuts();
		24	+
		25	+ TOIContest.timeOuts[ pid ] = TOIContest.SUBMISSION_TIMEOUT;
		26	+
		27	+ TOIContest.refreshTimeOutMessages();
		28	+ }
		29	+ }
		30	+ return result;
19	},	31	},
20		32
21	refreshTimeOutMessages: function() {	33	refreshTimeOutMessages: function() {
@@ -26,12 +38,40
26	var minLeft = parseInt(timeOut / 60);	38	var minLeft = parseInt(timeOut / 60);
27	var secLeft = parseInt(timeOut % 60);	39	var secLeft = parseInt(timeOut % 60);
28	$('submission_time_left_' + pid + '_id').innerHTML = '\| <b>เหลือเวลาอีก ' + minLeft + ':' + secLeft + ' นาที</b>';	40	$('submission_time_left_' + pid + '_id').innerHTML = '\| <b>เหลือเวลาอีก ' + minLeft + ':' + secLeft + ' นาที</b>';
		41	+ $('submission_form_'+ pid + '_id').show();
29	} else {	42	} else {
30	$('submission_time_left_' + pid + '_id').innerHTML = '\| <b>หมดเวลาส่ง</a>';	43	$('submission_time_left_' + pid + '_id').innerHTML = '\| <b>หมดเวลาส่ง</a>';
31	$('submission_form_'+ pid + '_id').hide();	44	$('submission_form_'+ pid + '_id').hide();
32	}	45	}
		46	+ } else {
		47	+ $('submission_form_'+ pid + '_id').hide();
33	}	48	}
34	}	49	}
35	- }	50	+ },
		51	+
		52	+ refreshTimeOuts: function() {
		53	+ if ( TOIContest.timeStarted == 0 ) {
		54	+ TOIContest.timeStarted = (new Date()).getTime();
		55	+ }
		56	+
		57	+ var timeElapsed = ((new Date()).getTime() - TOIContest.timeStarted)/1000;
		58	+ for ( var pid in TOIContest.timeOuts ) {
		59	+ var timeOut = TOIContest.timeOuts[ pid ];
		60	+ if ( timeOut > timeElapsed ) {
		61	+ TOIContest.timeOuts[ pid ] -= timeElapsed;
		62	+ } else if ( timeOut > 0 ) {
		63	+ TOIContest.timeOuts[ pid ] = 0;
		64	+ }
		65	+ }
		66	+ },
		67	+
		68	+ registerRefreshEvent: function() {
		69	+ TOIContest.timeStarted = (new Date()).getTime();
		70	+ setTimeout(function () {
		71	+ TOIContest.refreshTimeOuts();
		72	+ TOIContest.refreshTimeOutMessages();
		73	+ TOIContest.registerRefreshEvent();
		74	+ }, 1000);
		75	+ },
36	};	76	};
37		77

app/controllers/main_controller.rb ¶ +41 -11

+            + # coding: utf-8
               class MainController < ApplicationController
                 before_filter :authenticate, :except => [:index, :login]
                 end
                 def list
+            +     if session[:current_problem_id]
+            +       @current_problem = Problem.find(session[:current_problem_id])
+            +       session[:current_problem_id] = nil
+            +     end
                   prepare_list_information
                 end
                   user = User.find(session[:user_id])
                   @submission = Submission.new
-            -     @submission.problem_id = params[:submission][:problem_id]
+            +     @current_problem = Problem.find(params[:submission][:problem_id])
+            +
+            +     if !@current_problem
+            +       flash[:notice] = 'Error: คุณยังไม่ได้ระบุข้อที่ต้องการส่ง'
+            +       redirect_to :action => 'list'
+            +     end
+            +
+            +     assignment = user.get_test_pair_assignment_for(@current_problem)
+            +     if !assignment
+            +       flash[:notice] = 'Error: คุณยังไม่ได้ดาวน์โหลดข้อมูลทดสอบ'
+            +       prepare_list_information
+            +       render :action => 'list' and return
+            +     end
+            +     if assignment.expired?
+            +       flash[:notice] = 'Error: หมดเวลาส่งสำหรับข้อนี้'
+            +       prepare_list_information
+            +       render :action => 'list' and return
+            +     end
+            +
+            +     @submission.problem = @current_problem
                   @submission.user = user
                   @submission.language_id = 0
                   if (params['file']) and (params['file']!='')
                     @submission.source = params['file'].read
                     @submission.source_filename = params['file'].original_filename
                   end
+            +     if (params['output_file']) and (params['output_file']!='')
+            +       @submission.output = params['output_file'].read
+            +     end
                   @submission.submitted_at = Time.new.gmtime
                   if GraderConfiguration.time_limit_mode? and user.contest_finished?
                     elsif Task.create(:submission_id => @submission.id,
                                       :status => Task::STATUS_INQUEUE) == false
               	flash[:notice] = 'Error adding your submission to task queue'
+            +       else
+            +         flash[:notice] = 'จัดเก็บคำตอบและโปรแกรมที่คุณส่งเรียบร้อย'
                     end
                   else
                     prepare_list_information
                     render :action => 'list' and return
                   end
+            +
+            +     if @current_problem
+            +       session[:current_problem_id] = @current_problem.id
+            +     end
                   redirect_to :action => 'list'
                 end
                     test_pair = TestPair.get_for(problem, true)
                     user = User.find(session[:user_id])
-            -       assignent = user.get_test_pair_assignment_for(problem)
+            +       assignment = user.get_test_pair_assignment_for(problem)
-            -       if !assignent
+            +       if !assignment
-            -         assignent = TestPairAssignment.new
+            +         assignment = TestPairAssignment.create_for(user, problem, test_pair)
-            -         assignent.user = user
+            +         assignment.save
-            -         assignent.problem = problem
-            -         assignent.test_pair = test_pair
-            -         assignent.submitted = false
-            -         assignent.save
                     end
                     send_data(test_pair.input,
                     redirect_to :action => 'list' and return
                   end
+            +     @current_problem = problem
                   test_pair = TestPair.get_for(problem, false)
                   if (params['output_file']) and (params['output_file']!='')
                     output = params['output_file'].read
-            -       @current_problem = problem
                     @grading_result = grade(output, test_pair.solution)
                     prepare_list_information
                     render :action => 'list' and return
                   else
                     flash[:notice] = 'Error: output file errors'
-            -       redirect_to :action => 'list'
+            +       prepare_list_information
+            +       render :action => 'list' and return
                   end
                 end

app/models/submission.rb ¶ +5

                 validates_presence_of :source
                 validates_length_of :source, :maximum => 100_000, :allow_blank => true, :message => 'too long'
                 validates_length_of :source, :minimum => 1, :allow_blank => true, :message => 'too short'
+            +
+            +   validates_presence_of :output
+            +   validates_length_of :output, :maximum => 100_000, :allow_blank => true, :message => 'too long'
+            +   validates_length_of :output, :minimum => 1, :allow_blank => true, :message => 'too short'
+            +
                 validate :must_have_valid_problem
                 validate :must_specify_language

app/models/test_pair_assignment.rb ¶ +10

@@ -6,4 +6,14
6	def expired?	6	def expired?
7	return created_at + TEST_ASSIGNMENT_EXPIRATION_DURATION < Time.new.gmtime	7	return created_at + TEST_ASSIGNMENT_EXPIRATION_DURATION < Time.new.gmtime
8	end	8	end
		9	+
		10	+ def self.create_for(user, problem, test_pair)
		11	+ assignment = TestPairAssignment.new
		12	+ assignment.user = user
		13	+ assignment.problem = problem
		14	+ assignment.test_pair = test_pair
		15	+ assignment.submitted = false
		16	+ assignment.save
		17	+ return assignment
		18	+ end
9	end	19	end

app/views/main/_submission_box.html.erb ¶ +7 -5

+            + <% selected_problem_id = @current_problem ? @current_problem.id : -1 %>
               <div class="submitbox">
                 <b>Problem:</b> <%= select 'submission', 'problem_id',
                            	      [['กรุณาเลือกข้อที่ต้องการส่งหรือทดสอบ','-1']] +
               		      @problems.collect {|p| [p.full_name, p.id]},
-            - 		      { :selected => '-1' },
+            + 		      { :selected => selected_problem_id },
                                     { :onchange => 'TOIContest.problemSelectClick()' } %>
               </div>
               <% @problems.each do |problem| %>
-            -   <div class="submission-submit-divs" id="submission_submit_div_<%= problem.id %>_id" style="displanny: none;">
+            +   <div class="submission-submit-divs" id="submission_submit_div_<%= problem.id %>_id" style="display: none;">
                   <div style="border: 1px solid #c0c0c0; padding: 5px; margin: 5px 0 5px 0;">
                     <b><%= problem.full_name %>: ข้อมูลสำหรับตรวจสอบ</b> (สามารถดาวน์โหลดและส่งกี่ครั้งก็ได้,ไม่มีคะแนน):
                     <%= link_to 'ดาวน์โหลด input', :action => 'verifying_testcase', :id => problem.id %>
                   </div>
                   <div style="border: 1px solid #c0c0c0; padding: 5px; margin: 5px 0 5px 0;">
                     <b><%= problem.full_name %>: ข้อมูลทดสอบจริง</b> (ส่งกี่ครั้งก็ได้ภายในเวลา 5 นาทีหลังดาวน์โหลด):
-            -       <%= link_to 'ดาวน์โหลด input และเริ่มจับเวลา', { :action => 'testcase', :id => problem.id}, { :onclick => 'return TOIContest.confirmDownload()' } %>
+            +       <%= link_to 'ดาวน์โหลด input และเริ่มจับเวลา', { :action => 'testcase', :id => problem.id}, { :onclick => "return TOIContest.confirmDownload(#{problem.id})" } %>
                     <span id="submission_time_left_<%= problem.id %>_id"></span>
-            -       <%= form_tag({:controller => 'main', :action => 'submit', :id => problem.id}, {:method => 'post', :multipart => true, :id => "submission_form_#{problem.id}_id" }) do %>
+            +       <%= form_tag({:controller => 'main', :action => 'submit'}, {:method => 'post', :multipart => true, :id => "submission_form_#{problem.id}_id" }) do %>
+            +         <%= hidden_field_tag 'submission[problem_id]', problem.id %>
                       ข้อมูลส่งออก: <%= file_field_tag 'output_file' %>
-            -         โปรแกรมคำตอบ: <%= file_field_tag 'source_file' %>
+            +         โปรแกรมคำตอบ: <%= file_field_tag 'file' %>
                       <%= submit_tag 'Submit' %>
                     <% end %>
                   </div>

app/views/main/list.html.haml ¶ +3

                 = "Announcement.refreshUrl = '#{url_for :controller => 'main', :action => 'announcements'}';"
                 Announcement.registerRefreshEventTimer();
+            +   TOIContest.problemSelectClick()
+            +
                 TOIContest.timeOuts = {};
                 - @problems.each do |p|
                   - if (@submission_timeouts.has_key? p.id) and (@submission_timeouts[p.id] != nil)
                     = "TOIContest.timeOuts[#{p.id}] = TOIContest.NO_TIMEOUT;"
                 TOIContest.refreshTimeOutMessages();
+            +   TOIContest.registerRefreshEvent();

db/schema.rb ¶ +2 -1

               #
               # It's strongly recommended to check this file into your version control system.
-            - ActiveRecord::Schema.define(:version => 20150128165518) do
+            + ActiveRecord::Schema.define(:version => 20150129021221) do
                 create_table "announcements", :force => true do |t|
                   t.string   "author"
                   t.text     "grader_comment"
                   t.integer  "number"
                   t.string   "source_filename"
+            +     t.text     "output"
                 end
                 add_index "submissions", ["user_id", "problem_id", "number"], :name => "index_submissions_on_user_id_and_problem_id_and_number", :unique => true

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r383:31aa87b1b1e9 - Thu, 29 Jan 2015 03:09:12 - 8 files changed: 116 inserted, 20 deleted

Sign in to your account

Commit r383:31aa87b1b1e9 public

Author

r383:31aa87b1b1e9 - Thu, 29 Jan 2015 03:09:12 - 8 files changed: 116 inserted, 20 deleted

Commit `r383:31aa87b1b1e9` public