Я пытаюсь получить данные из одной базы данных (db), которая находится на другом сервере, и вставить полученные данные в другую базу данных (db1) на другом сервере. Пока я могу выбирать данные из db, но не могу вставить их в db1. Я получаю нулевые значения в базе данных. Любая помощь будет оценена.

Мой фрагмент кода ниже:

func searchHandler(w http.ResponseWriter, r *http.Request) {
dsn := "server=*****.ecc4q.****.com; user id=******; password=***************"
db, err := sql.Open("mssql", dsn) 
if err != nil {
    log.Fatal(err)
}
err = db.Ping()
if err != nil {
    log.Fatal(err)
}


rows, err := db.Query("select Incident_Number, Last_Resolved_Date, Corporate_ID from ITSM.dbo.HPD_Help_Desk_Classic")
if err != nil {
    log.Println(err)
}
defer db.Close()

dsn1 := "server=******.md3q.***.com;user id=*****;password=********"
db1, err := sql.Open("mssql", dsn1) 
if err != nil {
    log.Fatal(err)
}
err = db1.Ping()
if err != nil {
    log.Fatal(err)
}

for rows.Next() {
    err := rows.Scan(&Incident_Number, &Last_Resolved_Date, &Corporate_ID)
        if err != nil {
            log.Fatal(err)
        }

    stmt1, err := db1.Prepare("INSERT INTO mdesk.dbo.tbl_tcktinfo(TicketNumber, ResolvedDate, CDSID) VALUES(?,?,?)")
        if err != nil {
            log.Println(err)
        }

    res, err := stmt1.Exec(TicketNumber, ResolvedDate, CDSID)
        if err != nil {
            log.Println(err)
        }
    log.Println(Incident_Number, Last_Resolved_Date, Corporate_ID)
}
defer db1.Close() 

}

2
Dipikanta Dutta 14 Сен 2018 в 12:52

2 ответа

Лучший ответ

Вы просматриваете записи в Incident_Number, Last_Resolved_Date, Corporate_ID из таких строк, как:

err := rows.Scan(&Incident_Number, &Last_Resolved_Date, &Corporate_ID)

Но вы передаете другой аргумент в функции Exec(). Вы должны передать отсканированные переменные для вставки записей в другую таблицу базы данных.

stmt1, err := db1.Prepare("INSERT INTO mdesk.dbo.tbl_tcktinfo(TicketNumber, ResolvedDate, CDSID) VALUES(?,?,?)")
if err != nil {
    log.Fatal(err)
}
defer stmt1.Close() // always clean up after yourself, preferably return errors here, too

for rows.Next() {
    err := rows.Scan(&Incident_Number, &Last_Resolved_Date, &Corporate_ID)
        if err != nil {
            log.Fatal(err)
        }


    res, err := stmt1.Exec(Incident_Number, Last_Resolved_Date, Corporate_ID)
        if err != nil {
            log.Println(err)
        }
    log.Println(Incident_Number, Last_Resolved_Date, Corporate_ID)
}
3
Elias Van Ootegem 14 Сен 2018 в 11:41

Дополнение к принятому ответу: я бы порекомендовал вам сделать еще несколько вещей, чтобы обеспечить последовательность, и более «ответственно» обращаться с вашими ресурсами. Вы, конечно же, закрываете свои соединения с БД, но я бы привык обращаться с другими ресурсами почти таким же образом.

Более того, если что-то пойдет не так на полпути, вместо того, чтобы усекать таблицу и перезапускать процесс, я настоятельно рекомендую вам обернуть вставки в транзакцию:

defer rows.Close() // close your rows resource
defer db1.Close() // close at the very end
tx, err := db1.BeginTx(context.Background(), nil) // pass in a context with timeout or some other cancelable context if needed
if err != nil {
    return err // I'm returning, you can do as you like with this
}
stmt, err := tx.Prepare("INSERT INTO ...") // create the statement once
defer stmt.Close() // add to defer stack
for rows.Next() {
     // instead of your Camel_Snake_Cased vars, I'm using the preferred style and camelCase your var names instead...
    if err := rows.Scan(&incidentNumber, &lastResolvedDate, &corporateID); err != nil {
       tx.Rollback() // rollback transaction
       return err
    }

    if err := stmt.Exec(incidentNumber, lastResolvedDate, corporateID); err != nil {
        tx.Rollback() // again, rollback...
        return err
    }
}
if err := tx.Commit(); err !≃ nil {
    tx.Rollback() // not needed, but I add it as a habit
    return err
}
return nil

Теперь важно отметить, что все эти функции Close() и tx.Rollback() вызывают все ошибки возврата. Всегда лучше хотя бы заключить их в вызов журнала следующим образом:

log.Printf("Rolling back: %+v", tx.Rollback())
// and
log.Printf("Closing DB/ROWS/STMT: %+v", db1.Close()) // same for rows, stmt etc..
0
Elias Van Ootegem 14 Сен 2018 в 11:51