C# 脚本在不应该运行时运行执行行答案

【问题标题】：C# script running execute line when it shouldn'tC# 脚本在不应该运行时运行执行行
【发布时间】：2013-04-19 12:41:23
【问题描述】：

我是 C# 编程新手。我编写了一个查询 oracle dB 的脚本，如果行数 > 0，它应该运行一个 bat 文件，但如果不是，它应该结束。

这是我目前写的代码：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using Oracle.DataAccess.Client; // ODP.NET Oracle managed provider
using Oracle.DataAccess.Types;

namespace trigOra
{
    class Program
    {
        static void Main(string[] args)
        {
            string oradb = "Data Source=OMP1;User Id=user;Password=pwd;";

            OracleConnection conn = new OracleConnection(oradb); // C#

            conn.Open();

            OracleCommand cmd = new OracleCommand();
            cmd.Connection = conn;
            cmd.CommandText = "SELECT cast(Count(*) as varchar(20)) as trig FROM DMSN.DS3R_1XRTT_VOICE_TRIGGER";
            //cmd.CommandType = CommandType.Text;

            OracleDataReader dr = cmd.ExecuteReader();
            dr.Read();

            int cnt;
            if (int.TryParse(dr.GetString(0), out cnt))
            {
                if (cnt > 0)
                {
                    System.Diagnostics.Process.Start(@"C:\AutoBatch\backup\trigger.bat");
                    cmd.CommandText = "TRUNCATE TABLE DMSN.DS3R_1XRTT_VOICE_TRIGGER";
                    cmd.ExecuteNonQuery();
                }
            }
            conn.Dispose();

        }
    }
}

代码可以正常工作，只是它似乎总是执行这部分：

if (cnt > 0)
{
    System.Diagnostics.Process.Start(@"C:\AutoBatch\backup\trigger.bat");
    cmd.CommandText = "TRUNCATE TABLE DMSN.DS3R_1XRTT_VOICE_TRIGGER";
    cmd.ExecuteNonQuery();

即使返回的行数为 0。hmmmm...

【问题讨论】：

为什么需要cast as varchar？你可以改用dr.GetInt32/64()
另外，您似乎需要使用Process.Start(...).WaitForExit()，否则您将在备份完成之前运行 truncate。
尝试初始化 int cnt = 0;然后看看
我怀疑你的 Count 返回超过 0
@DanilaPolevshikov 我的值返回 0

标签： c# plsql

【解决方案1】：

您的 bat 文件必须使用 WaitForExit() 函数运行：

int exitCode = RunProcessForExitCode(filename, arguments);

    private static int RunProcessForExitCode(string processFilename, string processArguments)
    {
        //make process output possible to machine-reading
        Process process = new Process
        {
            StartInfo = new ProcessStartInfo
            {
                FileName = processFilename,
                Arguments = processArguments,
                UseShellExecute = false,
                RedirectStandardOutput = true,
                CreateNoWindow = true
            }
        };
        process.Start();
        Console.Write(process.StandardOutput.ReadToEnd());
        process.WaitForExit();
        return process.ExitCode;
    }

【讨论】：