5/31宿題

[cr4ftsm4n]

• 014_armstrong-numbers
• 015_rotational-cipher
• 016_difference-of-squares
• 017_anagram
• 018_allergies

[cr4ftsm4n]

cuiHaiL

• 014_armstrong-numbers OK
• 015_rotational-cipher OK, but by defining the constants will make the answer better

  ORD_A = ord("A") // 65
  ORD_Z = ord("Z") // 90
  ORD_LOWER_A = ord("a") //97
  ORD_LOWER_Z = ord("z") //122

• 016_difference-of-squares LGTM, but can be solved by sum and list comprehension
• 017_anagram where is the function detect_anagrams :)

  
  Ran 12 tests in 0.002s

FAILED (failures=6)

* 018_allergies
素晴らしい

[cr4ftsm4n]

yan19900908

• 014_armstrong-numbers

  tostrnum = str(number)
  ...
  if tostrnum.isdigit(): # もちろん数字だよ

• 無駄なCPU cycle
• raiseの書き方が変
• 015_rotational-cipher LGTM

  for i in text:

  ⬇️

  for c in text:

  charで遍歴するから、変数名はi(intの場合多用)よりc(charの場合多用)の方が相応しい気がする。

• 016_difference-of-squares 素晴らしい でも

  from functools import reduce

  忘れてない

• 017_anagram OK, でも、テステケースに合わせた感じがする。
• 018_allergies dictitemを全部Allergies.dictitemに直したら、テストOK

[cr4ftsm4n]

fangcaoyouqing

• 014_armstrong-numbers LGTM
• 015_rotational-cipher 大文字と小文字別々処理しないとね
• 016_difference-of-squares 素晴らしい、sum_of_squaresもlist comprehensionにしたら。。。
• 017_anagram 頑張ってね
• 018_allergies iPad大変でしょうね

[cr4ftsm4n]

tyoukoumei

• 014_armstrong-numbers LGTM
• 015_rotational-cipher OK, but
  1. elif x.isupper() == True: => elif x.isupper():
  2. from string import ascii_lowercase, ascii_uppercase
• 016_difference-of-squares Test OK,
  1. range利用した方が
  2. list comprehensionを活用した方が

  3.  def difference(count):
          return square_of_sum(count) - sum_of_squares(count)

• 017_anagram Test OK, break ではない、continueではないでしょうか。
• 018_allergies 頑張ってね

[cr4ftsm4n]

wangqingwei029

• 014_armstrong-numbers OK
• 015_rotational-cipher OK changeStrのlet引数は、状態二種類であれば、is_lowerのbool引数にしたほがよい、 複数の状態場合、

  STATE_A = 1
  STATE_B = 2
  ...

  のようにintにした方がよい

• 016_difference-of-squares LGTM
• 017_anagram LGTM

  def detect_anagrams(word, candidates):
  s = []
  for x in candidates:
      if len(word) == len(x):
          ls1 = list(word.lower())
          ls2 = list(x.lower())
          if ls1 == ls2:
              break
          ls1.sort()
          ls2.sort()
          if ls1 == ls2:
              s.append(x)
  return s

• 018_allergies LGTM

全問正解、素晴らしい

[cr4ftsm4n]

Michaelsya9

• 014_armstrong-numbers LGTM n=number変数のrenameあんまりいらない気がする、やっぱ変な変数名だと、気に入らない。
• 015_rotational-cipher 問題の解き方、発想は素晴らしい 変数名はsnake_case
• 016_difference-of-squares TEST OK,

     if(i == count):
       break 

  最後に行ったら、わざわざbreakしなくても、loopから抜けるので、

• 017_anagram TEST OK, 最後のテストケースに合わせた気がする
• 018_allergies 頑張ってね

[cr4ftsm4n]

zhou1224271

• 014_armstrong-numbers OK 空行入れすぎで、逆に読みづらい
• 015_rotational-cipher LGTM
• 016_difference-of-squares LGTM
• 017_anagram LGTM

• 018_allergies LGTM

  def is_allergic_to(self, item):
  
      return item in self.lst(self)

  でpython3.6に下記エラー発生する。 TypeError: 'list' object is not callable

  def is_allergic_to(self, item):
  
      return item in self.lst

  だと、テストOK

ほぼ全問正解

[cr4ftsm4n]

ignited0411

• 014_armstrong-numbers OK, but

  for i  in range(0,length):
      sum = int(str(number)[i:i+1])**length + sum

  ↓

  for c  in str(number):
      sum += int(c) ** length

  の方がよい。

• 015_rotational-cipher OK 65, 91, 97, 123は、何処から来たの、ぱっと見ではわからないな。 ord("A"), ord("Z")+1, ord("a"), ord("z")+1 の方がよいと思う。
• 016_difference-of-squares 素晴らしい。list comprehensionも完璧。
• 017_anagram LGTM
• 018_allergies OK

  全問正解、素晴らしい

[cr4ftsm4n]

zhangyan0814

pull requestしてくれて、ありがとう。そちらにレビューするので、そちらに参照してくささい。

[cr4ftsm4n]

• @cuiHaiL
• @yan19900908
• @fangcaoyouqing
• @tyoukoumei
• @wangqingwei029
• @Michaelsya9
• @zhou1224271
• @ignited0411
• @zhangyan0814

お疲れ様です。コメントしました。 全体的に以下の問題散見します。

• naming、pythonの変数はsnake_caseなので、camelCaseではない。
• 良くつけた変数名は、ドキュメント代わりに使える。
• anagram 最後のテストケースは、wordと一致する（無視upper,lower）場合、anagramではない。
• allergiesはbitwise処理が必要です。( &, | , ~, ^ ) 0b0100 & 0b1111 = 0b0100 0b0100 & 0b1000 = 0b0000
• python 2
  1. xrange()
  2. raise 'error message'
  3. reduce()グローバル関数