Форум на Исходниках.RU

[Py] Учу питон, поругайте за код

Ho Im — Sun, 02 Jan 2011 10:22:36 +0000

Ho Im: Какая сказочная бредятина космического масштаба.

А для Фаренгейта мы вместо "->>" сделаем ">-(**)-<" (оператор-крабе). Просто чтобы жизнь малиной не казалась (и чтобы нанотехнологии прославить, оператор «шмеле» еще надо придумать).

И не дай Бог надо из Фаренгейта в Реомюра перевести, потому что мы умеем только из Фаренгейта в Цельсия, а из Реомюра только в Ромера, а из Ромера только в Ньютона...

[Py] Учу питон, поругайте за код

amk — Sun, 02 Jan 2011 10:19:03 +0000

amk: Во-первых, у тебя неправильный сдвиг шкал градусы Кельвина - градусы Цельсия
Вдобавок, для Фаренгейта придется, кроме сдвига шкал еще и масштаб придется менять так что твой конструктор функций не сработает, придется новый писать.
Шкала Реомюра не помню чем отличается - слишком старая.
То, что ты предложил, еще хуже чем мультиметоды. Тебе в каждом месте придется ручками вписывать из каких конкретно единиц в какие другие единицы ты собираешься преобразовывать, и никакой проверки при этом.

[Py] Учу питон, поругайте за код

Vadim11 — Sun, 02 Jan 2011 06:43:48 +0000

Vadim11: Зачем к лопате приделывать грабли и вилы - работать не удобно.
Лучше разделять, примерно так:

def make_inc(t): return lambda x,y=t: x+t
cel2kel = make_inc(237) # Цельсий-Кельвин
kel2cel = make_inc(-237) # Кельвин-Цельсий

Можно обобщить (хотя это не есть хорошо)
def CelKel(s,t):
if s == "->>": return cel2kel(t)
if s == "<<-": return kel2cel(t)
Так же можно сделать для других шкал, например для Фаренгейта или Руомера.

[Py] Учу питон, поругайте за код

Мяут-Настоящий — Sat, 01 Jan 2011 09:26:54 +0000

Мяут-Настоящий: Может ребят вы в холивары? :)

[Py] Учу питон, поругайте за код

Ho Im — Sat, 01 Jan 2011 00:51:37 +0000

Ho Im: Как пример, можно себе представить задачу с конвертированием символов из одной кодировки в другую. В прошлом веке, когда юникод был экзотикой или даже до того, каждый русскоязычный компьютерщик рано или поздно постигал, что такое Codepage 866, 1125, 1251, KOI8-R/U, ISO8859-5. Сейчас это не так актуально, но все же необходимость перекодировывать тексты из и в случается. У нас в юниксах для этого есть утилиты iconv и recode, какую использовать — дело вкуса и религии, но.

Вот что говорит мне iconv, когда я прошу вывести список поддерживаемых кодировок.

Сразу говорю, там 143 кодировки

