使用 Typescript 和 React.Konva 指定 onClick 事件类型

IT技术 reactjs typescript konvajs
2021-04-03 02:14:04

我试图摆脱我的 tslint 错误,Type declaration of 'any' loses type-safety.但我正在努力弄清楚事件的正确类型是什么。

我正在通过 Lynda “构建和部署全栈 React 应用程序”,同时尝试将其转换为 Typescript。

以下是导致问题的具体线路:

onClick={(event: any) => {
 makeMove(ownMark, event.target.index)
}}

我试图将事件声明为几种不同的类型,例如React.MouseEvent<HTMLElement>,以及 HTMLElement 上的其他一些子类型,但没有成功,因为 target.index 不是我能想到的任何类型的属性。我可以从检查员看到 currentTarget 是 Konva.Text 并且索引设置为0但不确定对我有帮助,因为我无法将类型设置为Konva.Text,这对我来说很有意义,但这也不起作用。

React Konva 事件目标索引

这是我完整的 React 功能组件:

export const Squares = ({units, coordinates, gameState, win, gameOver, yourTurn, ownMark, move}: SquaresProps) => {
  let squares = coordinates.map( (position: number, index: number) => { 
    let makeMove = move
    let mark = gameState[index] !== 'z' ? gameState[index] : false
    let fill = 'black'

    // when someone wins you want the square to turn green
    if (win && win.includes(index)) {
      fill = 'lightGreen'
    }

    if (gameOver || !yourTurn || mark) {
      makeMove = () => console.log('nope!')
    }

    return (
      <Text
        key={index}
        x={position[0]}
        y={position[1]}
        fontSize={units}
        width={units}
        text={mark}
        fill={fill}
        fontFamily={'Helvetica'}
        aligh={'center'}
        onClick={(event: any) => {
          makeMove(ownMark, event.target.index)
        }}
      />
    )
  })

  return (
    <Layer>
      {squares}
    </Layer>
  )
}

这是我的package.json依赖项:

  "dependencies": {
    "konva": "^1.6.3",
    "material-ui": "^0.18.4",
    "react": "^15.6.1",
    "react-dom": "^15.6.1",
    "react-konva": "^1.1.3",
    "react-router": "~3.0.0",
    "react-tap-event-plugin": "^2.0.1",
    "styled-components": "^2.1.0"
  },

认为索引是由 Konva Layer 类添加的,但我对整个 React 生态系统还很陌生,所以仍然试图将我的大脑全部包裹起来。

更新:

我能够使用 Tyler Sebastion 的声明合并建议来定义使 tslint 静音的目标上的索引。我不确定这是最好的方法,因为它对我来说有点脆弱。

这是额外的接口代码和更新的 onclick 事件:

interface KonvaTextEventTarget extends EventTarget {
  index: number
}

interface KonvaMouseEvent extends React.MouseEvent<HTMLElement> {
  target: KonvaTextEventTarget
}

...

return (
  <Text
    key={index}
    x={position[0]}
    y={position[1]}
    fontSize={units}
    width={units}
    text={mark}
    fill={fill}
    fontFamily={'Helvetica'}
    aligh={'center'}
    onClick={(event: KonvaMouseEvent) => {
      makeMove(ownMark, event.target.index)
    }}
  />
)
6个回答

如果没有一些 hack-y 解决方法,您可能会不走运

你可以试试

onClick={(event: React.MouseEvent<HTMLElement>) => {
 makeMove(ownMark, (event.target as any).index)
}}

我不确定你的 linter 有多严格——这可能会让它闭嘴一点

我玩了一会儿,无法弄清楚,但您也可以考虑编写自己的增强定义:https : //www.typescriptlang.org/docs/handbook/declaration-merging.html

编辑:请使用此回复中的实现,解决此问题的正确方法(并在您使用时为他点赞)。

@AlexeySh。他们气馁,是的。我是从 OP 的代码中复制过来的。
2021-05-27 02:14:04
由于影响渲染性能,JSX 属性中禁止使用 Lambda
2021-05-30 02:14:04
为什么您编辑中的回复是正确的?你的方法有什么问题?
2021-06-04 02:14:04
谢谢你的想法。我打算尝试投掷目标,但从未尝试过。我肯定会试一试,尽管这绝对是一种解决方法。我肯定会阅读合并声明,因为我以前从未见过。
2021-06-07 02:14:04
这可以轻松地移入 aReact.useCallback以节省 lambda
2021-06-22 02:14:04

React.MouseEvent 对我有用:

private onClick = (e: React.MouseEvent<HTMLInputElement>) => {
  let button = e.target as HTMLInputElement;
}

正如我在上面的更新中发布的那样,一个潜在的解决方案是使用@Tyler-sebastion 建议的声明合并。我能够定义两个额外的接口并EventTarget以这种方式添加 index 属性

interface KonvaTextEventTarget extends EventTarget {
  index: number
}

interface KonvaMouseEvent extends React.MouseEvent<HTMLElement> {
  target: KonvaTextEventTarget
}

然后我可以KonvaMouseEvent在我的 onclick MouseEventHandler 函数中声明该事件

onClick={(event: KonvaMouseEvent) => {
          makeMove(ownMark, event.target.index)
}}

如果这是最好的方法,我仍然不是 100%,因为它感觉有点笨拙而且过于冗长,只是为了克服 tslint 错误。

您应该使用 event.currentTarget。React 反映了 currentTarget(事件附加到的元素)和 target(事件当前发生的元素)之间的差异。由于这是一个鼠标事件,因此两者在类型上可能不同,即使单击没有意义。

https://github.com/facebook/react/issues/5733 https://developer.mozilla.org/en-US/docs/Web/API/Event/currentTarget

你可以这样做

   onClick: React.MouseEventHandler<HtmlInputElement> = (e) => {
     const button = e.target as HTML
}
您能否详细说明为什么这可以解决问题?
2021-06-06 02:14:04