このコードはDBに1行しか書いていませんか？

Question

このコードはDBに1行しか書いていません

このコードではエラーは見つかりませんでした。 。 。

しかし、なぜこれが最初の行以上を挿入していないのですか？

def transremovechars(): 
    cur.execute('drop table transforms') 
    char_cfg = config.get('Transform_Variables', 'Chars_to_be_removed')  #Reads all the special chars to be removed from specialchars.txt# 
    cur.execute('select * from originallist') 
    for row in cur:               #Applies transformation to remove chars for each row in a loop# 
     company = row[0] 
     for specialchars in char_cfg: 
      company = company.replace(specialchars, '') 
     cur.execute('Insert into transforms (Transresult1) values (\'' + company + '\')') 
    con.commit() 

Answer 1

あなたはcur.fetchall()を忘れてしまった：

def transremovechars(): 
    cur.execute('drop table transforms') 
    char_cfg = config.get('Transform_Variables', 'Chars_to_be_removed')  #Reads all the special chars to be removed from specialchars.txt# 
    cur.execute('select * from originallist') 
    for row in cur.fetchall():               #Applies transformation to remove chars for each row in a loop# 
     company = row[0] 
     for specialchars in char_cfg: 
      company = company.replace(specialchars, '') 
     cur.execute('Insert into transforms (Transresult1) values (\'' + company + '\')') 
    con.commit() 

Answer 2

あなたはそれを使用する前に、あなたのテーブルtransformsをドロップするように見えます。あなたはそれをしてもよろしいですか？あるいは、もう一度それを作成するコードを表示することを忘れてしまったでしょうか？

select *は、第1列のみを使用すると過剰になることがあります。おそらく、そのフィールドの名前をSELECTにしたいと思うかもしれません。

以外にも、あなたがカーソルの上に

cur.execute('Insert into transforms (Transresult1) values (?)', company) 

反復処理を使用してINSERT行を置き換える必要がありますが、問題ないはずです。たぶん、あなたのforループにいくつかprintステートメントを挿入することができます...

Answer 3

cur.fetchall()とそれ以上の反復を行うべきであるとのコメントはうまくいくでしょう。あなたのコードの真の欠点は、一度curを挿入すると、それは「新しいもの」で、元のジェネレータはリセットされます（curはnext()です）。

fetchallを使わずにcurを使用することができます.2番目のカーソルins_cur = con.curson()を作成し、両方を使用します。もっと多くののエフェクトは、1つの接続で複数のカーソルを開いたり、複数のカーソルを開いたりすることで実現できます。

また、正しいdbapiモジュールのvariable bindingを使用してください。