Программа синтаксического анализа: как я неправильно загружаю свой объект? - PullRequest
Новая
       27

Программа синтаксического анализа: как я неправильно загружаю свой объект?

0 голосов
/ 10 января 2012

По моему мнению, блоки if (isStart($line)){} и if (isEnd($line)) помещают вещи в неправильную область видимости.Проблемная область комментируется как "/***PROBLEM AREA */".

Это моя программа синтаксического анализа: 

<code><?php

  //**CLASS AND OBJECT */
  class Entry
  {
    private $reason;
    private $s_id;

    public function __construct()
    {
      $this->reason     = '';
      $this->s_id       = '';
    }

      //** GETTERS AND SETTERS */
    public function SetReason($reason)
    {
      $this->reason = $reason;
    }

    public function GetReason()
    {
      return $this->reason;
    }

    public function SetS_id($s_id)
    {
      $this->s_id = $s_id;
    }

    public function GetS_id()
    {
      return $this->s_id;
    }
  }

  //** EXTRACTION FUNCTION(S)
  function extractReason($line)
  {
    $matches;
    preg_match('/^Reason:\s+(.*)\s+$/', $line, $matches);
    return $matches[1];
  }  

  function extractS_id($line)
  {
    $matches;
    preg_match('/^S_id:\s+(.*)\s+$/', $line, $matches);
    return $matches[1];
  }

  //** LINE CONTAINST DESIRED EXTRACTION CHECK */
  function isStart($line)
  {
    return preg_match('/^Start$/', $line);
  }

  function isReason($line)
  {
    return preg_match('/^Reason:\s+(.*)$/', $line);
  }

  function isS_id($line)
  {
    return preg_match('/^S_id:\s+(.*)$/', $line);
  }

  function isContent($line)
  {
    return preg_match('/.*$/', $line);
  }

  function isEnd($line)
  {
    return preg_match('/^End$/', $line);
  }




  //** DEFINITION */
  $fName = 'obfile_extractsample.txt';
  $fh    = fopen($fName, 'r');
  $line;
  $entry;
  $entrys = array();

  //** PARSE OPERATION
  if ($fh === FALSE)
    die ('Failed to open file.');

  //**START PROBLEM AREA */
  while (($line = fGets($fh)) !== FALSE)
  {
    if (isStart($line)){
      $entry = new Entry();
      if (isReason($line)){
        $entry->SetReason(extractReason($line));
      }
      if (isS_id($line)){
        $entry->SetS_id(extractS_id($line));
      }
      if (isEnd($line)){
        $entrys[] = $entry;
      }
    }
  }
  //***END PROBLEM AREA */

  echo "<pre>";
    print_r($entrys);
  echo "
"; fclose ($ fh);?>

Thisмой пример файла: 

Start
Name:      David Foster  
Out Time:  4:36 p.m.    
Back Time: 4:57 p.m.
Reason:    Lunch
S_id:      0611125
End

Start
Name:      Brenda Banks  
Out Time:  5:53 a.m.    
Back Time: 6:30 a.m.
Reason:    Personal
S_id:      0611147
End

Это вывод: 

Array()

ВАЖНОЕ РЕДАКТИРОВАНИЕ

Строка регулярного выражения в * 1021Функции * и isEnd были неправильными. До конца строки было несколько невидимых символов. Правильные шаблоны регулярных выражений: '/^Start.*$/' и '/^End.*$/'

1 Ответ

1 голос
/ 10 января 2012

Этот бит: 

    if (isContent($line)){
      $entry = new Entry();
      $entry->SetReason(extractReason($line));
      $entry->SetS_id(extractS_id($line));
      $entrys[] = $entry;
    }

всегда создаст новый Entry и добавит его в ваш массив, но единственные поля, которые он может обнаружить, это Reason: ... и Style: ....Таким образом, большинство строк приводят к пустым Entry с.

...