ANSI_X3.4-1968 ANSI_X3.4-1986 ASCII CP367 IBM367 ISO-IR-6 ISO646-US ISO_646.IRV:1991 US US-ASCII CSASCII
UTF-8
UTF-8-MAC UTF8-MAC
ISO-10646-UCS-2 UCS-2 CSUNICODE
UCS-2BE UNICODE-1-1 UNICODEBIG CSUNICODE11
UCS-2LE UNICODELITTLE
ISO-10646-UCS-4 UCS-4 CSUCS4
UCS-4BE
UCS-4LE
UTF-16
UTF-16BE
UTF-16LE
UTF-32
UTF-32BE
UTF-32LE
UNICODE-1-1-UTF-7 UTF-7 CSUNICODE11UTF7
UCS-2-INTERNAL
UCS-2-SWAPPED
UCS-4-INTERNAL
UCS-4-SWAPPED
C99
JAVA
CP819 IBM819 ISO-8859-1 ISO-IR-100 ISO8859-1 ISO_8859-1 ISO_8859-1:1987 L1 LATIN1 CSISOLATIN1
ISO-8859-2 ISO-IR-101 ISO8859-2 ISO_8859-2 ISO_8859-2:1987 L2 LATIN2 CSISOLATIN2
ISO-8859-3 ISO-IR-109 ISO8859-3 ISO_8859-3 ISO_8859-3:1988 L3 LATIN3 CSISOLATIN3
ISO-8859-4 ISO-IR-110 ISO8859-4 ISO_8859-4 ISO_8859-4:1988 L4 LATIN4 CSISOLATIN4
CYRILLIC ISO-8859-5 ISO-IR-144 ISO8859-5 ISO_8859-5 ISO_8859-5:1988 CSISOLATINCYRILLIC
ARABIC ASMO-708 ECMA-114 ISO-8859-6 ISO-IR-127 ISO8859-6 ISO_8859-6 ISO_8859-6:1987 CSISOLATINARABIC
ECMA-118 ELOT_928 GREEK GREEK8 ISO-8859-7 ISO-IR-126 ISO8859-7 ISO_8859-7 ISO_8859-7:1987 ISO_8859-7:2003 CSISOLATINGREEK
HEBREW ISO-8859-8 ISO-IR-138 ISO8859-8 ISO_8859-8 ISO_8859-8:1988 CSISOLATINHEBREW
ISO-8859-9 ISO-IR-148 ISO8859-9 ISO_8859-9 ISO_8859-9:1989 L5 LATIN5 CSISOLATIN5
ISO-8859-10 ISO-IR-157 ISO8859-10 ISO_8859-10 ISO_8859-10:1992 L6 LATIN6 CSISOLATIN6
ISO-8859-11 ISO8859-11 ISO_8859-11
ISO-8859-13 ISO-IR-179 ISO8859-13 ISO_8859-13 L7 LATIN7
ISO-8859-14 ISO-CELTIC ISO-IR-199 ISO8859-14 ISO_8859-14 ISO_8859-14:1998 L8 LATIN8
ISO-8859-15 ISO-IR-203 ISO8859-15 ISO_8859-15 ISO_8859-15:1998 LATIN-9
ISO-8859-16 ISO-IR-226 ISO8859-16 ISO_8859-16 ISO_8859-16:2001 L10 LATIN10
KOI8-R CSKOI8R
KOI8-U
KOI8-RU
CP1250 MS-EE WINDOWS-1250
CP1251 MS-CYRL WINDOWS-1251
CP1252 MS-ANSI WINDOWS-1252
CP1253 MS-GREEK WINDOWS-1253
CP1254 MS-TURK WINDOWS-1254
CP1255 MS-HEBR WINDOWS-1255
CP1256 MS-ARAB WINDOWS-1256
CP1257 WINBALTRIM WINDOWS-1257
CP1258 WINDOWS-1258
850 CP850 IBM850 CSPC850MULTILINGUAL
862 CP862 IBM862 CSPC862LATINHEBREW
866 CP866 IBM866 CSIBM866
CP1131
MAC MACINTOSH MACROMAN CSMACINTOSH
MACCENTRALEUROPE
MACICELAND
MACCROATIAN
MACROMANIA
MACCYRILLIC
MACUKRAINE
MACGREEK
MACTURKISH
MACHEBREW
MACARABIC
MACTHAI
HP-ROMAN8 R8 ROMAN8 CSHPROMAN8
NEXTSTEP
ARMSCII-8
GEORGIAN-ACADEMY
GEORGIAN-PS
KOI8-T
CP154 CYRILLIC-ASIAN PT154 PTCP154 CSPTCP154
KZ-1048 RK1048 STRK1048-2002 CSKZ1048
MULELAO-1
CP1133 IBM-CP1133
ISO-IR-166 TIS-620 TIS620 TIS620-0 TIS620.2529-1 TIS620.2533-0 TIS620.2533-1
CP874 WINDOWS-874
VISCII VISCII1.1-1 CSVISCII
TCVN TCVN-5712 TCVN5712-1 TCVN5712-1:1993
ISO-IR-14 ISO646-JP JIS_C6220-1969-RO JP CSISO14JISC6220RO
JISX0201-1976 JIS_X0201 X0201 CSHALFWIDTHKATAKANA
ISO-IR-87 JIS0208 JIS_C6226-1983 JIS_X0208 JIS_X0208-1983 JIS_X0208-1990 X0208 CSISO87JISX0208
ISO-IR-159 JIS_X0212 JIS_X0212-1990 JIS_X0212.1990-0 X0212 CSISO159JISX02121990
CN GB_1988-80 ISO-IR-57 ISO646-CN CSISO57GB1988
CHINESE GB_2312-80 ISO-IR-58 CSISO58GB231280
CN-GB-ISOIR165 ISO-IR-165
ISO-IR-149 KOREAN KSC_5601 KS_C_5601-1987 KS_C_5601-1989 CSKSC56011987
EUC-JP EUCJP EXTENDED_UNIX_CODE_PACKED_FORMAT_FOR_JAPANESE CSEUCPKDFMTJAPANESE
MS_KANJI SHIFT-JIS SHIFT_JIS SJIS CSSHIFTJIS
CP932
ISO-2022-JP CSISO2022JP
ISO-2022-JP-1
ISO-2022-JP-2 CSISO2022JP2
CN-GB EUC-CN EUCCN GB2312 CSGB2312
GBK
CP936 MS936 WINDOWS-936
GB18030
ISO-2022-CN CSISO2022CN
ISO-2022-CN-EXT
HZ HZ-GB-2312
EUC-TW EUCTW CSEUCTW
BIG-5 BIG-FIVE BIG5 BIGFIVE CN-BIG5 CSBIG5
CP950
BIG5-HKSCS:1999
BIG5-HKSCS:2001
BIG5-HKSCS BIG5-HKSCS:2004 BIG5HKSCS
EUC-KR EUCKR CSEUCKR
CP949 UHC
CP1361 JOHAB
ISO-2022-KR CSISO2022KR
CP856
CP922
CP943
CP1046
CP1124
CP1129
CP1161 IBM-1161 IBM1161 CSIBM1161
CP1162 IBM-1162 IBM1162 CSIBM1162
CP1163 IBM-1163 IBM1163 CSIBM1163
DEC-KANJI
DEC-HANYU
437 CP437 IBM437 CSPC8CODEPAGE437
CP737
CP775 IBM775 CSPC775BALTIC
852 CP852 IBM852 CSPCP852
CP853
855 CP855 IBM855 CSIBM855
857 CP857 IBM857 CSIBM857
CP858
860 CP860 IBM860 CSIBM860
861 CP-IS CP861 IBM861 CSIBM861
863 CP863 IBM863 CSIBM863
CP864 IBM864 CSIBM864
865 CP865 IBM865 CSIBM865
869 CP-GR CP869 IBM869 CSIBM869
CP1125
EUC-JIS-2004 EUC-JISX0213
SHIFT_JIS-2004 SHIFT_JISX0213
ISO-2022-JP-2004 ISO-2022-JP-3
BIG5-2003
ISO-IR-230 TDS565
ATARI ATARIST
RISCOS-LATIN1

