تساعد على تحسين هذا INI تحليل التعليمات البرمجية

StackOverflow https://stackoverflow.com/questions/146943

  •  02-07-2019
  •  | 
  •  

سؤال

هذا شيء بسيط خطرت على هذا السؤال.أنا لست سعيدا تماما مع ذلك و رأيت أنها فرصة للمساعدة في تحسين استخدام المحكمة تيارات القائمة على البرمجة.

std::wifstream file(L"\\Windows\\myini.ini");
if (file)
{
  bool section=false;
  while (!file.eof())
  {
    std::wstring line;
    std::getline(file, line);
    if (line.empty()) continue;

    switch (line[0])
    {
      // new header
      case L'[':
      {
        std::wstring header;
        size_t pos=line.find(L']');
        if (pos!=std::wstring::npos)
        {
          header=line.substr(1, pos);
          if (header==L"Section")
            section=true;
          else
            section=false;
        }
      }
  break;
      // comments
      case ';':
      case ' ':
      case '#':
      break;
      // var=value
      default:
      {
        if (!section) continue;

// what if the name = value does not have white space?
// what if the value is enclosed in quotes?
        std::wstring name, dummy, value;
        lineStm >> name >> dummy;
        ws(lineStm);
        WCHAR _value[256];
        lineStm.getline(_value, ELEMENTS(_value));
        value=_value;
      }
    }
  }
}

كيف يمكن تحسين ذلك ؟ من فضلك لا ننصح البديلة المكتبات - أريد فقط طريقة بسيطة لتحليل بعض config سلاسل من INI.

هل كانت مفيدة؟

المحلول

// ماذا لو كان اسم = القيمة لا يكون أبيض الفضاء ؟
// ما إذا كانت القيمة المغلقة في الاقتباس ؟

وأود أن استخدام دفعة::regex لتتناسب مع كل نوع مختلف من عنصر شيئا مثل:

boost::smatch matches;
boost::regex name_value("(\S+)\s*=\s*(\S+)");
if(boost::regex_match(line, matches, name_value))
{
    name = matches[1];
    value = matches[2];
}

التعبيرات العادية التي قد تحتاج الى بعض التغيير والتبديل.

وأود أيضا استبدال دي تيار.getline مع std::getline, التخلص من ساكنة char array.

نصائح أخرى

هذا:

for (size_t i=1; i<line.length(); i++)
        {
          if (line[i]!=L']')
            header.push_back(line[i]);
          else
            break;
        }

ينبغي تبسيط دعوة إلى wstrchr, wcschr, WSTRCHR ، أو أي شيء آخر ، اعتمادا على منصة ما كنت على.

// كيفية الحصول على الخط في سلسلة واحدة ؟

استخدام (غير عضو) getline وظيفة من سلسلة القياسية رأس.

مرخصة بموجب: CC-BY-SA مع الإسناد
لا تنتمي إلى StackOverflow
scroll top