2016-07-14 2 views
-2

これは、トレースバックの呼び出しがエラーを示し、ここでは私のコードスニペットです:私はここに全体のコードを与えられていないなぜ私はTypeErrorを取得していますか?整数が必要ですか?

Traceback (most recent call last): 
File "categorize_words.py", line 93, in <module> 
query_database() 
File "categorize_words.py", line 45, in query_database 
categorize(row) 
File "categorize_words.py", line 67, in categorize 
for i in xrange (start,end): 
TypeError: an integer is required 

def categorize(title): 
    with conn: 
     cur= conn.cursor() 
     title_str= str(title) 
     title_words= re.split('; |, |\*|\n',title_str) 
     key_list= list(dictionary.keys()) 
     flag2= 1 
     for word in title_words: 
     title_letters= list(word) 
     flag1= 1 
     for key in key_list: 
      if key==title_letters[0]: 
       flag1= 0 
       break 

     if flag1== 0: 

     start=dictionary.get(title_letters[0]) 
     end= next_val(title_letters[0]) 

     for i in xrange (start,end): 
      if word==transfer_bag_of_words[i]: 
       flag2= 0 
       break 

     if flag2== 0: 
     cur.execute("select Id from articles where title=title") 
     row_id= cur.fetchone() 
     value= (row_id,'1') 
     s= str(value) 
     f.write(s) 
     f.write("\n") 
     break 

    return 


def next_val(text): 
    for i,v in enumerate(keyList): 
     if text=='t': 
     return len(transfer_bag_of_words) 
     elif v==text: 
     return dictionary[keyList[i+1]] 

これは、トレースバックコールです。しかし、私が何をしようとしているのかを説明します。私はsqliteデータベースから特定のフィールドをインポートしようとしていると、フィールドの単一の単語が私のプログラムですでに持っている単語の特定のバッグと一致するかどうかをチェックしています。私は単語の袋をアルファベット順にソートし、新しい文字の始まりがすべてpython辞書を使ってそのインデックスに割り当てられるようにしました。これは私がやったことで、単語の袋に現れているフィールドの単語をチェックする度に、単語の袋全体をループする必要はありません。むしろ、私は単語の最初の文字のインデックスからループを開始することができます。

len()とdictionary [keylist [i + 1]]の両方がint型であるため、辞書のget()の戻り値の型がintで、関数nextValもintを返すことを確認しました。

助けてください。

EDIT

これは私の全体のコードです:

import sqlite3 as sql 
import re 

conn= sql.connect('football_corpus/corpus2.db') 

transfer_bag_of_words=['transfer','Transfer','transfers','Transfers','deal','signs','contract','rejects','bid','rumours','swap','moves', 
        'negotiation','negotiations','fee','subject','signings','agreement','personal','terms','pens','agent','in','for', 
        'joins','sell','buy','confirms','confirm','confirmed','signing','renew','joined','hunt','excited','move','sign', 
        'loan','loaned','loans','switch','complete','offer','offered','interest','price','tag','miss','signed','sniffing', 
        'remain','plug','pull','race','targeting','targets','target','eye','sale','clause','rejected', 
        'interested'] 

dictionary={} 
dictionary['a']=0; 
keyList=[] 
f= open('/home/surya/Twitter/corpus-builder/transfer.txt','w') 

def map_letter_to_pos(): 
    pos=0 
    transfer_bag_of_words.sort() 
    for word in transfer_bag_of_words: 
     flag=1 
     letters= list(word) 
     key_list= list(dictionary.keys()) 
     for key in key_list: 
     if key==letters[0]: 
      flag=0 
      break 

     if flag==1: 
     dictionary[letters[0]]=pos 
     pos+=1 
     else: 
     pos+=1 

    keyList= sorted(dictionary.keys()) 

def query_database(): 
    with conn: 
     cur= conn.cursor() 
     cur.execute("select title from articles") 
     row_titles= cur.fetchall() 

     for row in row_titles: 
     categorize(row) 

def categorize(title): 
    with conn: 
     cur= conn.cursor() 
     title_str= str(title) 
     title_words= re.split('; |, |\*|\n',title_str) 
     key_list= list(dictionary.keys()) 
     flag2= 1 
     for word in title_words: 
     title_letters= list(word) 
     flag1= 1 
     for key in key_list: 
      if key==title_letters[0]: 
       flag1= 0 
       break 

     if flag1== 0: 

     start=dictionary.get(title_letters[0]) 
     end= next_val(title_letters[0]) 

     for i in xrange (start,end): 
      if word==transfer_bag_of_words[i]: 
       flag2= 0 
       break 

     if flag2== 0: 
     cur.execute("select Id from articles where title=title") 
     row_id= cur.fetchone() 
     value= (row_id,'1') 
     s= str(value) 
     f.write(s) 
     f.write("\n") 
     break 

    return 


def next_val(text): 
    for i,v in enumerate(keyList): 
     if text=='t': 
     return len(transfer_bag_of_words) 
     elif v==text: 
     return dictionary[keyList[i+1]] 

if __name__=='__main__': 
    map_letter_to_pos() 
    query_database() 

そして、これがhttp://wikisend.com/download/702374/corpus2.db

+2

[mcve]が必要です。これらの2つの変数の型が本当にintだった場合、これは問題ではありません。 –

+0

@joelgoldstick分かりません。私はエラーを与えるコードの一部を提供しました –

+1

'start'や' end'は整数ではありません。私はあなたが彼らがいることを確認したと言ったが、あなたは間違っていると知っている。 –

答えて

0

map_letter_to_posがそれを指定せずにグローバル変数キーリストを変更しようとするデータベースファイルへのダウンロードリンクですグローバル変数として使用するため、keyListのローカルコピーを変更してから捨てるだけです。これにより、next_valには何も反復されないので、if elifには決して到達せず、Noneを返します。

end = None 
range(start,end) # None is not an int 
+0

も破棄されました。あなたが見たことがないとき、あなたは始まりと終わりがintであると確信していました。 :) –

関連する問題