В общем и целом предлагаю охватить масштабность проекта «конвертор из любой кодировки в любую» на мультиметодах. Если исключить варианты «в самого себя», получим 20306 методов. Возможно, если автор — рантье и ему не надо работать, чтобы себя кормить, он сможет до пенсии это написать, правда, кому оно надо, еще вопрос.

Ну и появление новой кодировки — это неизменный facepalm для любителя мультиметодов.

[Py] Учу питон, поругайте за код

amk — Fri, 31 Dec 2010 14:59:49 +0000

amk:

Цитата korvin @ 31.12.10, 11:53

ой ли?

Ну смотри, если у тебя есть десяток конвертируемых вариантов, то тебе нужно написать 100 (!) мультиметодов, включая десяток тривиальных. Для варианта исходный-основа-итоговый только 20, и из них только 2 тривиальных для основного варианта.
При добавлении еще одного варианта в случае мультиметода тебе придется написать еще 21 конвертор (1 тривиальный), а для варианта исходный-основа-итог только два - для нового варианта.
Работать будет может быть чуть медленнее, но зато куда проще сопровождать.

[Py] Учу питон, поругайте за код

korvin — Fri, 31 Dec 2010 11:53:02 +0000

korvin:

Цитата amk @ 30.12.10, 11:44

Цитата korvin @ 29.12.10, 17:57

мне вообще мультиметоды больше нравятся

Только вот именно в этой задаче пара конверторов из/в работает и сопровождается лучше.

ой ли?

