Как преобразовать точный список страниц из любого в любой [] при назначении его полю страницы? - PullRequest
0 голосов
/ 29 мая 2020
• 1000 существуют в последней строке функции totalNoOfPages в этой строке
this.pageField = this.exactPageList;
this.pageField = 2; not correct

Я ожидал, что будет

this.pageField = [1,2];

, что означает, что мне нужно преобразовать this.exactPageList; в массив, который должен быть принят, назначить полю страницы Как это сделать?

pageField:any[];
exactPageList:any;
totalNoOfPages() {  

    this.paginationData = Number(this.totalReportCount / this.ReportPerPage);  
    console.log("pagination data :" + this.paginationData)
    let tempPageData = this.paginationData.toFixed(); 
    console.log("tempPageData data :" + tempPageData) 
    if (Number(tempPageData) < this.paginationData) {  
      this.exactPageList = Number(tempPageData) + 1;  
      this.paginationService.exactPageList = this.exactPageList;  
      console.log("exactPageList1  data :" + this.exactPageList ) 
    } else {  
      this.exactPageList = Number(tempPageData);  
      this.paginationService.exactPageList = this.exactPageList  
      console.log("exactPageList2  data" + this.exactPageList ) 
    }  
    this.paginationService.pageOnLoad();  
    this.pageField = this.exactPageList;    

  }  

Результат кода выше, как показано ниже:

pagination data1.0666666666666667
reportdetails.component.ts:265 tempPageData data1
reportdetails.component.ts:269 exactPageList1  data2
reportdetails.component.ts:263 pagination data1.0666666666666667
reportdetails.component.ts:265 tempPageData data1
reportdetails.component.ts:269 exactPageList1  data2

Ожидаемый результат

this.pageField = [1,2];

1 Ответ

0 голосов
/ 29 мая 2020

Некоторые пояснения с использованием комментариев:

this.paginationData = Number(this.totalReportCount / this.ReportPerPage);  
console.log("pagination data :" + this.paginationData)
// Use Math.round instead. Then you get an numeral and not a string.
let tempPageData = this.paginationData.toFixed(); 
console.log("tempPageData data :" + tempPageData) 
// Then Number() is not necessary here anymore.
// Moreover as far as I understand, you actually want to use always the 'bigger number'. 
// That means, actually it would make sense to use Math.ceil() above.
// In that case, you don't need to use a condition at all.
if (Number(tempPageData) < this.paginationData) {  
  this.exactPageList = Number(tempPageData) + 1;  
  this.paginationService.exactPageList = this.exactPageList;  
  console.log("exactPageList1  data :" + this.exactPageList ) 
} else {  
  // You are doing almost the same like in the condition clause above.
  // It would be enough to set this.exactPageList condtionally and do the rest afterwards.
  this.exactPageList = Number(tempPageData);  
  this.paginationService.exactPageList = this.exactPageList  
  console.log("exactPageList2  data" + this.exactPageList ) 
}  
this.paginationService.pageOnLoad();  
this.pageField = this.exactPageList; 

Что бы я сделал:

this.paginationData = Number(this.totalReportCount / this.ReportPerPage);  
console.log("pagination data :", this.paginationData)
let tempPageData = Math.ceil(this.paginationData);
console.log("tempPageData data :", tempPageData) 
// Let's create an array from 1 .. N
// [...Array(4).keys()] would create an array with [0,1,2,3]
// but we want to have [1,2,3,4], that's why we are incrementing every value by using map(). 
// Think about moving this to an own method 'getOneToNArray' or so for a better readability.
this.exactPageList = [...Array(tempPageData).keys()].map(x => ++x);;
this.paginationService.exactPageList = this.exactPageList;  
console.log("exactPageList data :", this.exactPageList )   
this.paginationService.pageOnLoad();  
this.pageField = this.exactPageList; 
...