Печать из структуры, переданной по значению в C - PullRequest
0 голосов
/ 22 февраля 2019

TL; DR:

printf() печатает мусор в printLotInfo() при передаче по значению.

Код

#include <stdio.h>
#include <stdlib.h>
#include <stddef.h>
#include <math.h>
#include <string.h>

typedef struct Time
{
   int hour;                              //Hour of day
   int minute;                            //Minute of hour
} Time;

typedef struct Car
{
   char * plateNumber;                    //String to hold plate
   char hasPermit;                        //True/False
   Time * enteringTime;                   //Time Struct
   int lotParkedIn;                       //Where is the car located
} Car;

typedef struct ParkingLot
{
   int lotNumber;                         //Lot identifier
   double hourlyRate;                     //$$/h
   double maxCharge;                      //Maximum daily charge
   int capacity;                          //How many cars can be parked in the lot?
   int currentCarCount;                   //Tracks # of cars in lot
   double revenue;                        //How much money has this lot made?
} ParkingLot;

// Sets the hours and minutes amount for the given time t based
// on the specified hours h.  (e.g., 1.25 hours would be 1 hour
// and 15 minutes)
void setHours(Time *t, double h)
{
   if(h != -1)
   {
      t->hour = (int) h;                  //Bad practice but no overflow expected here
      t->minute = (h - t->hour) * 60.0;   //Cast truncates h. h - t.hour is h-truncated h
   }
   else
   {
      t->hour = -1;
      t->minute = -1;
   }
}

// Takes two Time objects (not pointers) and computes the difference
// in time from t1 to t2 and then stores that difference in the diff
// Time (which must be a pointer)
void difference(Time t1, Time t2, Time *diff)
{
   diff->hour = t2.hour - t1.hour;
   diff->minute = t2.minute - t1.minute;

   if(diff->minute < 0)
   {
      diff->hour--;                       //If minutes are negative, decrement hour
      diff->minute += 60;                 //and set minutes to complement of 60.
   }
}


// Initialize the car pointed to by c to have the given plate and
// hasPermit status.  The car should have it’s lotParkedIn set to
// 0 and enteringTime to be -1 hours and -1 minutes.
void initializeCar(Car *c, char *plate, char hasPermit)
{
   Time * t = malloc(sizeof(Time));       //Allocate memory for time object
   setHours(t, -1);                       //Set time with call to setHours()

   c->plateNumber = plate;
   c->hasPermit = hasPermit;              //Set variables
   c->lotParkedIn = 0;
   c->enteringTime = t;
}

/********************** PROBLEM SECTION ************************************/
// Initialize the lot pointed to by p to have the given number,
// capacity, hourly rate and max charge values.  The currentCarCount
// and revenue should be at 0.
void initializeLot(ParkingLot * p, int num, int cap, double rate, double max)
{
   p->lotNumber = num;
   p->hourlyRate = rate;
   p->maxCharge = max;
   p->capacity = cap;
   p->currentCarCount = 0;
   p->revenue = 0;
   printf("PRINTING IN initializeLot():\nLot is: %d Capacity is: %d Rate is: %.2lf Maximum is: %.2lf\n",
      p->lotNumber, p->capacity, p->hourlyRate, p->maxCharge);
}

// Print out the parking lot parameters so that is displays as
// follows:   Parking Lot #2 - rate = $3.00, capacity 6, current cars 5
void printLotInfo(ParkingLot p)
{
   printf("PRINTING IN printLotInfo():\n");
   printf("Parking Lot #%d - rate = $%.2lf, capacity %d, current cars %d\n",
      p.lotNumber, p.capacity, p.currentCarCount);
}
/************************* END PROBLEM SECTION *****************************/

// Simulate a car entering the parking lot
// ...

void carEnters(ParkingLot * p, Car * c, int hour, int minute)
{
   if((p->capacity - p->currentCarCount) <= 0)
   {
      printf("Car %s arrives at Lot %d at %d:%1.2d, but the lot is full\n",
         c->plateNumber, p->lotNumber, hour, minute);
      return;
   }

   double timeToSet = (hour + (minute / 60.0));
   setHours(c->enteringTime, timeToSet);

   c->lotParkedIn = p->lotNumber;
   p->currentCarCount += 1;

   printf("Car %s arrives at Lot %d at %d:%1.2d.\n", c->plateNumber, 
      p->lotNumber, hour, minute);
}

// Simulate a car leaving the parking lot
// ...

void carLeaves(ParkingLot * p, Car * c, int hour, int minute)
{
   Time * leaveTime = malloc(sizeof(Time));      
   Time * timeDifference = malloc(sizeof(Time));        //Allocate memory for time object

   double timeToSet = (hour + (minute / 60.0));
   setHours(leaveTime, timeToSet);                      //Set time with call to setHours()
   difference(*(c->enteringTime), *leaveTime, timeDifference);

   if(c->hasPermit == 0)
   {
      double carRevenue = p->hourlyRate * timeDifference->hour;
      if(timeDifference->minute != 0)
         carRevenue += p->hourlyRate;
      if(carRevenue > p->maxCharge)
         carRevenue = p->maxCharge;
      p->revenue += carRevenue;
      printf("Car %s leaves Lot %d at %d:%1.2d paid $%.2lf.\n", 
         c->plateNumber, p->lotNumber, hour, minute, carRevenue);
   }
   else
      printf("Car %s leaves Lot %d at %d:%1.2d.\n", 
         c->plateNumber, p->lotNumber, hour, minute);
   p->currentCarCount--;

   free(c->enteringTime);
   free(c);
   free(leaveTime);
   free(timeDifference);
}