Добавлено 31.12.10, 11:56

Цитата Lumberjack @ 29.12.10, 23:31

Ты можешь не заморачиваться и воспользоваться непечатаемыми символами Юникода чтобы не нагромождать пробелами или табами текст. А вообще по сути ты рассуждаешь не как программист а как капризный пацан, реально метод предложенный Ho Im я поддерживаю, т.к он во всех книгах написан и еще ни один учебник мне не показал что нужно "принтить". Да прибудет с тобой сила Any key

я как бы стараюсь придерживаться отступов, выпирающие строки выглядят просто ужасно. я не знаком с питоном, но меня бы вполне устроил вариант а ля:

...

"abcde"

"fghij"

"klmno"

...

Добавлено 31.12.10, 11:58

Цитата Dark_Sup @ 30.12.10, 19:36

Дык на то нам дана подсветка кода! :-? У меня на этих "длинных строках" глаз даже не задерживается.

а меня жутко раздражает, независимо от подцветки. вообще полагаться на подцветку при написании кода -- не очень здравая затея.

[Py] Учу питон, поругайте за код

Dark_Sup — Thu, 30 Dec 2010 19:36:26 +0000

Dark_Sup:

Цитата korvin @ 29.12.10, 17:57

а мне не нравится портить форматирование кода "многострочными строками",

Дык на то нам дана подсветка кода! :-? У меня на этих "длинных строках" глаз даже не задерживается.

Кстати, функции в таблице по ключу - прикольно... возьму на вооружение. :)

[Py] Учу питон, поругайте за код

amk — Thu, 30 Dec 2010 13:14:03 +0000

amk: Эта проблема - преобразование из всего во все иногда встречается и в реальной работе. Почему бы не выделить ее в отдельную задачу?

[Py] Учу питон, поругайте за код

Мяут-Настоящий — Thu, 30 Dec 2010 12:25:26 +0000

Мяут-Настоящий:

Цитата amk @ 30.12.10, 11:44

Только вот именно в этой задаче пара конверторов из/в работает и сопровождается лучше.

Это можно назвать задачей? :huh:

[Py] Учу питон, поругайте за код

amk — Thu, 30 Dec 2010 11:44:27 +0000

amk:

Цитата korvin @ 29.12.10, 17:57

мне вообще мультиметоды больше нравятся

Только вот именно в этой задаче пара конверторов из/в работает и сопровождается лучше.

[Py] Учу питон, поругайте за код

Lumberjack — Wed, 29 Dec 2010 23:31:42 +0000

Lumberjack: Ты можешь не заморачиваться и воспользоваться непечатаемыми символами Юникода чтобы не нагромождать пробелами или табами текст. А вообще по сути ты рассуждаешь не как программист а как капризный пацан, реально метод предложенный Ho Im я поддерживаю, т.к он во всех книгах написан и еще ни один учебник мне не показал что нужно "принтить". Да прибудет с тобой сила Any key

[Py] Учу питон, поругайте за код

korvin — Wed, 29 Dec 2010 17:57:56 +0000

korvin: ну вот, заодно и я может питон изучу =)

Добавлено 29.12.10, 18:00

Цитата Ho Im @ 23.12.10, 20:24

А вообще путь настоящих джедаев — это таблица конверсий всего в кельвины и из кельвинов во все. Таким образом можно избежать комбинаторного взрыва при попытки создать таблицы перевода из всего во все.

мне вообще мультиметоды больше нравятся

Добавлено 29.12.10, 18:04

Цитата Ho Im @ 25.12.10, 10:29

Тем более. Меня просто один вид шести подряд идущих print'ов покоробил, помню, на одной из прошлых работ за подобное за горло брали и канделябрами били.

а мне не нравится портить форматирование кода "многострочными строками", впрочем может я не знаю, но как поступать в этом случае:

...

print """abcdef

ghjklm

nopqrs

tuvwxyz"""

...

[Py] Учу питон, поругайте за код

Ho Im — Sat, 25 Dec 2010 10:29:22 +0000

Ho Im: Тем более. Меня просто один вид шести подряд идущих print'ов покоробил, помню, на одной из прошлых работ за подобное за горло брали и канделябрами били.

[Py] Учу питон, поругайте за код

