Skip to content

muchild 13.10 作业 -已批改2019.4.14#10

Open
muchild wants to merge 5 commits intozhangting85:masterfrom
muchild:master
Open

muchild 13.10 作业 -已批改2019.4.14#10
muchild wants to merge 5 commits intozhangting85:masterfrom
muchild:master

Conversation

@muchild
Copy link
Copy Markdown

@muchild muchild commented Mar 14, 2019

提交13.10 作业

Copy link
Copy Markdown
Owner

@zhangting85 zhangting85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

以下是各个测试用例
python currency_convert.py --field 2 --multiplier 0.8 < data.csv > data-fr.csv
python currency_convert.py --field 2 --multiplier 0.8 -i data.csv > data-fr.csv
python currency_convert.py --field 2 --multiplier 0.8 -i data.csv -o data-fr2.csv
python currency_convert.py --field 2 --multiplier 0.8 -i data.csv
python currency_convert.py --help
python currency_convert.py --field 2 --multiplier 0.8 -i "pivotsense,7.99,Pivotal Sense (ltd.),1483820006,/dl area/pivotsense.tgz,/r/ps.tgz"
python currency_convert.py --field 2 --multiplier 0.8 -i data-fr.csv > data3.csv

第一个测试用例就挂了:编码问题,详见这个回复#3 (comment)

请修改过代码并自行运行上述测试用例后重新提交。

(venv) C:\Users\colin.zt\Desktop\Homework_20180914\13.10>python currency_convert.py --field 2 --multiplier 0.8 < data.csv > data-fr.csv
Traceback (most recent call last):
File "currency_convert.py", line 114, in
export_file(data)
File "currency_convert.py", line 93, in export_file
print(e[i] + ',', end='')
UnicodeEncodeError: 'gbk' codec can't encode character '\u20ac' in position 0: illegal multibyte sequence


import _io
import sys, argparse, os

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

开始批改

@zhangting85 zhangting85 changed the title muchild 13.10 作业 muchild 13.10 作业 -已批改2019.4.14 Apr 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants