fix(striker-ui): avoid crash in Memory when returned data is incomplete

main
Tsu-ba-me 3 years ago
parent c413e62798
commit 737c8598d3
  1. 28
      striker-ui/components/Memory.tsx

@ -1,12 +1,13 @@
import { useContext } from 'react'; import { useContext } from 'react';
import { Box } from '@mui/material'; import { Box } from '@mui/material';
import * as prettyBytes from 'pretty-bytes'; import * as prettyBytes from 'pretty-bytes';
import { Panel } from './Panels';
import { AnvilContext } from './AnvilContext';
import { AllocationBar } from './Bars'; import { AllocationBar } from './Bars';
import { HeaderText, BodyText } from './Text'; import { Panel } from './Panels';
import periodicFetch from '../lib/fetchers/periodicFetch'; import periodicFetch from '../lib/fetchers/periodicFetch';
import { AnvilContext } from './AnvilContext';
import Spinner from './Spinner'; import Spinner from './Spinner';
import { HeaderText, BodyText } from './Text';
const Memory = (): JSX.Element => { const Memory = (): JSX.Element => {
const { uuid } = useContext(AnvilContext); const { uuid } = useContext(AnvilContext);
@ -14,46 +15,39 @@ const Memory = (): JSX.Element => {
`${process.env.NEXT_PUBLIC_API_URL}/get_memory?anvil_uuid=${uuid}`, `${process.env.NEXT_PUBLIC_API_URL}/get_memory?anvil_uuid=${uuid}`,
); );
const memoryData = const { allocated = 0, total = 0, reserved = 0 } = data ?? {};
isLoading || !data ? { total: 0, allocated: 0, reserved: 0 } : data;
return ( return (
<Panel> <Panel>
<HeaderText text="Memory" /> <HeaderText text="Memory" />
{!isLoading ? ( {!isLoading ? (
<> <>
{' '}
<Box display="flex" width="100%"> <Box display="flex" width="100%">
<Box flexGrow={1}> <Box flexGrow={1}>
<BodyText <BodyText
text={`Allocated: ${prettyBytes.default(memoryData.allocated, { text={`Allocated: ${prettyBytes.default(allocated, {
binary: true, binary: true,
})}`} })}`}
/> />
</Box> </Box>
<Box> <Box>
<BodyText <BodyText
text={`Free: ${prettyBytes.default( text={`Free: ${prettyBytes.default(total - allocated, {
memoryData.total - memoryData.allocated,
{
binary: true, binary: true,
}, })}`}
)}`}
/> />
</Box> </Box>
</Box> </Box>
<Box display="flex" width="100%"> <Box display="flex" width="100%">
<Box flexGrow={1}> <Box flexGrow={1}>
<AllocationBar <AllocationBar allocated={(allocated / total) * 100} />
allocated={(memoryData.allocated / memoryData.total) * 100}
/>
</Box> </Box>
</Box> </Box>
<Box display="flex" justifyContent="center" width="100%"> <Box display="flex" justifyContent="center" width="100%">
<BodyText <BodyText
text={`Total: ${prettyBytes.default(memoryData.total, { text={`Total: ${prettyBytes.default(total, {
binary: true, binary: true,
})} | Reserved: ${prettyBytes.default(memoryData.reserved, { })} | Reserved: ${prettyBytes.default(reserved, {
binary: true, binary: true,
})}`} })}`}
/> />

Loading…
Cancel
Save