Мяут-Настоящий — Sat, 25 Dec 2010 09:33:12 +0000

Мяут-Настоящий:

Цитата Ho Im @ 23.12.10, 20:24

Современные ребята делают так:

Я тут узнал, что современные ребята делают так:

parser = OptionParser()

parser.add_option('-d', '--directory', action="store",

type="string", dest="directory", metavar='DIR',

help='Explorer directory path')

parser.add_option('-t', '--tar-file', action="store",

type="string", dest="tarfile", metavar='TARFILE',

help='Explorer tar file path')

parser.add_option('-c', '--config', action="store",

type="string", dest="config", metavar='CONFIG',

help='Explorer analyzer configuration file')

(options, args) = parser.parse_args()

И не парятся :D

[Py] Учу питон, поругайте за код

Lumberjack — Sat, 25 Dec 2010 03:51:02 +0000

Lumberjack: Ho Im, жжошь :D

[Py] Учу питон, поругайте за код

Ho Im — Thu, 23 Dec 2010 20:24:13 +0000

Ho Im:

Цитата korvin @ 05.09.10, 07:39

print >> sys.stderr, 'wrong integer: %s' % arg

Современные ребята делают так:

sys.stderr.wrtite("Wrong integer: {0}\n".format(arg))

Цитата

def usage():

print 'usage:'

print '\t-h, --help \tprint usage summary'

print '\t --modes \tprint available modes'

print '\t-d \tshow debugging information while running'

print '\t-m <m>, --mode=<m> \tuse <m> as conversion mode'

print '\t-t <t>, --temp=<t> \tuse <t> as temperature to convert'

За это убивают. На каждую строчку print? Помилуйте, это же не GW-BASIC!

def usage():

usage = """Usage:

-h, --help print this message

--modes print available modes

-d show debugging information while running

-m MODE, --mode=MODE

use MODE as conversion mode

-t TEMP, --temp=TEMP

use TEMP as temperature to convert

\n"""

sys.stdout.write(usage)

Добавлено 23.12.10, 20:28
А вообще путь настоящих джедаев — это таблица конверсий всего в кельвины и из кельвинов во все. Таким образом можно избежать комбинаторного взрыва при попытки создать таблицы перевода из всего во все.

И тогда режим задается как:
tempconv --from F --to C 100

[Py] Учу питон, поругайте за код

Мяут-Настоящий — Thu, 23 Dec 2010 20:08:43 +0000

Мяут-Настоящий:

Цитата purpe @ 23.12.10, 15:11

А какова цель ?

Выучить питон :tong:

[Py] Учу питон, поругайте за код

Ho Im — Thu, 23 Dec 2010 16:36:09 +0000

Ho Im: Ты не поверишь: рубить капусту.

[Py] Учу питон, поругайте за код

purpe — Thu, 23 Dec 2010 15:11:47 +0000

purpe:

Цитата Мяут-Настоящий @ 12.08.10, 20:10

Учу питон

А какова цель ?

[Py] Учу питон, поругайте за код

korvin — Sun, 05 Sep 2010 07:39:50 +0000

korvin: вобщем, я бы сделал так:

#!/usr/bin/python

# -*- coding: utf-8 -*-

import sys

import getopt

Mode = dict()

Mode['c2f'] = lambda temp: temp * 9 / 5 + 32

Mode['f2c'] = lambda temp: (temp - 32) * 5 / 9

def modes():

print 'available modes:'

for m in Mode.keys():

print '\t%s' % m

def usage():

print 'usage:'

print '\t-h, --help \tprint usage summary'

print '\t --modes \tprint available modes'

print '\t-d \tshow debugging information while running'

print '\t-m <m>, --mode=<m> \tuse <m> as conversion mode'

print '\t-t <t>, --temp=<t> \tuse <t> as temperature to convert'

def convert(mode, temp):

if mode in Mode.keys():

return Mode[mode](temp)

else:

print >> sys.stderr, 'wrong mode "%s"' % mode

modes()

sys.exit(2)

def main(argv):

temp = 0

mode = Mode.keys()[0]

try:

opts, args = getopt.getopt(argv, "hm:t:d", ["help", "mode=", "temp=", "modes"])

except getopt.GetoptError:

usage()

sys.exit(2)

for opt, arg in opts:

