El código tal como está escrito ni siquiera llegó a guardar un archivo para mí. Parece que hay algunos problemas. No estoy seguro si este es el código real o si algunas cosas se perdieron en el copiar y pegar. Sin embargo, según lo que tengas:
Uno grande es que nunca te conectas a la base de datos en tu código con connection.connect().
El código que desea ejecutar una vez conectado debe estar dentro de la devolución de llamada de connection.connect(). por ejemplo
connection.connect(function (err, empty) {
if (err)
throw new Error ('Panic');
// if no error, we are off to the races...
}
Sin embargo, incluso si refactoriza rápidamente su código para envolver sus últimas líneas dentro de esa devolución de llamada de conexión, aún tendrá problemas, porque está destruyendo la conexión antes de que se realicen las diversas llamadas SQL, por lo que querrá mover el código en algún tipo de devolución de llamada final.
Incluso después de hacer eso, aún tendrá un archivo vacío, porque está llamando a save_backup desde su devolución de llamada 'MOSTRAR TABLAS' en lugar de después de haberlo completado a través de la devolución de llamada interna donde obtiene la declaración CREATE TABLE y completa el propiedad de copia de seguridad.
Esta es la reescritura mínima de su código que hará lo que pretende. Una cosa importante a tener en cuenta es el "contador" que administra cuándo escribir el archivo y cerrar la conexión. Haría otros cambios si fuera mío, incluyendo:
- Usar 'yo' en lugar de 'yo'
- Utilizar un bucle for numérico en lugar de la sintaxis for (... en ...)
- Hacer que mis propias devoluciones de llamada caigan en la convención de nodo de (err, esas cosas)
- Un cambio más sustancial es que volvería a escribir esto para usar promesas, ya que hacerlo puede ahorrarle algo de dolor con la confusión inherente a las devoluciones de llamada profundamente anidadas. Personalmente, me gusta la biblioteca Q, pero aquí hay varias opciones.
Espero que esto haya ayudado.
var mysql_backup = function(){
this.backup = '';
this.mysql = require('mysql');
this.init = function(){
this.connection = this.mysql.createConnection({
user : 'root',
password : 'root',
database : 'test'
});
};
this.query = function(sql, callback) {
this.connection.query(sql, function (error, results, fields) {
if (error) {
throw error;
}
if (results.length > 0) {
callback(results);
}
});
};
this.get_tables = function(callback){
var counter = 0;
var me = this;
this.query('SHOW TABLES',
function(tables) {
for (table in tables){
counter++;
me.query(
'SHOW CREATE TABLE ' + tables[table].Tables_in_mvc,
function(r){
for (t in r) {
me.backup += "DROP TABLE " + r[t].Table + "\n\n";
me.backup += r[t]["Create Table"] + "\n\n";
}
counter--;
if (counter === 0){
me.save_backup();
me.connection.destroy();
}
}
)
}
});
};
this.save_backup = function(){
var fs = require('fs');
fs.writeFile("./backup_test.txt", this.backup, function(err) {
if(err) {
console.log(err);
} else {
console.log("The file was saved!");
}
});
}
};
var db = new mysql_backup;
db.init();
db.connection.connect(function (err){
if (err) console.log(err);
db.get_tables(function(x){;});
});
Actualización:si tiene curiosidad, aquí hay una implementación muy comentada que usa promesas. Tenga en cuenta que sin los comentarios que explican las funciones de la biblioteca Qpromise, es algo más breve que la versión original y también ofrece un manejo de errores más completo.
var MysqlBackup = function(connectionInfo, filename){
var Q = require('q');
var self = this;
this.backup = '';
// my personal preference is to simply require() inline if I am only
// going to use something a single time. I am certain some will find
// this a terrible practice
this.connection = require('mysql').createConnection(connectionInfo);
function getTables(){
// return a promise from invoking the node-style 'query' method
// of self.connection with parameter 'SHOW TABLES'.
return Q.ninvoke(self.connection,'query', 'SHOW TABLES');
};
function doTableEntries(theResults){
// note that because promises only pass a single parameter around,
// if the 'denodeify-ed' callback has more than two parameters (the
// first being the err param), the parameters will be stuffed into
// an array. In this case, the content of the 'fields' param of the
// mysql callback is in theResults[1]
var tables = theResults[0];
// create an array of promises resulting from another Q.ninvoke()
// query call, chained to .then(). Note that then() expects a function,
// so recordEntry() in fact builds and returns a new one-off function
// for actually recording the entry (see recordEntry() impl. below)
var tableDefinitionGetters = [];
for (var i = 0; i < tables.length ; i++){
// I noticed in your original code that your Tables_in_[] did not
// match your connection details ('mvc' vs 'test'), but the below
// should work and is a more generalized solution
var tableName = tables[i]['Tables_in_'+connectionInfo.database];
tableDefinitionGetters.push(Q.ninvoke(self.connection, 'query', 'SHOW CREATE TABLE ' + tableName)
.then(recordEntry(tableName)) );
}
// now that you have an array of promises, you can use Q.allSettled
// to return a promise which will be settled (resolved or rejected)
// when all of the promises in the array are settled. Q.all is similar,
// but its promise will be rejected (immediately) if any promise in the
// array is rejected. I tend to use allSettled() in most cases.
return Q.allSettled(tableDefinitionGetters);
};
function recordEntry (tableName){
return function(createTableQryResult){
self.backup += "DROP TABLE " + tableName + "\n\n";
self.backup += createTableQryResult[0][0]["Create Table"] + "\n\n";
};
};
function saveFile(){
// Q.denodeify return a promise-enabled version of a node-style function
// the below is probably excessively terse with its immediate invocation
return (Q.denodeify(require('fs').writeFile))(filename, self.backup);
}
// with the above all done, now you can actually make the magic happen,
// starting with the promise-return Q.ninvoke to connect to the DB
// note that the successive .then()s will be executed iff (if and only
// if) the preceding item resolves successfully, .catch() will get
// executed in the event of any upstream error, and finally() will
// get executed no matter what.
Q.ninvoke(this.connection, 'connect')
.then(getTables)
.then(doTableEntries)
.then(saveFile)
.then( function() {console.log('Success'); } )
.catch( function(err) {console.log('Something went awry', err); } )
.finally( function() {self.connection.destroy(); } );
};
var myConnection = {
host : '127.0.0.1',
user : 'root',
password : 'root',
database : 'test'
};
// I have left this as constructor-based calling approach, but the
// constructor just does it all so I just ignore the return value
new MysqlBackup(myConnection,'./backup_test.txt');
