Пропуск строк комментариев, начиная с // в C++

Question

У меня есть задание для моего класса Software Engineering, который заставляет меня бананы. Мне было предложено создать счетчик строк, который учитывает только логические строки кода для любого заданного файла. Он должен опускать пустые строки и комментарии.

У меня код очень много работает, за исключением того, что он перечитывает номера строк на 2 строки независимо от того, какой файл я передаю в него. Я не могу для жизни меня видеть, где моя проблема, и было интересно, может ли кто-нибудь помочь мне.

Вот мой код:

#include <iostream> 
#include <fstream> 
#include <string> 
#include <cstring> 
#include <stdio.h> 

using namespace std; 

int main() { 

    // Initialize variables 
    ifstream infile; 
    string filename; 
    int line = 0; 

    // Get file input 
    cout << "Enter the filename" << endl; 
    cin >> filename; 

    // open the file 
    infile.open(filename.c_str()); 

    // read the lines and skip blank lines and comments 
    while(getline(infile, filename)) { 
     if(filename.empty() || filename.find("//") == true) { 
      continue; 
     } 

     // increment the line number 
     ++line; 
    } 

    // close the file 
    infile.close(); 

    // display results 
    cout << "There are " << line << " lines of code in this file." << endl; 
} 

Счетчик гласит в терминале: «Есть 24 строк кода в этом файле.»

Согласно моим расчетам, должно быть только 22 строки логического кода. Помощь была бы оценена.

Answer 1

Почему вы не добавляете инструкцию печати, такую ​​как cout << filename << '\n';, чтобы идентифицировать линии, которые она идентифицирует? Я подозреваю, что вы увидите несколько пустых строк.

Я подозреваю, что вам нужно обрезать пробелы из ваших строк. Вероятно, у вас есть пустые строки, содержащие пробелы или вкладки. Следовательно, они не являются технически пустыми до значения str :: empty.

Кроме того, выполняя обрезку и исправляя другую ошибку, которую я вижу в вашем коде, рассматривая комментарий «//» в качестве комментария.

Следовательно, это становится простым исправлением с обрезками.

while(getline(infile, filename)) { 

    filename = ltrim(filename); // remove leading whitespace 
    filename = rtrim(filename); // remove trailing whitespace 

    if(filename.empty() || (filename.find("//") == 0)) { 
     continue; 
    } 

    // increment the line number 
    ++line; 
} 

Вы можете найти реализацию RTRIM и LTrim on this other SO answer here.

Answer 2

Заменить filename.find("//") == true с filename.find("//") == 0, чтобы найти строки, которые начинаются с //, так что он не будет думать, что линии, как int i = 0; // comment не являются строки кода.

Answer 3

1. empty() означает «никакого персонажа вообще», в то время как вы также хотите отфильтровать строки, содержащие только пробелы ant tabs.
2. вы можете иметь код в строках, которые содержат // (например int i; // loop index, поэтому фильтрация их получите неправильные результаты

верный способ делать вещи будет:.

1. стрип комментарии
2. полоса
пробельные символы

, а затем увидеть, если оставшаяся строка пуста или нет.

Обработка строк в C/C++ является жалкой, поэтому вы будете 1.000.000 бедным парнем, вынужденным писать (или копировать) код, необходимый для выравнивания пространств на обоих концах строки.

кстати. вызывающий filename переменная, которая должна удерживать текущую строку, не является ярлыком из банановой рощи.