This commit is contained in:
Dan Poltawski 2013-06-18 10:35:47 +08:00
commit 83b8e822f1
6 changed files with 68 additions and 27 deletions

View File

@ -366,8 +366,12 @@ class qtype_multianswer_edit_form extends question_edit_form {
} }
foreach ($subquestion->answer as $key => $answer) { foreach ($subquestion->answer as $key => $answer) {
if ($subquestion->qtype == 'numerical' && $key == 0) { if ($subquestion->qtype == 'numerical' && $key == 0) {
$default_values[$prefix.'tolerance['.$key.']'] = if (isset($subquestion->tolerance[0])) {
$default_values[$prefix.'tolerance['.$key.']'] =
$subquestion->tolerance[0]; $subquestion->tolerance[0];
} else {
$default_values[$prefix.'tolerance['.$key.']'] = 0;
}
} }
if (is_array($answer)) { if (is_array($answer)) {
$answer = $answer['text']; $answer = $answer['text'];
@ -472,10 +476,11 @@ class qtype_multianswer_edit_form extends question_edit_form {
$trimmedanswer = trim($answer); $trimmedanswer = trim($answer);
if ($trimmedanswer !== '') { if ($trimmedanswer !== '') {
$answercount++; $answercount++;
if ($subquestion->qtype == 'numerical' && if ($subquestion->qtype == 'numerical' && $trimmedanswer != '*') {
!($this->is_valid_number($trimmedanswer) || $trimmedanswer == '*')) { if (!$this->is_valid_number($trimmedanswer)) {
$errors[$prefix.'answer['.$key.']'] = $errors[$prefix.'answer['.$key.']'] =
get_string('answermustbenumberorstar', 'qtype_numerical'); get_string('answermustbenumberorstar', 'qtype_numerical');
}
} }
if ($subquestion->fraction[$key] == 1) { if ($subquestion->fraction[$key] == 1) {
$maxgrade = true; $maxgrade = true;

View File

@ -262,9 +262,9 @@ define('ANSWER_ALTERNATIVE_REGEX_FEEDBACK', 5);
// NUMBER_FORMATED_ALTERNATIVE_ANSWER_REGEX is used // NUMBER_FORMATED_ALTERNATIVE_ANSWER_REGEX is used
// for identifying numerical answers in ANSWER_ALTERNATIVE_REGEX_ANSWER. // for identifying numerical answers in ANSWER_ALTERNATIVE_REGEX_ANSWER.
define('NUMBER_REGEX', define('NUMBER_REGEX',
'-?(([0-9]+[.,]?[0-9]*|[.,][0-9]+)([eE][-+]?[0-9]+)?)'); '-?(([0-9]+[., ]?[., 0-9]*|[., ][0-9]+)([eE][-+]?[0-9]+)?)');
define('NUMERICAL_ALTERNATIVE_REGEX', define('NUMERICAL_ALTERNATIVE_REGEX',
'^(' . NUMBER_REGEX . ')(:' . NUMBER_REGEX . ')?$'); '^(' . NUMBER_REGEX . ')([ ]*:' . NUMBER_REGEX . ')?$');
// Parenthesis positions for NUMERICAL_FORMATED_ALTERNATIVE_ANSWER_REGEX. // Parenthesis positions for NUMERICAL_FORMATED_ALTERNATIVE_ANSWER_REGEX.
define('NUMERICAL_CORRECT_ANSWER', 1); define('NUMERICAL_CORRECT_ANSWER', 1);

View File

@ -59,7 +59,7 @@ class qtype_numerical_edit_form extends question_edit_form {
$tolerance = $mform->createElement('text', 'tolerance', $tolerance = $mform->createElement('text', 'tolerance',
get_string('answererror', 'qtype_numerical'), array('size' => 15)); get_string('answererror', 'qtype_numerical'), array('size' => 15));
$repeatedoptions['tolerance']['type'] = PARAM_FLOAT; $repeatedoptions['tolerance']['type'] = PARAM_RAW;// This parameter will be validated in validation().
$repeatedoptions['tolerance']['default'] = 0; $repeatedoptions['tolerance']['default'] = 0;
$elements = $repeated[0]->getElements(); $elements = $repeated[0]->getElements();
$elements[0]->setSize(15); $elements[0]->setSize(15);
@ -293,7 +293,7 @@ class qtype_numerical_edit_form extends question_edit_form {
if ($data['fraction'][$key] == 1) { if ($data['fraction'][$key] == 1) {
$maxgrade = true; $maxgrade = true;
} }
if ($answer !== '*' && !is_numeric($data['tolerance'][$key])) { if ($answer !== '*' && !$this->is_valid_number($data['tolerance'][$key])) {
$errors['answeroptions['.$key.']'] = $errors['answeroptions['.$key.']'] =
get_string('xmustbenumeric', 'qtype_numerical', get_string('xmustbenumeric', 'qtype_numerical',
get_string('acceptederror', 'qtype_numerical')); get_string('acceptederror', 'qtype_numerical'));

View File

@ -106,10 +106,6 @@ class qtype_numerical_question extends question_graded_automatically {
return false; return false;
} }
if ($this->ap->contains_thousands_seaparator($response['answer'])) {
return false;
}
return true; return true;
} }
@ -131,11 +127,6 @@ class qtype_numerical_question extends question_graded_automatically {
return get_string('unitnotselected', 'qtype_numerical'); return get_string('unitnotselected', 'qtype_numerical');
} }
if ($this->ap->contains_thousands_seaparator($response['answer'])) {
return get_string('pleaseenteranswerwithoutthousandssep', 'qtype_numerical',
$this->ap->get_separator());
}
return ''; return '';
} }

View File

@ -632,17 +632,35 @@ class qtype_numerical_answer_processor {
// Strip spaces (which may be thousands separators) and change other forms // Strip spaces (which may be thousands separators) and change other forms
// of writing e to e. // of writing e to e.
$response = str_replace(' ', '', $response); $response = str_replace(' ', '', $response);
$response = preg_replace('~(?:e|E|(?:x|\*|×)10(?:\^|\*\*))([+-]?\d+)~', 'e$1', $response); // Strip thousand separators like half space.
if (!in_array($this->thousandssep, array(',', '.', ' '))) {
// If a . is present or there are multiple , (i.e. 2,456,789 ) assume , if (strpos($response, $this->thousandssep) !== false) {
// is a thouseands separator, and strip it, else assume it is a decimal $response = str_replace($this->thousandssep, '', $response);
// separator, and change it to .. }
if (strpos($response, '.') !== false || substr_count($response, ',') > 1) { }
$response = str_replace(',', '', $response); $response = preg_replace('~(?:e|E|(?:x|\*|×)10(?:\^|\*\*))([+-]?\d+)~', 'e$1', $response);
} else { // Dot . is mostly a decimal separator there a few exceptions where it is a thousand separator
$response = str_replace(',', '.', $response); // If a . is present or there are multiple , (i.e. 2,456,789 ) assume,
// is a thousands separator and strip it or else assume it is a decimal
// separator and change it to .
// if only one and it is , then change to .
if (substr_count($response, ',')+ substr_count($response, '.') == 1 ) {
if (strpos($response, ',') !== false) {
$response = str_replace(',', '.', $response);
}
} else if (substr_count($response, ',') == 1 && substr_count($response, '.') == 1) {
if (strpos($response, '.') > strpos($response, ',')) { // Then , is thousand.
$response = str_replace(',', '', $response);
} else {
$response = str_replace('.', '', $response);
$response = str_replace(',', '.', $response);
}
} else if (substr_count($response, ',') > 1) {
$response = str_replace(',', '', $response);
} else if (substr_count($response, '.') > 1) {
$response = str_replace('.', '', $response);
$response = str_replace(',', '.', $response);
} }
$regex = '[+-]?(?:\d+(?:\\.\d*)?|\\.\d+)(?:e[-+]?\d+)?'; $regex = '[+-]?(?:\d+(?:\\.\d*)?|\\.\d+)(?:e[-+]?\d+)?';
if ($this->unitsbefore) { if ($this->unitsbefore) {
$regex = "/$regex$/"; $regex = "/$regex$/";

View File

@ -131,9 +131,36 @@ class qtype_numerical_answer_processor_test extends advanced_testcase {
$ap = new qtype_numerical_answer_processor(array(), false, ',', ' '); $ap = new qtype_numerical_answer_processor(array(), false, ',', ' ');
$this->assertEquals(array(-1000, '', null), $ap->apply_units('-1 000')); $this->assertEquals(array(-1000, '', null), $ap->apply_units('-1 000'));
$this->assertEquals(array(-1000000, '', null), $ap->apply_units('-1 000 000'));
$this->assertEquals(array(-1000000, '', null), $ap->apply_units('-1.000.000'));
$this->assertEquals(array(-1000000, '', null), $ap->apply_units('-1,000,000'));
$this->assertEquals(array(-1000.1, '', null), $ap->apply_units('-1000,1'));
$this->assertEquals(array(-1000.1, '', null), $ap->apply_units('-1.000,1'));
$this->assertEquals(array(-1000.1, '', null), $ap->apply_units('-1,000.1'));
$this->assertEquals(array(3.14159, '', null), $ap->apply_units('3,14159')); $this->assertEquals(array(3.14159, '', null), $ap->apply_units('3,14159'));
} }
public function test_deutsch_style() {
$ap = new qtype_numerical_answer_processor(array(), false, ',', '.');
$this->assertEquals(array(-1000, '', null), $ap->apply_units('-1 000'));
$this->assertEquals(array(-1000000, '', null), $ap->apply_units('-1 000 000'));
$this->assertEquals(array(-1000000, '', null), $ap->apply_units('-1.000.000'));
$this->assertEquals(array(-1000000, '', null), $ap->apply_units('-1.000.000,'));
$this->assertEquals(array(-1000.1, '', null), $ap->apply_units('-1000,1'));
$this->assertEquals(array(-1000.1, '', null), $ap->apply_units('-1.000,1'));
$this->assertEquals(array(-1000.1, '', null), $ap->apply_units('-1,000.1'));
$this->assertEquals(array(3.14159, '', null), $ap->apply_units('3.14159'));
$this->assertEquals(array(3.14159, '', null), $ap->apply_units('3,14159'));
}
public function test_unusual_thousandseps_style() {
// Test with ' as thousandsep as used for currency in Swizterland.
$ap = new qtype_numerical_answer_processor(array(), false, '.', "'");
$this->assertEquals(array(-1000, '', null), $ap->apply_units("-1'000"));
$this->assertEquals(array(-1000000, '', null), $ap->apply_units("'-1'000'000"));
$this->assertEquals(array(-1000000, '', null), $ap->apply_units("-1'000'000."));
}
public function test_percent() { public function test_percent() {
$ap = new qtype_numerical_answer_processor(array('%' => 100), false, '.', ','); $ap = new qtype_numerical_answer_processor(array('%' => 100), false, '.', ',');