/*BEGIN PROFESSOR'S CODE*/

int main() {
  Car  car1, car2, car3, car4, car5, car6, car7, car8, car9;
  ParkingLot p1, p2;

  // Set up 9 cars
  initializeCar(&car1, "ABC 123", 0);
  initializeCar(&car2, "ABC 124", 0);
  initializeCar(&car3, "ABD 314", 0);
  initializeCar(&car4, "ADE 901", 0);
  initializeCar(&car5, "AFR 304", 0);
  initializeCar(&car6, "AGD 888", 0);
  initializeCar(&car7, "AAA 111", 0);
  initializeCar(&car8, "ABB 001", 0);
  initializeCar(&car9, "XYZ 678", 1);

  // Set up two parking lots
  initializeLot(&p1, 1, 4, 5.5, 20.0);
  initializeLot(&p2, 2, 6, 3.0, 12.0);

  printLotInfo(p1);
  printLotInfo(p2);
  printf("\n");

  // Simulate cars entering the lots
  carEnters(&p1, &car1, 7, 15);
  carEnters(&p1, &car2, 7, 25);
  carEnters(&p2, &car3, 8,  0);
  carEnters(&p2, &car4, 8, 10);
  carEnters(&p1, &car5, 8, 15);
  carEnters(&p1, &car6, 8, 20);
  carEnters(&p1, &car7, 8, 30);
  carEnters(&p2, &car7, 8, 32);
  carEnters(&p2, &car8, 8, 50);
  carEnters(&p2, &car9, 8, 55);

  printf("\n");
  printLotInfo(p1);
  printLotInfo(p2);
  printf("\n");

  // Simulate cars leaving the lots
  carLeaves(&p2, &car4, 9, 0);
  carLeaves(&p1, &car2, 9, 5);
  carLeaves(&p1, &car6, 10, 0);
  carLeaves(&p1, &car1, 10, 30);
  carLeaves(&p2, &car8, 13, 0);
  carLeaves(&p2, &car9, 15, 15);
  carEnters(&p1, &car8, 17, 10);
  carLeaves(&p1, &car5, 17, 50);
  carLeaves(&p2, &car7, 18, 0);
  carLeaves(&p2, &car3, 18, 15);
  carLeaves(&p1, &car8, 20, 55);

  printf("\n");
  printLotInfo(p1);
  printLotInfo(p2);
  printf("\n");

  // Display the total revenue
  printf("Total revenue of Lot 1 is $%4.2f\n", p1.revenue);
  printf("Total revenue of Lot 2 is $%4.2f\n", p2.revenue);
}

Приведенный выше код предназначен для назначения.Нам дали прототипы функций и поручили заставить их выполнять определенные действия, связанные со структурой парковки.Манипуляции с данными на самом деле работают как задумано, давая ожидаемый результат для обоих лотов в конце программы.

Это заставляет меня думать, что printf() вызывает проблемы, а не лежащие в основе ценности.Единственное отличие между выделенной функцией printLotInfo() и предварительным оператором printf в initializeLot() состоит в том, что один печатает из struct *, а другой - из структуры.

Я потратил несколько часов на изучение этого, и я наткнулся на множество потоков, показывающих, как печатать из struct * в функции или структуре в main, но я не нашел хорошего примерапечати из структуры в отдельном вызове функции.Я думаю, что упускаю что-то очевидное, например, неправильный оператор разыменования или что-то в этом роде.

Ответы [ 2 ]

0 голосов
/ 22 февраля 2019

Не имеет прямого отношения к вопросу, но с несколькими изменениями программа может стать немного более надежной, а также допустимой программой на C ++.Это облегчает поиск некоторых вещей, которые улучшают программу на Си.

Заставьте Car содержать указатель на постоянные символы.

typedef struct Car
{
    char const *plateNumber;  //String to hold plate
...

То же изменение в интерфейсе initializeCar:

void initializeCar(Car *c, char const *plate, char hasPermit)
{
    Time * t =(Time*) malloc(sizeof(Time));       //Allocate memory for time object

Приведите возвращаемое значение malloc к нужному типу.(См. Выше и ниже.)

void carLeaves(ParkingLot * p, Car * c, int hour, int minute)
{
    Time * leaveTime = (Time*)malloc(sizeof(Time));
    Time * timeDifference = (Time*)malloc(sizeof(Time));        //Allocate memory for time object

И, наконец, подпись main намеренно говорит, что она принимает нулевые аргументы.

int main(void) {

Без void старый интерфейс сказал этопринимает от нуля до неограниченных аргументов.(В C ++ main() уже означает ноль аргументов.)

0 голосов
/ 22 февраля 2019

Вам не хватает аргумента:

printf("Parking Lot #%d - rate = $%.2lf, capacity %d, current cars %d\n",
  p.lotNumber, p.capacity, p.currentCarCount);

Ваша строка формата имеет 4 спецификатора формата, но вы даете только 3 аргумента.В результате второй спецификатор формата %f ищет double, где был передан int, а четвертый ищет int, где нет параметра.Использование неправильного спецификатора формата или отсутствие достаточного количества аргументов вызывает неопределенное поведение .

Использовать необходимость в поле hourlyRate в списке параметров:

printf("Parking Lot #%d - rate = $%.2lf, capacity %d, current cars %d\n",
  p.lotNumber, p.hourlyRate, p.capacity, p.currentCarCount);
...