if opt in ("-h", "--help"):

usage()

sys.exit()

elif opt == '--modes':

modes()

sys.exit()

elif opt == '-d':

global _debug

_debug = 1

elif opt in ("-m", "--mode"):

mode = arg

elif opt in ("-t", "--temp"):

try:

temp = int(arg)

except ValueError:

print >> sys.stderr, 'wrong integer: %s' % arg

sys.exit(2)

else:

usage()

sys.exit()

print convert(mode, temp)

if __name__ == "__main__":

main(sys.argv[1:])

[Py] Учу питон, поругайте за код

korvin — Sat, 04 Sep 2010 17:37:12 +0000

korvin:

Цитата Мяут-Настоящий @ 12.08.10, 20:10

Всем привет. Написал первую программу на питоне. Поругайте, ЧЯДНТ, что можно сделать лучше 8-)
Задача классическая - температурный конвертер в обе стороны

#!/usr/bin/python

# -*- coding: utf-8 -*-

import sys

mode = 'NULL'

mlist = ('c2f', 'f2c')

print 'Temperature converter'

while mode != 'exit':

# Вводим режим перевода

mode = raw_input('Enter mode [exit, c2f, f2c]:\t')

if mode == 'exit':

break

# Проверяем режим

if not mlist.count(mode):

print >> sys.stderr, 'You\'d entered wrong conv. mode'

continue

# Вводится температура

rtr = 5

while rtr:

try:

temp = int(raw_input('Enter temperature in %s: \t' % {'c2f': 'C', 'f2c' : 'F'}[mode] ))

rtr = 0

except ValueError:

rtr -= 1;

if not rtr:

print >> sys.stderr, 'Failed five times, exiting';

sys.exit()

print >> sys.stderr, 'Wrong integer. %d retries left' % rtr;

#Переводим фарингейты

if mode == 'c2f':

print 'Temperature %d C or %d F' % (temp, temp * 9 / 5 + 32)

else:

print 'Temperature %d F or %d C' % (temp, (temp - 32) * 5 / 9)

print '\n--------------------\n'

while mode != 'exit':

# Вводим режим перевода

mode = raw_input('Enter mode [exit, c2f, f2c]:\t')

if mode == 'exit':

break

двойная проверка -- как-то тупо, лучше наверное сделать так:

prompt = 'Enter mode [exit, c2f, f2c]:\t'

mode = raw_input(prompt)

while mode != 'exit':

# ...

# и в самом конце тела цикла

mode = raw_input(prompt)

и ещё, "задача" конечно простая и особо изощрятся может и не стоит, но я бы разместил (лямбда-)функции конвертации в хеш-таблице с ключём mode =)

и если не ошибаюсь, отступ в два пробела не принят, лучше ставить 4

[Py] Учу питон, поругайте за код

Мяут-Настоящий — Thu, 12 Aug 2010 20:10:20 +0000

Мяут-Настоящий: Всем привет. Написал первую программу на питоне. Поругайте, ЧЯДНТ, что можно сделать лучше 8-)
Задача классическая - температурный конвертер в обе стороны

#!/usr/bin/python

# -*- coding: utf-8 -*-

import sys

mode = 'NULL'

mlist = ('c2f', 'f2c')

print 'Temperature converter'

while mode != 'exit':

# Вводим режим перевода

mode = raw_input('Enter mode [exit, c2f, f2c]:\t')

if mode == 'exit':

break

# Проверяем режим

if not mlist.count(mode):

print >> sys.stderr, 'You\'d entered wrong conv. mode'

continue

# Вводится температура

rtr = 5

while rtr:

try:

temp = int(raw_input('Enter temperature in %s: \t' % {'c2f': 'C', 'f2c' : 'F'}[mode] ))

rtr = 0

except ValueError:

rtr -= 1;

if not rtr:

print >> sys.stderr, 'Failed five times, exiting';

sys.exit()

print >> sys.stderr, 'Wrong integer. %d retries left' % rtr;

#Переводим фарингейты

if mode == 'c2f':

print 'Temperature %d C or %d F' % (temp, temp * 9 / 5 + 32)

else:

print 'Temperature %d F or %d C' % (temp, (temp - 32) * 5 / 9)

print '\n--------------------\n'